Message47517
Logged In: YES
user_id=1174944
This patch will be welcomed by all of that have had to write
"datetime(*(time.strptime(date_string, format)[0:6]))".
I don't understand the C API well enough to check if
reference counts are handled properly, but otherwise the
implementation looks straight forward.
Documentation looks good and the test passes on my machine.
Two suggestions:
1. In the time module, the strptime() function's format
parameter is optional. For consistency's sake, I'd expect
datetime.strptime()'s format parameter also to be optional.
(On the other hand, the default value for the format is not
very useful.)
2. Since strftime is supported by datetime.time,
datetime.date and datetime.datetime, I'd also expect
strptime to be supported by all three classes. Could you add
that now, or would it be better to do it as a separate patch? |
|
Date |
User |
Action |
Args |
2007-08-23 15:41:15 | admin | link | issue1100942 messages |
2007-08-23 15:41:15 | admin | create | |
|