This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[PING^2] [PATCH v3] [BZ #14161] Fix strptime %z calculation, extend ranges


Hello,

The v3 version of my patches I posted Dec 03 have addressed all
concerns raised. As I am not a committer, a primary maintainer
will need to merge them for release. Changes include:

1) Add test cases to verify proper conversion of minutes, showing
   failure of existing code.
2) Fix strptime %z minutes calculation, causing tests to pass.
3) Add test cases to verify that existing timezone offsets (-1200
   through +1400), historical offsets, POSIX ranges (-2359 to
   -2359), and largest possible offset (-9959 to +9959) are
   correctly converted; all except -1200 fail in existing code.
4) Fix strptime %z range limits by extending to -9959 to +9959,
   causing tests to pass.

For reference, the patches are in the mailing list archive at
the URLs listed below:

[PATCH v3 1/4] [BZ #14161] tst-strptime2.c: add %z minute and range limit tests
   https://sourceware.org/ml/libc-alpha/2014-12/msg00097.html

[PATCH v3 2/4] [BZ #16141] strptime: fix %z minutes calculation
   https://sourceware.org/ml/libc-alpha/2014-12/msg00100.html

[PATCH v3 3/4] [BZ #14161] tst-strptime2.c: test %z range to +/-9959
   https://sourceware.org/ml/libc-alpha/2014-12/msg00098.html

[PATCH v3 4/4] [BZ #16141] strptime: extend %z range to +/-9959
   https://sourceware.org/ml/libc-alpha/2014-12/msg00099.html

Thanks,
James
--
James Perkins <james@loowit.net>        www.loowit.net/~james


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]