At Tue, 20 Mar 2001 11:18:47 -0500 (EST), Dave Cross <[EMAIL PROTECTED]> wrote:
> 
> Take a look at this <http://search.cpan.org/search?dist=Date-MMDDYY>.
> 
> Now give me:
> 
> a) a two reasons why this module should never have been written, and
> b) as many flaws as possible in the implementation.

I think this is my favourite bit:

$temp_time = gmtime($_[0]);

and then later:

@time_array = split(/\s+/, $temp_time);

$temp_day = $time_array[2];
$temp_month = $time_array[1];
$temp_year = $time_array[4];

And the whole $delem (because it holds the 'delemiter') thing cracks
me up.

I really think I should drop the author a polite note offering him a 
patch or three.

Dave...

Reply via email to