Re: [PATCH 10/12] MINGW: config.mak.uname: drop USE_NED_ALLOCATOR

2014-04-29 Thread Felipe Contreras
Marat Radchenko wrote: > We need to make a decision: drop nedalloc, update nedalloc to later release, > patch nedalloc to make it work under MinGW-W64 or disable nedalloc under > MinGW-W64 (still leaving it enabled under MinGW). I say go for the latter (disable for mingw-264). It can be fixed lat

Re: [PATCH 10/12] MINGW: config.mak.uname: drop USE_NED_ALLOCATOR

2014-04-29 Thread Marat Radchenko
On Mon, Apr 28, 2014 at 05:23:25PM +0200, Erik Faye-Lund wrote: > On Mon, Apr 28, 2014 at 3:51 PM, Marat Radchenko > wrote: > > nedalloc was initially added in f0ed82 to fix slowness of standard WinXP > > memory allocator. Since WinXP is EOLed, this point is no longer valid. > > > > The actual re

Re: [PATCH 10/12] MINGW: config.mak.uname: drop USE_NED_ALLOCATOR

2014-04-28 Thread Philip Oakley
From: "Felipe Contreras" Marat Radchenko wrote: nedalloc was initially added in f0ed82 to fix slowness of standard WinXP memory allocator. Since WinXP is EOLed, this point is no longer valid. The fact that WinXP is EOLed doesn't mean people are not using it any more =/ I think it should b

RE: [PATCH 10/12] MINGW: config.mak.uname: drop USE_NED_ALLOCATOR

2014-04-28 Thread Felipe Contreras
Marat Radchenko wrote: > nedalloc was initially added in f0ed82 to fix slowness of standard WinXP > memory allocator. Since WinXP is EOLed, this point is no longer valid. The fact that WinXP is EOLed doesn't mean people are not using it any more =/ I think it should be supported by Git at least f

Re: [PATCH 10/12] MINGW: config.mak.uname: drop USE_NED_ALLOCATOR

2014-04-28 Thread Jonathan Nieder
Erik Faye-Lund wrote: > On Mon, Apr 28, 2014 at 3:51 PM, Marat Radchenko > wrote: >> nedalloc was initially added in f0ed82 to fix slowness of standard WinXP >> memory allocator. Since WinXP is EOLed, this point is no longer valid. >> >> The actual reason behind this commit is incompatibility of

Re: [PATCH 10/12] MINGW: config.mak.uname: drop USE_NED_ALLOCATOR

2014-04-28 Thread Marat Radchenko
> Did you measure that malloc on newer Windows-versions are actually faster? No, I didn't. As I said, real reason for this patch is that Git version of nedalloc fails to compile under MinGW-W64. If we still want to use nedalloc under MinGW, this patch should be replaced with: a) Updating nedal

Re: [PATCH 10/12] MINGW: config.mak.uname: drop USE_NED_ALLOCATOR

2014-04-28 Thread Erik Faye-Lund
On Mon, Apr 28, 2014 at 3:51 PM, Marat Radchenko wrote: > nedalloc was initially added in f0ed82 to fix slowness of standard WinXP > memory allocator. Since WinXP is EOLed, this point is no longer valid. > > The actual reason behind this commit is incompatibility of malloc.c.h > with MinGW-W64 hea

[PATCH 10/12] MINGW: config.mak.uname: drop USE_NED_ALLOCATOR

2014-04-28 Thread Marat Radchenko
nedalloc was initially added in f0ed82 to fix slowness of standard WinXP memory allocator. Since WinXP is EOLed, this point is no longer valid. The actual reason behind this commit is incompatibility of malloc.c.h with MinGW-W64 headers. Alternative solution implies updating nedalloc to something