[Coin-discuss] Re: various warnings by .NET compiler

[Matthew Galati]

Laci,

After changing to:
#define BLOCKSQSHIFT (BLOCKSHIFT+BLOCKSHIFT)

The warnings go away.

Matt


> > ClpCholeskyDense.cpp
> > c:\cygwin\home\magala\COIN\Clp\ClpCholeskyDense.cpp(646) : warning 
> > C4554: '<<' : check operator precedence for possible error; use 
> > parentheses to clarify precedence
> > c:\cygwin\home\magala\COIN\Clp\ClpCholeskyDense.cpp(647) : warning 
> > C4554: '<<' : check operator precedence for possible error; use 
> > parentheses to clarify precedence
> > c:\cygwin\home\magala\COIN\Clp\ClpCholeskyDense.cpp(648) : warning 
> > C4554: '<<' : check operator precedence for possible error; use 
> > parentheses to clarify precedence
> 
> I'm not certain, but I believe these are the result of not 
> having parentheses in the #define of BLOCKSQSHIFT (line 77). 
> I can't test my theory (g++ doesn't complain), but could you 
> test it and let me know? Just replace the line:
>       #define BLOCKSQSHIFT BLOCKSHIFT+BLOCKSHIFT with
>       #define BLOCKSQSHIFT (BLOCKSHIFT+BLOCKSHIFT) (While you 
> are at it, you could do the same in the preceeding line for 
> the definition of BLOCKSQ.)
> 
> Let me know whether this helps; if it does, I'll commit the change.
> 
> Thanks,
> --Laci