Re: CPP Check 1.1.31

[James McKenzie]

Nikolay Sivov wrote:
> 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.

Thank you for the clarification on the use of sizeof.

James McKenzie