This is the mail archive of the 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]

Re: [PATCH 4/5] Refactor tst-strtod-round.c

On 05/17/2016 11:14 AM, Joseph Myers wrote:
> On Tue, 17 May 2016, Paul E. Murphy wrote:
>>> So far this message has reached the libc-alpha list twice (you don't see 
>>> it twice in the archives because it has the same message-id each time).  
>>> I've seen this issue before with large messages coming to libc-alpha from 
>>> IBM (many duplicate copies of them arrive on the list), as if there is 
>>> some problem with IBM's mail server timing out before sourceware accepts 
>>> the message, or something like that.  Please try to get that mail system 
>>> problem fixed.
>> Ugh.  Thanks for pointing that out.  That is annoying.  I'll try.
> It's *also* a good idea to omit large generated file changes from patch 
> postings, unless they are important for the review.  (When constant 
> suffixes are eliminated from auto-libm-test-out - desirable for the same 
> reasons as for the present patch - that will rewrite the whole file - a > 
> 60MB diff - and you certainly don't want to send that to the list.)

Admittedly, my cutoff for "too big" was still too big.  Lesson learned.

Anyhow, for those tests, we do have the opportunity to post-process
the content as part of building the tests.  In that (yet to be submitted
to libc-alpha) patchset, I updated to strip C99 literal
suffixes, then wrap them with a new macro named LIT() to apply the
appropriate suffix depending on the type under test. With the exception
of nexttoward and its long double second argument.

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