This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author terry.reedy
Recipients terry.reedy, thatiparthy
Date 2013-11-16.01:51:43
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1384566704.45.0.566084227718.issue19562@psf.upfronthosting.co.za>
In-reply-to
Content
Looking further, the current code has a message object, the month that fails the test and your patch removes that in adding the redundant message. I also see that your change would make the first assert match the next 2. But I would rather change the next two.

Sequences like
_DI4Y   = _days_before_year(5) 
# A 4-year cycle has an extra leap day over what we'd get from pasting
# together 4 single years.
assert _DI4Y == 4 * 365 + 1

are bizarre. The constant should be directly set to 4*365 + 1 and then _days_before_year(5) == _DI4Y tested in test_datetime.
History
Date User Action Args
2013-11-16 01:51:44terry.reedysetrecipients: + terry.reedy, thatiparthy
2013-11-16 01:51:44terry.reedysetmessageid: <1384566704.45.0.566084227718.issue19562@psf.upfronthosting.co.za>
2013-11-16 01:51:44terry.reedylinkissue19562 messages
2013-11-16 01:51:43terry.reedycreate