Baptiste Mispelon added the comment:

I updated the patch based on Raymond's feedback.

I don't know C at all and I tried to mimic the namedtuple usage of timemodule.c 
as much as I could (until the code compiled and the test suite passed).

I still have two questions:

* It seems that the Python implementation (Lib/datetime.py) is not tested when 
I run `./python -m test test_datetime` (I introduced a deliberate error in 
datetime.isocalendar() there but the test still succeeded). Is there a 
particular procedure to trigger those tests?

* I'm currently using the name `IsoCalendarDate` as suggested by user taleinat. 
However, shouldn't it be `ISOCalendarDate`?

Thanks.

----------
Added file: http://bugs.python.org/file44750/issue24416_5.diff

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

Reply via email to