2013/6/11 Rico Schüller <kgbric...@web.de>

> On 11.06.2013 22:08, Christian Costa wrote:
>
>> Fixes bug 26598.
>> ---
>>   dlls/d3dx9_36/effect.c       |    4 ++++
>>   dlls/d3dx9_36/tests/effect.c |   17 +++++++++++++++++
>>   2 files changed, 21 insertions(+)
>>
>> diff --git a/dlls/d3dx9_36/effect.c b/dlls/d3dx9_36/effect.c
>> index 1924c07..bab4560 100644
>> --- a/dlls/d3dx9_36/effect.c
>> +++ b/dlls/d3dx9_36/effect.c
>> @@ -5792,6 +5792,10 @@ HRESULT WINAPI D3DXCreateEffectEx(struct
>> IDirect3DDevice9 *device, const void *s
>>
>>       *effect = &object->ID3DXEffect_iface;
>>
>> +    /* Must be set to NULL if no compilation error */
>> +    if (compilation_errors)
>> +        *compilation_errors = NULL;
>> +
>>
> No, this is wrong! Your test case doesn't cover all cases.
>
> Which cases? Ssource effects or possible parsing error of binary ones?
Anyway my intention was to fix only this particular case as we don't
support source effect yet.
I'll use a different approach.


Reply via email to