Re: [PATCH 1/4] include: Add generic HeapAlloc() wrappers

[Henri Verbeet]

On 25 January 2018 at 23:25, Michael Stefaniuc <mstefani(a)winehq.org> wrote:
> +#ifdef __cplusplus
> +extern "C" {
> +#endif
> Strictly speaking that's superfluous for internal headers (and more so
> for ones that only contain static inline functions), I think.
> +static inline void * __WINE_ALLOC_SIZE(2) heap_realloc(void *mem, SIZE_T len)
> +{
> +    if (mem)
> +        return HeapReAlloc(GetProcessHeap(), 0, mem, len);
> +    else
> +        return HeapAlloc(GetProcessHeap(), 0, len);
> +}
> I'm partial to

if (!mem)
        return HeapAlloc(...);
    return HeapReAlloc(...);

but that's purely stylistic, and perhaps a preference that's not
universally shared.