(try 2)[PATCH 1/4] d3dcompiler: Add initial reflection parsing.
kgbricola at web.de
Tue Nov 9 13:48:24 CST 2010
Am 09.11.2010 18:39, schrieb Henri Verbeet:
> 2010/11/9 Rico Schüller<kgbricola at web.de>:
>> I made the assumption, because setting the vtbl is the first thing
>> d3dcompiler_shader_reflection_init() does.
> It is, but you're not supposed to care. Abstractions aren't very
> useful if you have to care about the details of their internals.
>> Well, a heap free isn't always enough, because there could already be
>> allocated data to the object, which then would resolve in lost memory.
> d3dcompiler_shader_reflection_init() should free anything it allocated
> if it's going to fail.
Ok, it does that, if HeapAlloc and IUnknown_Release() is moved to
d3dcompiler_shader_reflection_init(). So everything would be fine.
More information about the wine-devel