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] remove nested functions from regcomp.c

On 09/30/2014 04:13 AM, Will Newton wrote:
> On 29 September 2014 21:54, Konstantin Serebryany
> <> wrote:
>> Hi,
>> Please review the patch that removes nested functions from posix/regcomp.c.
>> The patch does not noticeably affect the generated code (same
>> instructions, a few differences in used registers, offsets, etc)
>> because all the affected functions have __attribute ((always_inline)).
>> No regressions in 'make check' on x86_64-linux-gnu (Ubuntu 14.04)
>> This code has some #ifdef branches for "not _LIBC",
>> please let me know if any separate testing is required for that case.
> This file is shared with gnulib so ideally the changes should be OKed
> there too. Unfortunately this particular file seems to have diverged a
> bit from gnulib so that might not be that straightforward.

The first step is to synchronize this file with gnulib.

This should be looked at first before considering any further changes,
however, Konstantin need not do this work himself, and we can help.

Lastly, we want *certainty* in our statements about compiler warnings
being wrong. A thorough reasoning needs to be brought forward to say
why the compiler warning is wrong. A hand-waving argument doesn't cut it.


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