<br style="font-family: courier new,monospace;"><br style="font-family: courier new,monospace;"><span style="font-family: courier new,monospace;">On Fri, Mar 18, 2011 at 11:47 AM, Shlomi Fish <<a href="mailto:shlomif@iglu.org.il">shlomif@iglu.org.il</a>> wrote:</span><br style="font-family: courier new,monospace;">

<span style="font-family: courier new,monospace;">> Hi,</span><br style="font-family: courier new,monospace;"><span style="font-family: courier new,monospace;">></span><br style="font-family: courier new,monospace;">

<span style="font-family: courier new,monospace;">> a few comments on your code.</span><br style="font-family: courier new,monospace;"><span style="font-family: courier new,monospace;">></span><br style="font-family: courier new,monospace;">

<span style="font-family: courier new,monospace;">> On Friday 18 Mar 2011 16:42:27 Fulko Hew wrote:</span><br style="font-family: courier new,monospace;"><span style="font-family: courier new,monospace;">>> I have the following (simplified, and artificial) scenario:</span><br style="font-family: courier new,monospace;">

<span style="font-family: courier new,monospace;">>></span><br style="font-family: courier new,monospace;"><span style="font-family: courier new,monospace;">>> package testClass;</span><br style="font-family: courier new,monospace;">

<span style="font-family: courier new,monospace;">></span><br style="font-family: courier new,monospace;"><span style="font-family: courier new,monospace;">> 1. A package should always start with an uppercase letter.</span><br style="font-family: courier new,monospace;">

<span style="font-family: courier new,monospace;">></span><br style="font-family: courier new,monospace;"><span style="font-family: courier new,monospace;">> 2. You should add "use strict;" and "use warnings;" somewhere.</span><br style="font-family: courier new,monospace;">

<br style="font-family: courier new,monospace;"><span style="font-family: courier new,monospace;">I _did_ say it was an artificial scenario!</span><br style="font-family: courier new,monospace;"><br style="font-family: courier new,monospace;">

<span style="font-family: courier new,monospace;">The code I was showing was an interpretation of code<br>that is actually programatically generated by my tool,<br>but... I did take</span><span style="font-family: courier new,monospace;"> your advise.</span><br style="font-family: courier new,monospace;">

<br style="font-family: courier new,monospace;"><span style="font-family: courier new,monospace;">a) I added the strict and warning to the code generator's output,</span><br style="font-family: courier new,monospace;">

<span style="font-family: courier new,monospace;">    and although the generated code worked before, It did uncover<br>   some issue that</span><span style="font-family: courier new,monospace;"> makes my code generator output better<br>

   ('more correct').</span><br style="font-family: courier new,monospace;"><br style="font-family: courier new,monospace;"><span style="font-family: courier new,monospace;">b) The package names generated are actually derived from the name<br>

   of a 'thing'</span><span style="font-family: courier new,monospace;"> in the source file that the code generator consumes.<br><br>   I wanted</span><span style="font-family: courier new,monospace;"> the package name to be directly compatible with the<br>

   source</span><span style="font-family: courier new,monospace;"> so that it was more intuitive to the user (relate-able).<br><br>   After reading words</span><span style="font-family: courier new,monospace;"> around the naming convention on packages,<br>

   I decided that</span><span style="font-family: courier new,monospace;"> Perl pragma safety was more important than<br>   end-user intuitiveness, and so I now up-case the first letter<br>   of the input to generate the package name.<br>

<br>Fulko<br><br></span>