[Patches] [ python-Patches-663482 ] 658254: accept None for time.ctime() and friends

SourceForge.net noreply at sourceforge.net
Fri Jul 2 03:04:35 EDT 2004


Patches item #663482, was opened at 2003-01-06 22:42
Message generated for change (Comment added) made by rhettinger
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=663482&group_id=5470

Category: Modules
Group: Python 2.4
Status: Open
Resolution: None
Priority: 5
Submitted By: Christopher Blunck (blunck2)
>Assigned to: Fred L. Drake, Jr. (fdrake)
Summary: 658254: accept None for time.ctime() and friends

Initial Comment:
ctime(), gmtime(), and localtime() now support None and
treat it as tho you passed no arguments at all.

i am new to patching for py and am not sure if there
are unit tests i should update.  please advise...

----------------------------------------------------------------------

>Comment By: Raymond Hettinger (rhettinger)
Date: 2004-07-02 02:04

Message:
Logged In: YES 
user_id=80475

Fred, here is Blunk's patch.

----------------------------------------------------------------------

Comment By: Brett Cannon (bcannon)
Date: 2003-07-06 15:38

Message:
Logged In: YES 
user_id=357491

What do people think of this functionality?  I personally don't see a 
need for it.

----------------------------------------------------------------------

Comment By: Neal Norwitz (nnorwitz)
Date: 2003-01-07 17:33

Message:
Logged In: YES 
user_id=33168

There don't appear to be any direct tests, but it would
still be nice to call the functions and verify they work ok.
 You could compare that abs(ctime() - ctime(None)) < .5, for
example.  Look in Lib/test/test_time.py.

Also, the docs need to be updated, see Doc/lib/libtime.tex.
 That should be updated, ie change \optional(secs) with
\optional{secs\code{ = None}}.  Also, an entry in Misc/NEWS
should be added since this is an API change.

There is no attached file.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=663482&group_id=5470



More information about the Patches mailing list