Mathieu Dupuy added the comment: > Hum, you should use the same rounding method than datetime.datetime.fromtimestamp(): ROUND_HALF_UP, as round(). In practice, you can for example pass a floating point number as microseconds to datetime.datetime constructor.
Unfortunately, you're mistaking with the timedelta constructor. Datetime's one only take int :( But I figured out an elegant manner to cope with (in my opinion) > Since datetime is implemented in C, I'm not sure that using the re is the best choice. Since the regex looks simple enough, we may parse the string without the re module. Well, maybe only for the C implementation. No regex available at all in CPython ? Otherwise, yeah, if I have to, I can do it with strptime. > What is the behaviour is there are spaces before/after the string? What if there are other characters like letters before/after? You should add an unit test for that. I expect an error when parsing "t=2012-04-23T09:15:00" for example. Your regex ends with $ but doesn't start with ^. Using re.match(), ^ and $ are probably not needed, but I'm not confident when I use regex :-) re.match only look at the beginning of the string, so no need for '^'. And therefore, the case you mention is already handled :) joined to this mail the last revision of the feature, with correct rounding, more test and one useless line removed. Maybe the good one :) ? ---------- Added file: http://bugs.python.org/file41927/fromisoformat4.patch _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue15873> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
