On 20 August 2013 12:07, Stefan Dösinger <ste...@codeweavers.com> wrote:
> +void *wined3d_alloc_resource_mem(SIZE_T size)
> +{
> + void **p;
> + SIZE_T align = RESOURCE_ALIGNMENT - 1 + sizeof(*p);
> + void *mem;
> +
> + if (!(mem = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, size + align)))
> + return NULL;
> +
> + p = (void **)(((ULONG_PTR)mem + align) & ~(RESOURCE_ALIGNMENT - 1)) - 1;
> + *p = mem;
> +
> + return ++p;
> +}
I think this should either be called
wined3d_resource_allocate_sysmem() and live in resource.c, or accept
any np2 alignment and be called something along the lines of
wined3d_memalign(). The former probably makes the most sense at this
point.
Somewhat related, I don't think there's really a reason to have both
"allocatedMemory" and "heap_memory" anymore. It should probably just
be called "sysmem". That's a cleanup that doesn't necessarily belong
in this patch though.
