Re: [PATCH 3/5] usp10: Use heap_alloc_zero() instead of HeapAlloc() with HEAP_ZERO_MEMORY.

[Henri Verbeet]

On 9 March 2017 at 15:51, Sebastian Lackner  wrote:
> On 09.03.2017 10:37, Henri Verbeet wrote:
> > -static inline BOOL heap_free(void *mem)
> > -{
> > - return HeapFree(GetProcessHeap(), 0, mem);
> >
> > -}
>
> [...]
> > +static inline void heap_free(void *mem)
> > +{
> > - HeapFree(GetProcessHeap(), 0, mem);
> >
> > +}
>
> I'm not sure if Francois is happy with this change. In other dlls, he
> has made the exact opposite change in his effort to standardize those
> functions.

I wasn't aware of that. While I don't care particularly much if it's
there or not, I'd be somewhat surprised if there are legitimate uses
of the return value. At least inside usp10 it's always ignored.