[Insight-developers] Hashtable errors and kwsys

[Bradley Lowekamp]

Bill,

My compilation finally finished. It passed, but I now see the problem. 

These matchines have the following turned on:

ITK_USE_64BITS_IDS:BOOL=ON
This makes use of a 64-bit integer type, for the itk::IdentifierType. These particular systems don't have the hash for this type. Should be easy enough to add though.

No need to revert. We should be able to move forward.

Brad

On Jul 28, 2011, at 11:31 PM, Bill Lorensen wrote:

> I am not sure why kwsys has this functionality. Especially if it is neither used nor tested.
> 
> On Thu, Jul 28, 2011 at 5:33 PM, David Cole <david.cole at kitware.com> wrote:
> If the ITK version works, and the KWSys version does not, then we
> should update the KWSys version to match the ITK version...
> 
> 
> On Thu, Jul 28, 2011 at 4:29 PM, Bradley Lowekamp
> <blowekamp at mail.nih.gov> wrote:
> > Bill,
> > Thanks for the reminder. I didn't realize this effected VS8 and VS9. I for
> > some reason only thought that it was only VS8 ( which I don't have ).  But
> > since it effect VS9 too, I can work on that tomorrow. If I am not near a
> > solution on what's going on after tomorrow, we can revert the commit.
> > I just thought that KWSys had this class, and it' would be rather robust,
> > but it appears no one has been using that version either.
> > Brad
> > On Jul 28, 2011, at 4:12 PM, Bill Lorensen wrote:
> >
> > We need to resolve the lingering hast_table errors on Visual Studio. Perhaps
> > we should backtrack and use the itk_hash_x stuff since the kwsys stuff is
> > not currently portable and apparently untested.
> > Thank
> > We can't afford to have these nightly compiler errors:
> >
> > http://www.cdash.org/CDash/viewBuildError.php?buildid=1371577
> >
> > Bill
> >
> >
> > _______________________________________________
> > Powered by www.kitware.com
> >
> > Visit other Kitware open-source projects at
> > http://www.kitware.com/opensource/opensource.html
> >
> > Kitware offers ITK Training Courses, for more information visit:
> > http://kitware.com/products/protraining.html
> >
> > Please keep messages on-topic and check the ITK FAQ at:
> > http://www.itk.org/Wiki/ITK_FAQ
> >
> > Follow this link to subscribe/unsubscribe:
> > http://www.itk.org/mailman/listinfo/insight-developers
> >
> >
> 

========================================================
Bradley Lowekamp  
Lockheed Martin Contractor for
Office of High Performance Computing and Communications
National Library of Medicine 
blowekamp at mail.nih.gov


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20110729/3eb79da2/attachment.htm>