[Insight-developers] vxl update

[Sean McBride]

On Tue, 7 Dec 2010 18:19:19 -0500, Sean McBride said:

>>http://www.cdash.org/CDash/viewBuildError.php?buildid=795403
>
>My machine... the error is indeed correct, and I believe the problem
>stems from the fact that this is a Universal Binary, and that vxl is
>using a try-compile to decide that 'long' is 64 bit, but that's only
>true for half of the universal binary, for the other half, 'long' is 32
>bit, and the compiler rightly complains when they try to assign a 64 bit
>constant.
>
>Sorry I didn't catch that before the merge.  I don't have a universal
>vxl dashboard (will add one now), and only tried x86_64 on my own machine.
>
>I could fix this the same way it was fixed for VTK/ITK years ago: an #if
>APPLE that ignores the try-compile results.

Now I have a Univeral i386/x86_64 vxl dashboard, and it exhibits the
same issue (though apparently with clang it's a warning and with gcc
it's an error):

<http://www.cdash.org/CDash/viewBuildError.php?type=1&buildid=795916>

/vxl/core/vnl/tests/test_rational.cxx:130:13: warning: implicit
conversion from 'long long' to 'long' changes value from 1234321234321
to 1665620369 [-Wconstant-conversion]
  long l1 = 1234321234321L, l2 = 2*l1, l3 = 123456787654321L, l4 = l3+1;
       ~~   ^~~~~~~~~~~~~~

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