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

[Laszlo Ladanyi]

OK, the change is committed.

--Laci

On Mon, 22 May 2006, Matthew Galati wrote:

> 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
> 
> _______________________________________________
> Coin-discuss mailing list
> Coin-discuss at list.coin-or.org
> http://list.coin-or.org/mailman/listinfo/coin-discuss
>