<div dir="ltr">Just tested this and it worked fine. Thanks :)</div><div class="gmail_extra"><br><div class="gmail_quote">On 3 October 2015 at 13:20, Behdad Esfahbod <span dir="ltr"><<a href="mailto:behdad.esfahbod@gmail.com" target="_blank">behdad.esfahbod@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 15-10-02 10:00 PM, Jamie Dale wrote:<br>
> That's similar to what I originally had, so I'm fine with something like that.<br>
> My only question is, what would hb_malloc_impl, hb_calloc_impl,<br>
> hb_realloc_impl, and hb_free_impl be defined to in your example?<br>
<br>
</span>Define it to the name of your external function.  I don't want to fix a name<br>
in HarfBuzz itself.  Updated patch to add externs:<br>
<br>
diff --git a/src/hb-private.hh b/src/hb-private.hh<br>
index 07550cb..826e41a 100644<br>
--- a/src/hb-private.hh<br>
+++ b/src/hb-private.hh<br>
@@ -54,6 +54,23 @@<br>
<span class=""> #include <stdarg.h><br>
<br>
<br>
+/* Compile-time custom allocator support. */<br>
+<br>
+#if defined(hb_malloc_impl) \<br>
+ && defined(hb_calloc_impl) \<br>
+ && defined(hb_realloc_impl) \<br>
+ && defined(hb_free_impl)<br>
</span>+extern void* hb_malloc_impl(size_t size);<br>
+extern void* hb_calloc_impl(size_t nmemb, size_t size);<br>
+extern void* hb_realloc_impl(void *, size_t size);<br>
+extern void  hb_free_impl(void *ptr);<br>
<span class="">+#define malloc hb_malloc_impl<br>
+#define calloc hb_calloc_impl<br>
+#define realloc hb_realloc_impl<br>
+#define free hb_free_impl<br>
+#endif<br>
+<br>
<br>
</span>I'll go ahead and commit this.  Let me know if it doesn't work.<br>
<span class=""><br>
> I went with the extern function approach so I could inject my own allocator<br>
> functions into HarfBuzz at link time, rather than have to include any of our<br>
> code into HarfBuzz at compile time (since any local code changes we make have<br>
> to be re-applied against new versions).<br>
><br>
> For your reference, this was my original implementation:<br>
><br>
> /* Override the allocation functions when<br>
>  * HAVE_EXTERNAL_ALLOCATOR is defined. */<br>
> #ifdef HAVE_EXTERNAL_ALLOCATOR<br>
> extern void* _hb_malloc(size_t);<br>
> extern void* _hb_calloc(size_t, size_t);<br>
> extern void* _hb_realloc(void*, size_t);<br>
> extern void _hb_free(void*);<br>
> #define malloc _hb_malloc<br>
> #define calloc _hb_calloc<br>
> #define realloc _hb_realloc<br>
> #define free _hb_free<br>
> #endif<br>
><br>
> Failure to define those functions in something that linked to HarfBuzz would<br>
> produce an unresolved external symbol error at link time.<br>
><br>
> -Jamie.<br>
><br>
> On 2 October 2015 at 08:08, Behdad Esfahbod <<a href="mailto:behdad.esfahbod@gmail.com">behdad.esfahbod@gmail.com</a><br>
</span><div><div class="h5">> <mailto:<a href="mailto:behdad.esfahbod@gmail.com">behdad.esfahbod@gmail.com</a>>> wrote:<br>
><br>
>     Hi Jamie,<br>
><br>
>     I'm opposed to runtime custom allocators, but am fine with something like what<br>
>     you suggest.  However, that has two down sides:<br>
><br>
>       1. It touches a lot of callsites,<br>
><br>
>       2. I have to add macros to make sure stock malloc etc are not used by<br>
>     mistake.<br>
><br>
>     If doing 2., I might as well just define them to their custom behavior.  Ie,<br>
>     would this patch work for you:<br>
><br>
>     diff --git a/src/hb-private.hh b/src/hb-private.hh<br>
>     index 07550cb..167c067 100644<br>
>     --- a/src/hb-private.hh<br>
>     +++ b/src/hb-private.hh<br>
>     @@ -54,6 +54,19 @@<br>
>      #include <stdarg.h><br>
><br>
><br>
>     +/* Compile-time custom allocator support. */<br>
>     +<br>
>     +#if defined(hb_malloc_impl) \<br>
>     + && defined(hb_calloc_impl) \<br>
>     + && defined(hb_realloc_impl) \<br>
>     + && defined(hb_free_impl)<br>
>     +#define malloc hb_malloc_impl<br>
>     +#define calloc hb_calloc_impl<br>
>     +#define realloc hb_realloc_impl<br>
>     +#define free hb_free_impl<br>
>     +#endif<br>
>     +<br>
>     +<br>
>      /* Compiler attributes */<br>
><br>
><br>
><br>
><br>
>     On 15-09-30 03:49 PM, Jamie Dale wrote:<br>
>     > Since there was no movement here I went ahead and hacked in our allocator by<br>
>     > redefining malloc, calloc, realloc, and free to point to extern'd functions<br>
>     > that I define in our application. This yielded some performance benefits for<br>
>     > us, so I'm hoping that a more standard HarfBuzz solution might appear at<br>
>     some<br>
>     > point :)<br>
>     ><br>
>     > If you're not keen on the idea of allowing the allocator to be swapped<br>
>     out at<br>
>     > runtime, would you at least allow it to be extern'd at compile time? I'm not<br>
>     > familiar with HarfBuzz coding standards, but I'd propose adding<br>
>     something like<br>
>     > a HAVE_EXTERNAL_ALLOCATOR define to control this, and then define your own<br>
>     > allocator functions that you'd use in place of malloc, etc.<br>
>     ><br>
>     > Basically, something like this (I currently have this code in<br>
>     hb-private.hh):<br>
>     ><br>
>     > /* Define our allocation functions. These may be provided by an<br>
>     >  * external source when HAVE_EXTERNAL_ALLOCATOR is defined. */<br>
>     > #ifdef HAVE_EXTERNAL_ALLOCATOR<br>
>     > extern void* _hb_malloc(size_t);<br>
>     > extern void* _hb_calloc(size_t, size_t);<br>
>     > extern void* _hb_realloc(void*, size_t);<br>
>     > extern void _hb_free(void*);<br>
>     > #define hb_malloc _hb_malloc<br>
>     > #define hb_calloc _hb_calloc<br>
>     > #define hb_realloc _hb_realloc<br>
>     > #define hb_free _hb_free<br>
>     > #else<br>
>     > #define hb_malloc malloc<br>
>     > #define hb_calloc calloc<br>
>     > #define hb_realloc realloc<br>
>     > #define hb_free free<br>
>     > #endif<br>
>     ><br>
>     > You'd then replace calls to malloc, calloc, realloc, and free from within<br>
>     > HarfBuzz to use hb_malloc, hb_calloc, hb_realloc, and hb_free.<br>
>     ><br>
>     > This is basically the code I already have, except I<br>
>     > redefined malloc, calloc, realloc, and free rather than create the<br>
>     > hb_*variants, as I didn't want to change too much HarfBuzz code.<br>
>     ><br>
>     > Thoughts?<br>
>     ><br>
</div></div>>     > On 31 August 2015 at 22:17, Jamie Dale <<a href="mailto:jamiedale88%2Bharfbuzz@gmail.com">jamiedale88+harfbuzz@gmail.com</a> <mailto:<a href="mailto:jamiedale88%252Bharfbuzz@gmail.com">jamiedale88%2Bharfbuzz@gmail.com</a>><br>
>     > <mailto:<a href="mailto:jamiedale88%2Bharfbuzz@gmail.com">jamiedale88+harfbuzz@gmail.com</a> <mailto:<a href="mailto:jamiedale88%252Bharfbuzz@gmail.com">jamiedale88%2Bharfbuzz@gmail.com</a>>>><br>
<span class="">>     wrote:<br>
>     ><br>
>     >     Hey,<br>
>     ><br>
>     >     I'm not sure if this is correct place to post/discuss feature requests...<br>
>     >     hopefully it is.<br>
>     ><br>
>     >     Our project makes use of a custom memory allocator, and we like to ensure<br>
>     >     that our third-party libraries are using this where possible, as our<br>
>     >     allocator can offer significantly better performance, and can also allow<br>
>     >     us better memory profiling and debugging support.<br>
>     ><br>
>     >     A look through the HarfBuzz source code would suggest that it doesn't<br>
>     >     currently support custom memory allocators, and instead just directly<br>
>     >     calls the standard malloc/free/etc functions. I was wondering if you'd<br>
>     >     ever considered (or would consider) adding a way to override this behaviour?<br>
>     ><br>
>     >     In my case, I'd just need a single global override to be set before my<br>
>     >     first call to any other HarfBuzz code... basically something like<br>
>     >     the FT_Memory struct from FreeType:<br>
>     >     <a href="http://www.freetype.org/freetype2/docs/reference/ft2-system_interface.html#FT_Memory" rel="noreferrer" target="_blank">http://www.freetype.org/freetype2/docs/reference/ft2-system_interface.html#FT_Memory</a><br>
>     ><br>
>     >     Thanks,<br>
>     >     Jamie.<br>
>     ><br>
>     ><br>
>     ><br>
>     ><br>
>     > _______________________________________________<br>
>     > HarfBuzz mailing list<br>
</span>>     > <a href="mailto:HarfBuzz@lists.freedesktop.org">HarfBuzz@lists.freedesktop.org</a> <mailto:<a href="mailto:HarfBuzz@lists.freedesktop.org">HarfBuzz@lists.freedesktop.org</a>><br>
>     > <a href="http://lists.freedesktop.org/mailman/listinfo/harfbuzz" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/harfbuzz</a><br>
>     ><br>
><br>
><br>
</blockquote></div><br></div>