Georg Brandl <ge...@python.org> added the comment:

The patch is wrong: it hardcodes the number of characters that the time string 
has, but it can be more than 24 if the year is > 9999.  (Of course, the check 
for \n currently in the code is wrong too and must be fixed.)

Also, shouldn't the issue be handled as in ctime()?  There is a NULL check 
there, and by just doing that check we wouldn't depend on asctime_r().

----------
assignee:  -> belopolsky
nosy: +belopolsky, georg.brandl

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue8013>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to