[XML-SIG] Re: [XML-checkins]CVS: xml/xml/utils iso8601.py,1.6,1.7

Alexandre Alexandre.Fayolle@logilab.fr
Fri, 19 Apr 2002 08:51:45 +0200


On Thu, Apr 18, 2002 at 07:31:27PM -0700, Fred L. Drake wrote:
       if seconds:
>           seconds = float(seconds)
> !         if not 0 <= seconds <= 60:
>               raise ValueError, "illegal seconds number: " + m.group("seconds")

Now, this may be dumb, since I'm not very familiar with the intricacies
of real date and time manipulations, but is 60 an allowed value for
seconds? In other words, should not this read 

if not 0 <= seconds < 60: 


Alexandre "just asking" Fayolle
-- 
LOGILAB, Paris (France).
http://www.logilab.com   http://www.logilab.fr  http://www.logilab.org
Narval, the first software agent available as free software (GPL).