481586 - rename FT_Memory.free and FT_Memory.realloc to allow jemalloc to be enabled on windows ce

Status: RESOLVED INVALID

(Core :: General, defect)

Description

[Brad Lassey [:blassey] (use needinfo?)]

Attachment: patch v.1

For jemalloc we're defining free to moz_free, which breaks the build in freetype2.  This renames the free and realloc members of FT_Memory to freeMem and reallocMem respectively.

Comment 1

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Comment on attachment 365609 [details] [diff] [review]
patch v.1

This is fine, but make a local-patches or something dir underneath our freetype module, and stick this patch file in there and a README describing it (as well as the version of ft we're on), similar to what's in gfx/cairo -- otherwise this will probably get blown away when we take a new freetype.

Comment 2

[Brad Lassey [:blassey] (use needinfo?)]

Switching to #define free moz_free from #define free(x) moz_free(x) eliminates
the need for this patch for jemalloc.  As discussed on irc, we should explore
linker options as well.

Marking this as invalid.