Re: musl compatibility

2012-06-08 Thread Pedro Alves
compliant, so that gnulib's fallback doesn't even get built. It seems that nobody who is interested in musl has looked at gnulib's config.log to understand why does gnulib think musl's printf is not good enough. -- Pedro Alves

Re: [PATCH] glibc 64-bit obstack support

2014-07-29 Thread Pedro Alves
on this. https://sourceware.org/bugzilla/show_bug.cgi?id=14483 https://bugzilla.redhat.com/show_bug.cgi?id=231832 -- Thanks, Pedro Alves

Re: Cross-build issues in gdb/gnulib

2015-08-25 Thread Pedro Alves
On 08/25/2015 06:06 PM, paul_kon...@dell.com wrote: On Aug 25, 2015, at 12:58 PM, Paul Eggert egg...@cs.ucla.edu wrote: Pedro Alves wrote: # Test for the AIX locale name. if (LC_ALL=ja_JP LC_TIME= LC_CTYPE= ./conftest; exit) 2/dev/null; then gt_cv_locale_ja

[PATCH] Re: stdint.h, C++ and __STDC_LIMIT_MACROS

2015-11-02 Thread Pedro Alves
On 11/02/2015 11:03 AM, Pedro Alves wrote: > On 11/02/2015 10:36 AM, Pedro Alves wrote: > >> $ grep -rn gl_cv_header_working_stdint_h >> config.cache:628:gl_cv_header_working_stdint_h=${gl_cv_header_working_stdint_h=no} >> config.log:39493:gl_cv_header_working_stdint_h=n

stdint.h, C++ and __STDC_LIMIT_MACROS

2015-11-01 Thread Pedro Alves
X_MAX. " I can work around this by defining __STDC_LIMIT_MACROS in gdb before including stdint.h, but it sounds like I shouldn't need to. Thus I wonder what am I missing. I'm attaching a gzip'ed config.log. Thanks, Pedro Alves config.log.gz Description: application/gzip

Re: stdint.h, C++ and __STDC_LIMIT_MACROS

2015-11-02 Thread Pedro Alves
On 11/02/2015 10:36 AM, Pedro Alves wrote: > $ grep -rn gl_cv_header_working_stdint_h > config.cache:628:gl_cv_header_working_stdint_h=${gl_cv_header_working_stdint_h=no} > config.log:39493:gl_cv_header_working_stdint_h=no > > But, I don't get a replacement: > > $

Re: stdint.h, C++ and __STDC_LIMIT_MACROS

2015-11-02 Thread Pedro Alves
On 11/02/2015 01:36 AM, Paul Eggert wrote: > Pedro Alves wrote: >> and gnulib is deciding that the system's stdint.h is good enough >> At least, I don't get a stdint.h replacement. > > Since gnulib's test program contains this: > > intmax_t i = INTMAX_MAX; > >

Re: [PATCH] Re: stdint.h, C++ and __STDC_LIMIT_MACROS

2015-11-03 Thread Pedro Alves
On 11/03/2015 02:45 PM, Pedro Alves wrote: > On 11/03/2015 02:32 AM, Paul Eggert wrote: >> Pedro Alves wrote: >>> Instead of trying to detect the right types, detect >>> good-enough-pre-C++11 stdint.h and in such case make the substitute >>> stdi

Re: Cross-build issues in gdb/gnulib

2015-08-25 Thread Pedro Alves
Adding gnulib@ Original thread here: https://sourceware.org/ml/gdb/2015-08/msg00037.html Thanks, Pedro Alves On 08/25/2015 02:12 PM, paul_kon...@dell.com wrote: On Aug 25, 2015, at 5:56 AM, Pedro Alves pal...@redhat.com wrote: On 08/24/2015 10:20 PM, paul_kon...@dell.com wrote: I'm

[PATCH] Avoid having GNULIB_NAMESPACE::func always inject references to rpl_func

2016-11-12 Thread Pedro Alves
In any case, ending up with strong references to replacement functions even if they're not used means that binaries end up containing gnulib code they don't really use. So this seems like a good change even not considering the GDB's IPA. ChangeLog: 2016-11-12 Pedro Alves <pal...@redhat.

[PATCH] Fix gnulib C++ namespace support and std::frexp

2016-11-12 Thread Pedro Alves
std::frexp(float, int*)’ frexp(float __x, int* __exp) ^ ChangeLog: 2016-11-12 Pedro Alves <pe...@redhat.com> Fix gnulib C++ namespace support and std::frexp * lib/math.in.h (frexp): Use _GL_CXXALIASWARN1 instead of _GL_CXXALIASWARN. --- lib/math.in

[PATCH] C++: "#define timeval rpl_timeval" -> typedef in GNULIB_NAMESPACE

2016-11-14 Thread Pedro Alves
ame time tzset against gcc 4.7, 4.8, 4.9, 5.3, and 7/trunk on Fedora 23. 2016-11-15 Pedro Alves <pal...@redhat.com> * lib/sys_time.in.h [__cplusplus && defined GNULIB_NAMESPACE]: Define "timeval" in the GNULIB_NAMESPACE namespace, and #undef any t

[PATCH] Fix real-floating argument functions in C++ mode

2016-11-14 Thread Pedro Alves
st # gcc 5.3.1 on Fedora 23 CC="gcc -Wall" CXX="g++ -std=gnu++03 -Wall -Wno-unused-variable" $test CC="gcc -Wall" CXX="g++ -std=gnu++11 -Wall -Wno-unused-variable" $test # gcc trunk on Fedora 23 PREFIX="/opt/gcc/bin" CC="$PREFIX/gcc -Wall"

Re: [PATCH] Avoid having GNULIB_NAMESPACE::func always inject references to rpl_func

2016-11-21 Thread Pedro Alves
y-long lines? inline operator type () const { return reinterpret_cast((rettype2 (*) parameters2)(::func)); } Likewise the other similar cases. Thanks, Pedro Alves

Re: [PATCH] Avoid having GNULIB_NAMESPACE::func always inject references to rpl_func

2016-11-21 Thread Pedro Alves
On 11/21/2016 11:27 PM, Bruno Haible wrote: > Pedro Alves wrote: >> How about breaking this line like the rpl method was breaking it, in order >> to avoid overly-long lines? >> >>inline operator type () const >>{ return reinterpret_cast((ret

Re: status of C++ support with GNULIB_NAMESPACE

2016-11-21 Thread Pedro Alves
piler, but ended up dismissing it as both impractical, and unnecessary. BTW, do we know which programs use GNULIB_NAMESPACE? I believe the initial support was done for GNU Octave, but I see that quite recently Octave switched away from it, maybe because of the problems with newer GCCs (I believe fixed now). Thanks, Pedro Alves

Re: [PATCH] Fix real-floating argument functions in C++ mode

2016-11-14 Thread Pedro Alves
On 11/14/2016 09:19 PM, Pedro Alves wrote: > Let me try that and send a new patch. Here's an simpler implementation, IMO. I went ahead and added the _GL_BEGIN_NAMESPACE/_GL_END_NAMESPACE macros to c++defs.h as I'm fairly sure they'll end up being useful in more places. Passes same test

Re: [PATCH] Fix real-floating argument functions in C++ mode

2016-11-14 Thread Pedro Alves
On 11/14/2016 06:16 PM, Pedro Alves wrote: > On 11/12/2016 05:30 PM, Paul Eggert wrote: >> Thanks, I installed your two recent patches. > > Thanks! > >> I don't know C++ well; >> perhaps Bruno or another reviewer can double-check if they have the time. >>

Re: C++ aliases in

2016-12-16 Thread Pedro Alves
func arguments;\ } \ But, then you'd need to tweak the _GL_CXXALIAS_SYS macro too, to pass down the right "arguments", since that "arguments" doesn't exist today. Like: -_GL_CXXALIAS_SYS (freeaddrinfo, void, (struct addrinfo *ai)) +_GL_CXXALIAS_SYS (freeaddrinfo, void, (struct addrinfo *ai), (ai)) Thanks, Pedro Alves

Re: C++ aliases in

2016-12-16 Thread Pedro Alves
? Note there's the system struct timeval type, and then there's the typedef in the gnulib namespace, that will point to the rpl_timeval replacement type on Windows. Thanks, Pedro Alves

Re: C++ aliases in

2016-12-19 Thread Pedro Alves
On 12/17/2016 09:32 AM, Gisle Vanem wrote: > Pedro Alves wrote: > >> Can't see how that can run fine? The compiler will set up the call >> assuming cdecl convention, while the called function has stdcall >> convention. > > I would expect the 'reinterpr

Re: C++ aliases in

2016-12-19 Thread Pedro Alves
On 12/18/2016 01:16 AM, Bruno Haible wrote: > Pedro Alves wrote: >> m4/inet_pton.m4 has this: >> >> if test $ac_cv_have_decl_inet_pton = yes; then >> dnl It needs to be overridden, because the stdcall calling convention >> dnl is not compliant with