[vtk-developers] Some interesting warnings from the clang compiler

Sean McBride sean at rogue-research.com
Thu Jul 29 10:28:03 EDT 2010


On Thu, 29 Jul 2010 08:21:33 -0600, David Gobbi said:

>Wouldn't it be better for someone to actually read the code and find
>out it needs to be fixed, instead of just (ahem) guessing?

I looked at that one a few months ago when I started playing with clang,
and from the name of the case label:

 case VTK_PARSER_UNARY_MINUS:
   this->Stack[stackPosition] =- this->Stack[stackPosition];

I think

 case VTK_PARSER_UNARY_MINUS:
   this->Stack[stackPosition] = -(this->Stack[stackPosition]);

would be correct.

-- 
____________________________________________________________
Sean McBride, B. Eng                 sean at rogue-research.com
Rogue Research                        www.rogue-research.com 
Mac Software Developer              Montréal, Québec, Canada





More information about the vtk-developers mailing list