CPP Check 1.1.31
Nikolay Sivov
bunglehead at gmail.com
Mon Oct 12 14:31:09 CDT 2009
James McKenzie wrote:
> Nikolay Sivov wrote:
>
>> James McKenzie wrote:
>>
>>> Nikolay Sivov wrote:
>>>
>>>
>>>> chris ahrendt wrote:
>>>>
>>>>
>>>>> [/home/cahrendt/wine-git/dlls/kernel32/resource.c:1279]: (error)
>>>>> Division by zero
>>>>>
>>>>>
>>>> False positive I suppose:
>>>>
>>>> ---
>>>> static const BYTE pad[] = {
>>>>
>>>> 'P','A','D','D','I','N','G','X','X','P','A','D','D','I','N','G' };
>>>> DWORD i;
>>>>
>>>> -> for ( i = 0; i < size / sizeof pad; i++ )
>>>> ---
>>>>
>>>>
>>> Should pad be enclosed in parenthesis?
>>>
>>> James McKenzie
>>>
>>>
>> No, why?
>>
>>
>>
> I was thinking that c specifies sizeof needs the parameter to be
> enclosed in parathesis like this sizeof (pad) but I may be incorrect.
>
> James McKenzie
>
This is for type names, variables could be used without enclosing.
More information about the wine-devel
mailing list