This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [v4] Fix strptime era handling, add more era tests [BZ #24394]
- From: Carlos O'Donell <codonell at redhat dot com>
- To: Florian Weimer <fw at deneb dot enyo dot de>
- Cc: DJ Delorie <dj at redhat dot com>, TAMUKI Shoichi <tamuki at linet dot gr dot jp>, digitalfreak at lingonborough dot com, libc-alpha at sourceware dot org
- Date: Mon, 1 Apr 2019 16:36:35 -0400
- Subject: Re: [v4] Fix strptime era handling, add more era tests [BZ #24394]
- References: <xn5zs1pnol.fsf@greed.delorie.com> <65a2cd8a-525f-3bae-364d-8cd11a256752@redhat.com> <87ef6m6eaw.fsf@mid.deneb.enyo.de>
On 3/31/19 3:20 PM, Florian Weimer wrote:
* Carlos O'Donell:
.. and tst-strftime2.c is hideous, a terrible test with no comments
whose logic cannot be followed.
I think it's not too bad. It's one of these combination tests. I
can't read Japanese, so I have trouble understanding what it exactly
tests, but any test trying to get coverage like this will looke pretty
similar.
The lack of comments makes the test difficult to maintain.
My objection is not to combinatorial tests that auto-generate the testing
matrix, in fact you've shown that such tests are possible with good comments :-)
My usual review of these tests begins with: "Please add more comments."
You eventually forget why the test generates the permutation in this way or that.
I also support the heavy use of enums to make the test intent perfectly clear.
--
Cheers,
Carlos.