Re: [4/5] d3dx9: Implement D3DXMESHOPT_ATTRSORT vertex reordering.

[Matteo Bruni]

2011/5/6 Dylan Smith <dylan.ah.smith(a)gmail.com>:
> +struct vertex_attrib_duplication {
> +    DWORD attrib;
> +    DWORD vertex_index;
> +    struct vertex_attrib_duplication *ptr;
> +};
> ...
> +            struct vertex_attrib_duplication **heads = NULL; /* head of list for each vertex */

You probably want to use Wine lists (from include/wine/list.h) instead
of your own implementation.
> +            heads = HeapAlloc(GetProcessHeap(), 0, This->numvertices * sizeof(*heads) + max_entries * sizeof(*duplications));
> +            duplications = (struct vertex_attrib_duplication *)(heads + This->numvertices);

I'm not sure why you are usually sticking together many arrays in a
single allocation. Economizing on the number of HeapAllocs is not a
great reason IMHO.