[PATCH] Rework handling of basename variants

Corinna Vinschen vinschen@redhat.com
Wed Jul 8 15:05:00 GMT 2015


On Jul  6 13:10, Yaakov Selkowitz wrote:
> As a commonly-included header, the #define basename in <string.h> can
> affect code which uses "basename" for its own purposes (e.g. struct
> members or C++ namespaced functions).  When such cases occur and some
> code includes <string.h> and some not, then errors result.  OTOH,
> <libgen.h> is rarely used, and that's where the renaming occurs in
> glibc, so code using <libgen.h> should already be safe.
> ---
>  newlib/libc/include/libgen.h | 5 +++--
>  newlib/libc/include/string.h | 4 ++--
>  2 files changed, 5 insertions(+), 4 deletions(-)

Apart from the missing ChangeLog entry, the patch is ok.


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/newlib/attachments/20150708/bd4c05e1/attachment.sig>


More information about the Newlib mailing list