I must say however, that I was quite put off early into this process, when I found the following in the core P5EEx::Blue::P5EE module:
- # eliminate warnings about uninitialized values $SIG{__WARN__} = sub { warn @_ unless $_[0] =~ /Use of uninitialized v +alue/};
This line, trapping the __WARN__ signal and preventing the output of warnings where undefined variables are being used, seems in direct contrast the the concept of developing proven and complete code for the enterprise environment. Furthermore, the use of this code is mentioned in the P5EE perlstyle.pod, but gives no explanation for its use. eg.
- Warnings and Strict All code must compile and run cleanly with "use strict" enabled and the perl "-w" (warnings) option on. If you must do something that -w or strict complains about, there are workarounds, but the chances that you really need to do it that way are remote. The one exception is the "Use of uninitialized variable" warnings. We have those disabled in P5EE.pm, so by including "use P5EE" you are disabling that warning in your code, too, and you don't need to worry about them.
To my mind such errors represent poor initiation of code variables or failed trapping of return values from functions - Aspects of coding which should not be ignored, especially within a production environment.
Further to this, many of the modules in place within the P5EEx development tree are only placeholders at this stage, outlining what it is hoped that they will eventually perform, but with little in the way of firm code.
Please don't misinterpret this review as an attack on the P5EE project - I think this is an excellent direction for development and have subscribed to the mailing list to get up to date with the development. However at this point in time, this project is still quite embryonic and there are style and development issues which still need to be addressed. Perhaps, it can be said that the ulterior purpose behind this node, besides that stated to comment on P5EE progress, is to encourage other monks to contribute on this project - There is much which needs to be done and much that we can do to contribute.
perl -e 's&&rob@cowsnet.com.au&&&split/[@.]/&&s&.com.&_&&&print'
|
---|
Replies are listed 'Best First'. | |
---|---|
Re (tilly) 1: P5EE ... get involved!
by tilly (Archbishop) on Feb 03, 2002 at 18:53 UTC | |
by sintadil (Pilgrim) on Aug 30, 2004 at 23:44 UTC | |
Re: P5EE ... get involved!
by trs80 (Priest) on Feb 03, 2002 at 15:51 UTC | |
Re: P5EE ... get involved!
by coreolyn (Parson) on Feb 03, 2002 at 14:47 UTC | |
by Anonymous Monk on Feb 05, 2002 at 16:01 UTC | |
by coreolyn (Parson) on Feb 05, 2002 at 16:41 UTC | |
Re: P5EE ... get involved!
by gav^ (Curate) on Feb 03, 2002 at 07:07 UTC | |
by rob_au (Abbot) on Feb 03, 2002 at 07:13 UTC | |
by koolade (Pilgrim) on Feb 04, 2002 at 22:48 UTC |