quartz: check hr so we are not incrementing a NULL (coverity 106)

Aric Stewart aric at codeweavers.com
Mon Oct 6 07:42:58 CDT 2008


true, that would be cleaner.  I will resubmit.

-aric

Dmitry Timoshkov wrote:
> "Aric Stewart" <aric at codeweavers.com> wrote:
> 
>> -    wszPatternString++; /* skip ',' */
>> +    if (hr == S_OK)
>> +        wszPatternString++; /* skip ',' */
>>
>>     if (hr == S_OK)
>>     {
> 
> Why not just move it into the below block?
> 



More information about the wine-devel mailing list