[Insight-developers] Help to debug on LInux64 and Mac 10.5
Sean McBride
sean at rogue-research.com
Fri Dec 21 11:11:48 EST 2007
On 12/21/07 10:25 AM, Bill Lorensen said:
>I noticed that the "extra" stuff in 32 bits seems to be all zeroes. Not true
>in 64 bits.
I am still curious about what this stuff is...
>Anyway, I've made changes that restrict the hashing to the Point part of the
>object.
>
>I just checked in the code. Can you update and tell me if it works? I made
>changes in Code/Common, Code/Algorithms and Code/Review.
Tada! Passes in both 32 and 64, key size now 12 bytes in both cases,
and the output is identical.
Although not crucial, this investigation uncovered a couple of 64-to-32
truncations, and I have attached a small patch for your review. I think
it's quite safe, if you agree I/you can commit it.
Now how about the other 42 failing tests on our dashboard? :)
Happy holidays!
--
____________________________________________________________
Sean McBride, B. Eng sean at rogue-research.com
Rogue Research www.rogue-research.com
Mac Software Developer Montréal, Québec, Canada
-------------- next part --------------
Index: Code/Algorithms/itkStructHashFunction.h
===================================================================
RCS file: /cvsroot/Insight/Insight/Code/Algorithms/itkStructHashFunction.h,v
retrieving revision 1.4
diff -r1.4 itkStructHashFunction.h
55c55
< int len = sizeof( InputType );
---
> size_t len = sizeof( InputType );
Index: Testing/Code/Review/itkAutomaticTopologyQuadEdgeMeshSourceTest.cxx
===================================================================
RCS file: /cvsroot/Insight/Insight/Testing/Code/Review/itkAutomaticTopologyQuadEdgeMeshSourceTest.cxx,v
retrieving revision 1.2
diff -r1.2 itkAutomaticTopologyQuadEdgeMeshSourceTest.cxx
83c83
< for( IdentifierType i = 0; i < 8; i++ )
---
> for( unsigned int i = 0; i < 8; i++ )
252c252
< unsigned int i;
---
> unsigned long i;
333c333
< int numPoints = meshSource->GetOutput()->GetNumberOfPoints();
---
> unsigned long numPoints = meshSource->GetOutput()->GetNumberOfPoints();
343c343
< int numCells = meshSource->GetOutput()->GetNumberOfCells();
---
> unsigned long numCells = meshSource->GetOutput()->GetNumberOfCells();
More information about the Insight-developers
mailing list