This is the mail archive of the libc-alpha@sourceware.org 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: [GLIBC][PATCH v2] Remove strdup inlines


On Thu, 9 Feb 2017, Adhemerval Zanella wrote:

> Also for strdup I think __USE_XOPEN2K8 is not required, accordingly to manual
> it is defined for
> 
>  _SVID_SOURCE || _BSD_SOURCE || _XOPEN_SOURCE >= 500 || _XOPEN_SOURCE && _XOPEN_SOURCE_EXTENDED

strdup was XSI-shaded in the 2001 edition of POSIX but moved to CX-shaded 
in the 2008 edition.  That is, it should be declared for 
_POSIX_C_SOURCE=200809L.  That is, the existing __USE_XOPEN2K8 condition 
(which means non-XSI 2008 edition of POSIX) is right.

> And __USE_XOPEN_EXTENDED will be set for _XOPEN_SOURCE >= 500 or if
> _XOPEN_SOURCE_EXTENDED is defined.  At least all required defined are
> suffice on conform testcases for just 

This indicates you've found a bug in the conform/ expectations - strdup 
should be required there for POSIX2008.

-- 
Joseph S. Myers
joseph@codesourcery.com


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