Hows this for a patch:

package Date::MMDDYY;

use strict;
use vars qw(@ISA @EXPORT);

use Carp;
use Exporter;

@ISA = qw(Exporter);
@EXPORT = qw(datecon);

sub datecon {
        croak "Date::MMDDYY has been deprecated in favour of POSIX::strftime";
}

1;


Matt

At 17:16 20/03/01 +0000, you wrote:
>On Tue, Mar 20, 2001 at 05:07:28PM +0000, Leon Brocard wrote:
>> This isn't such a crazy idea. People keep on complaining about the
>> quality of modules on CPAN. So pick a random one and make it better
>> ;-P
>
>Well, with a module like Date::MMDDYY the implementation
>_is_ broken - it uses gmtime() instead of localtime() for
>example;
>
>but worse than that, the design and conception are flawed.
>Any drop-in replacement would inevitably suffer from the
>same flaws of conception and interface.
>
>There's no reason at all for anybody to use this module.
>Compare:
>
>  use Date::MMDDYY 'datecon';
>  print "The date is ", datecon(time()), "\n";
>
>to
>
>  use POSIX 'strftime';
>  print "The date is ", strftime("%m-%d-%y", localtime()), "\n";
>
>
>The module is redundant, not just poorly implemented.
>
> .robin.
>
>-- 
>A man, a plan, a cat, a ham, a yak, a yam, a hat, a canal--Panama!
>
>

Reply via email to