526389 - remove duplicated prototypes from jemalloc.h

Status: RESOLVED FIXED

(Core :: Memory Allocator, defect)

Description

[Takanori MATSUURA]

User-Agent:       Mozilla/5.0 (X11; U; Linux x86_64; ja; rv:1.9.3a1pre) Gecko/20091104 Fedora/3.7a1pre-0.a1pre.2009110411.hg_8ff817d76d2a.fc11 Minefield/3.7a1pre
Build Identifier: 

This bug is for the discussion of bug 422960 comment #7 and below.

malloc, valloc, calloc, realloc, free, posix_memalign, and memalign are already defined in system header (stdlib.h etc.) on some OSes. In glibc, some additional definition is added to these functions for exception handling (See bug 422960 comment #17).


So we should define prototypes above only for OSes which they are not defined.

I know 
MOZ_MEMORY_DARWIN platform have malloc, vallod, calloc, realloc, free, and posix_memalign
MOZ_MEMORY_LINUX platform have malloc, valloc, calloc, realloc, free, posix_memalign, and memalign

but I don't know MOZ_MEMORY_BSD, MOZ_MEMORY_SOLARIS, MOZ_MEMORY_WINCE, MOZ_MEMORY_WINCE6, and MOZ_MEMORY_WINDOWS.

Reproducible: Always

Comment 1

[Takanori MATSUURA]

Attachment: Skip redefinition of memory allocation functions for MacOS X and Linux.

Comment 2

[Dão Gottwald [:dao]]

http://hg.mozilla.org/mozilla-central/rev/7a2802932585