[issue12006] strptime should implement %G, %V and %u directives

Ashley Anderson report at bugs.python.org
Mon Aug 3 05:28:19 CEST 2015


Ashley Anderson added the comment:

Thanks for the review and the good suggestions. Hopefully this new patch is an improvement.

I didn't know about the context manager for assertRaises - I was just following the format for another ValueError test a few lines above.

The frozenset and re-wrapped comment were left from playing around with another way to do the checks, and I've corrected them.

I think the conditionals around calculating the julian and year are clearer now as well.

Please (obviously) let me know if there are further changes. Also please let me know if this is not the proper way to respond to the code review!

----------
Added file: http://bugs.python.org/file40113/issue12006_8_complete.patch

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue12006>
_______________________________________


More information about the Python-bugs-list mailing list