[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