From: Ramsay Jones <ramsay@ramsay1.demon.co.uk>
Date: Mon, 29 Apr 2013 23:09:15 GMT
Subject: Re: [PATCH 6/6] compat/nedmalloc: Fix compiler warnings on linux
Message-ID: <517EFD9B.4020608@ramsay1.demon.co.uk>
In-Reply-To: <20130429050616.GB8031@elie.Belkin>

Jonathan Nieder wrote:
> Hi,
> 
> Ramsay Jones wrote:
> 
>> --- a/compat/nedmalloc/malloc.c.h
>> +++ b/compat/nedmalloc/malloc.c.h
>> @@ -484,6 +484,10 @@ MAX_RELEASE_CHECK_RATE   default: 4095 unless not HAVE_MMAP
>>  #define DLMALLOC_VERSION 20804
>>  #endif /* DLMALLOC_VERSION */
>>
>> +#if defined(linux)
>> +#define _GNU_SOURCE 1
>> +#endif
> 
> If we want to do this, I'd suggest defining _GNU_SOURCE
> unconditionally.  Other platforms using glibc would benefit from it,
> too, and other platforms not using glibc shouldn't care.

My original version was unconditional, then I came over all
conservative ... :-D

Actually, in order to reduce the chance of future breakage, it
should really be placed at the beginning of the translation unit
(i.e. at the start of nedmalloc.c, before *any* #includes).

ATB,
Ramsay Jones