[Insight-developers] Problem again in NRRDIO

Tue Mar 23 16:29:00 EDT 2010

Hi,

so can you try changing in mangle.pl

         s|.* [TDRS] (.*)|$1|g;

to

         s|.* [TBDRS] (.*)|$1|g;

and send the resulting itk_NrrdIO_mangle.h to Daniel so he can see if  
it fixes the problem?

Gordon

On Mar 23, 2010, at 3:25 PM, Michel Audette wrote:

> defaultsNrrd.o:
> 0000000000000010 D nrrdDefaultCenter
> 0000000000000018 D nrrdDefaultSpacing
> 0000000000000004 D nrrdDefaultWriteBareText
> 0000000000000008 D nrrdDefaultWriteCharsPerLine
> 0000000000000000 D nrrdDefaultWriteEncodingType
> 000000000000000c D nrrdDefaultWriteValsPerLine
> 0000000000000020 D nrrdStateAlwaysSetContent
> 0000000000000008 B nrrdStateDisableContent
> 000000000000000c B nrrdStateGrayscaleImage3D
> 0000000000000004 B nrrdStateKeyValuePairsPropagate
> 0000000000000010 B nrrdStateKeyValueReturnInternalPointers
> 0000000000000014 B nrrdStateKindNoop
> 0000000000000028 D nrrdStateUnknownContent
> 0000000000000000 B nrrdStateVerboseIO
>
> We can talk on the phone if you prefer... Let me know.
>
> Cheers,
>
> Michel
>
> On Tue, Mar 23, 2010 at 4:23 PM, Gordon L. Kindlmann  
> <glk at uchicago.edu> wrote:
> Hello,
>
> Ok, so the question is, *how* is nrrdStateGrayscaleImage3D listed?   
> there is probably some cryptic single letter code that identifies  
> the type of symbol.  mangle.pl is using these letter codes to  
> distinguish between symbols that the NrrdIO library is actually  
> supplying, versus merely using, and the problem is that  
> nrrdStateGrayscaleImage3D is actually being supplied, but is not  
> being recorded as such in the output of mangle.pl.  Line 66 of  
> mangle.pl is where this decision is made; so it may need tweaking.
>
> Gordon
>
>
> On Mar 23, 2010, at 3:18 PM, Michel Audette wrote:
>
> Hi Gordon,
>
> According to this procedure, _nrrdStateGrayscaleImage3D fails to  
> appear, although nrrdStateGrayscaleImage3D is listed in the result  
> of nm.
>
> What next?
>
> Michel
>
> On Tue, Mar 23, 2010 at 2:07 PM, Gordon L. Kindlmann  
> <glk at uchicago.edu> wrote:
> Hello,
>
> Shoot- sorry about this; I was just thinking the NrrdIO update was  
> finished!
>
> There indeed probably aren't any tests for checking compiling  
> applications against both Teem (containing the air, biff, and nrrd  
> libraries that these symbols are coming from) and ITK (containing  
> NrrdIO, the fraction of Teem required for Nrrd file IO).  Aren't  
> Slicer builds doing this?  Are there analogous tests for, say,  
> compiling applications against non-ITK zlib and ITK name-mangled  
> zlib, which can be copied from?
>
> I can tell that these names are for the global variables that exist  
> in NrrdIO.  These are probably not getting correctly picked up from  
> the list of symbols in the NrrdIO library, which mangle.pl does by  
> using "nm".
>
> Michel, can go through the NrrdIO process on whatever platform you  
> used last, and at the point of running mangle.pl, can you run "nm"  
> manually and grep for, say, _nrrdStateGrayscaleImage3D, and see how  
> that symbol is identified?  Line 66 of mangle.pl apparently is  
> missing this.
>
> Gordon
>
>
> On Mar 23, 2010, at 12:33 PM, Daniel Blezek wrote:
>
> Hi,
>
>  We link both against NRRD and ITK in our applications.  At the end  
> of February, the mangled ITK names were changed from what I fixed.   
> Clearly, this is not tested properly, because today I updated ITK  
> and re-built:
>
> 34>ITKNrrdIO.lib(string.obj) : error LNK2005: _airStrtokQuoting  
> already defined in teem.lib(string.obj)
> 34>ITKNrrdIO.lib(defaultsNrrd.obj) : error LNK2005:  
> _nrrdStateVerboseIO already defined in teem.lib(defaultsNrrd.obj)
> 34>ITKNrrdIO.lib(defaultsNrrd.obj) : error LNK2005:  
> _nrrdStateKeyValuePairsPropagate already defined in  
> teem.lib(defaultsNrrd.obj)
> 34>ITKNrrdIO.lib(defaultsNrrd.obj) : error LNK2005:  
> _nrrdStateDisableContent already defined in teem.lib(defaultsNrrd.obj)
> 34>ITKNrrdIO.lib(defaultsNrrd.obj) : error LNK2005:  
> _nrrdStateGrayscaleImage3D already defined in  
> teem.lib(defaultsNrrd.obj)
> 34>ITKNrrdIO.lib(defaultsNrrd.obj) : error LNK2005:  
> _nrrdStateKeyValueReturnInternalPointers already defined in  
> teem.lib(defaultsNrrd.obj)
> 34>ITKNrrdIO.lib(defaultsNrrd.obj) : error LNK2005:  
> _nrrdStateKindNoop already defined in teem.lib(defaultsNrrd.obj)
> 34>ITKNrrdIO.lib(biffmsg.obj) : error LNK2005: __biffMsgNoop already  
> defined in teem.lib(biffmsg.obj)
> 34>ITKNrrdIO.lib(dio.obj) : error LNK2005: _airDisableDio already  
> defined in teem.lib(dio.obj)
>
> Could “Michel” (according to CVS) please have a closer look at  
> this?  I had tweaked this once before, and my changes were blown  
> away by the (broken) mangle.pl script.
>
> I’ll go back through and manually add the names if necessary.
>
> Cheers,
> -dan
>
> -- 
> Daniel Blezek, PhD
> Medical Imaging Informatics Innovation Center
>
> P 127 or (77) 8 8886
> T 507 538 8886
> E blezek.daniel at mayo.edu
>
> Mayo Clinic
> 200 First St. S.W.
> Harwick SL-44
> Rochester, MN 55905
> mayoclinic.org
>
> _______________________________________________
> 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
>
> _______________________________________________
> 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
>
>
>
> -- 
> Michel Audette, Ph.D.
> R & D Engineer,
> Kitware Inc.,
> Chapel Hill, N.C.
>
>
>
>
>
> -- 
> Michel Audette, Ph.D.
> R & D Engineer,
> Kitware Inc.,
> Chapel Hill, N.C.
>