[COMMIT LOGREPORT] service/all/lib Time.pm,1.28,1.29

[Egon Willighagen]

On Monday 22 October 2001 23:45, Joost van Baal wrote:
> On Sat, Oct 20, 2001 at 02:38:21AM -0700, Egon Willighagen wrote:
> > Update of /cvsroot/logreport/service/all/lib
> > In directory usw-pr-cvs1:/tmp/cvs-serv16025/all/lib
> >
> > Modified Files:
> > 	Time.pm
> > Log Message:
> > New "print" superservice with CUPS page_log plugin
> >
> > Index: Time.pm
> > ===================================================================
> > RCS file: /cvsroot/logreport/service/all/lib/Time.pm,v
> > retrieving revision 1.28
> > retrieving revision 1.29
>
> <snip>
>
> > +
> > +# convert [18/Mar/2001:15:59:30 +0100] to seconds since epoch, utc
> > +#
> > +sub clf2cal
> > +{
> > +    my $sub = 'clf2cal';
>
> This function is in www/lib/Apache.pm also.  Wouldn't it be better to
> have it in one module only?

Yes, you are right.
I also think the Time module need to be documented more...

Egon

-- 
To UNSUBSCRIBE, email to development-request at logreport.org with a subject of
"unsubscribe". Trouble? Send an email with subject "help" to
development-request at logreport.org