[vtk-developers] Recent CVS Changes for Images & vtkIdType

[Brad King]

Randall Hand wrote:
> I see that alot of changes went into CVS yesterday for the resolution of 
> bug 1867 ( http://www.vtk.org/Bug/bug.php?op=show&bugid=1867 
> <http://www.vtk.org/Bug/bug.php?op=show&bugid=1867> ).
> 
> Attempting to compile this results in alot of error messages for me, 
> with USE_64BIT_IDS enabled.  in vtkByteSwap.h, I get errors about 
> double-declared functions for everything in the #ifdef VTK_USE_64BIT_IDS 
> as the vtkIdType * argument version now collides with the long * 
> argument version.  Simply commenting out #ifdef blocks (as done in the 
> attached patch) seems to fix the problem, but i'm not sure it's the best 
> solution.

The changes giving these errors have nothing to do with the Images and 
vtkIdType changes.  Those changes were committed two days ago and did 
not have many problems.  The errors you have encountered were a result 
of some huge changes I made yesterday to add long long and __int64 
support to vtkTemplateMacro.

> (The included patch also has a fix for a function declaration error, 
> where it's prototyped as normal but implemented as "inline").

The vtkBitArray warnings were reported on the dashboard by the nightly 
builds and have been fixed.  The vtkByteSwap errors only occur on a 
platform for which "long" is 64-bits.  I'll fix these soon.

-Brad