[Insight-developers] Re: [Insight-users] GDCM 1.2 update status.

[Mathieu Malaterre]

Hans Johnson wrote:
> Mathieu,
> 
> I just wanted to let you know that some of the API changes in GDCM_LEGACY
> are not working fully.
> 
> === Old Code That Causes Segfaults ===
> gdcm::File GdcmHeader(path);
> 
> === New Code That Works ===
> gdcm::File GdcmHeader;
> GdcmHeader.SetFileName(path);
> GdcmHeader.Load();
> 
> =======================
> 
> 
>>From gdcmFile.h:
> GDCM_LEGACY(File( std::string const &filename ));
> 
> 
> My guess is that the GDCM_LEGACY(File...) call does not properly initialize
> the class.
> 
> We have already made all the necessary changes to support the new interface,
> but I thought you would like to know.

  $ cvs ci -m"ENH: This should fix the LEGACY code" 
 
/cvsroot/Insight/Insight/Utilities/gdcm/src/gdcmFile.cxx,v  <-- 
gdcmFile.cxx
new revision: 1.12; previous revision: 1.11

Thanks for report. I have updated gdcm to fix that.

Sorry for the trouble,
Mathieu