Message ID | 1417646760-19563-1-git-send-email-james@loowit.net |
---|---|
State | New |
Headers | show |
On 03 Dec 2014 14:45, James Perkins wrote: > * Two of the new offsets fail, showing that strptime incorrectly > calculates the seconds from minutes not divisible by 3 (+1158, +1159). these are good datasets to have as they show the bug exists before your fix, but generally we don't add tests to the tree that fail right away. for example, this makes bisecting harder. instead you should merge the test updates with the code fix so there is only one patch. > 2014-12-03 James Perkins james@loowit.net your email address should have angle brackets like <foo@com> > * time/strptime2.c (tests[]): Add tests for the strptime %z when fixing bugzilla bugs, the ChangeLog should have a line above it here: [BZ #12345] when scoping the changelog entry, we just use the name, not its type. i.e. drop the [] so you have "(tests)". also, your subject line seems to have a typo -- you want #16141, not #14161. -mike
diff --git a/time/tst-strptime2.c b/time/tst-strptime2.c index bcd6cc8..5f9860a 100644 --- a/time/tst-strptime2.c +++ b/time/tst-strptime2.c @@ -17,6 +17,13 @@ static const struct { "1113472456 -1030", -37800 }, { "1113472456 +0030", 1800 }, { "1113472456 -0030", -1800 }, + { "1113472456 +1157", 43020 }, + { "1113472456 +1158", 43080 }, + { "1113472456 +1159", 43140 }, + { "1113472456 +1200", 43200 }, + { "1113472456 -1200", -43200 }, + { "1113472456 +1201", LONG_MAX }, + { "1113472456 -1201", LONG_MAX }, { "1113472456 -1330", LONG_MAX }, { "1113472456 +1330", LONG_MAX }, { "1113472456 -1060", LONG_MAX },