[PATCH RFC] explicit_bzero, again

[Zack Weinberg]

On Mon, Aug 10, 2015 at 10:40 AM, Joseph Myers <joseph@codesourcery.com> wrote:
> On Sun, 9 Aug 2015, Zack Weinberg wrote:
>
>>       * sysdeps/unix/sysv/linux/powerpc/powerpc32/fpu/libc.abilist
>>       * sysdeps/unix/sysv/linux/powerpc/powerpc32/nofpu/libc.abilist
>>       * sysdeps/unix/sysv/linux/powerpc/powerpc64/libc.abilist
>
> sysdeps/unix/sysv/linux/powerpc/powerpc64/libc-le.abilist also needs
> updating.

Did that just get added, like, yesterday?  I am not sure why `find`
didn't find it.

>> +libc_hidden_def(__memset_chk)
>
> Missing space before '('.
[etc]

Thanks, I'm a bit out of practice coding in GNU style.

> I think you should use __may_alias__ here (there's no reason to take the
> symbol may_alias from the user's namespace, notwithstanding that this is
> __USE_MISC).

I thought attribute names didn't conflict with any other namespace?
But sure, I can make that change.

> Stray patch hunk adding a blank line.

That was actually intentional -- this file seems to separate groups of
functions with double blank lines.  But I can take it back out again.

Does anyone have more substantive concerns with this patch, or are we
good to go once I fix up the style?

zw