[vtk-developers] Problems with DICOM reader

[Jeff Lee]

can we shut it off altogether with a flag in CMake?  Right now, the only 
way to exclude it is to not use ansi, or manually cull it out of the 
CMakeLists.txt.
-Jeff

Turek, Matthew W (Research) wrote:

>>1. It only works when ansi streams are on.
>>    
>>
>
>Yes. I am willing to change the DICOMParser library to allow ansi/non-ansi
>if it's not too difficult. The DICOMParser is used outside of vtk, though,
>so I'll probably have to copy the vtk logic.
> 
>  
>
>>2. DICOMParser library does not follow static / shared flag of the rest
>>of VTK. That means it is always build static.
>>    
>>
>
>Yes. The DICOMParser has templated classes in the API. This is still a
>problem for creating DLLs if I remember correctly. I'm open to suggestions
>here.
>
>  
>
>>3. Indentation of files is wrong.
>>    
>>
>
>I have fixed (I think) the indentation of vtkDICOMImageReader.h
>vtkDICOMImageReader.cxx. 
>
>  
>
>>4. There is a prototype for class myvector in the header file of a vtk
>>class. It should really be vtkDICOMImageReaderVector the way it is used.
>>    
>>
>
>I have fixed this.
>
>  
>
>>5. There are references to std namespace (std::string), which should be
>>vtkstd.
>>    
>>
>
>I have changed this in vtkDICOMImageReader. I'm leaving DICOMParser as is,
>since it's used outside of vtk.
>
>
>Matt
>_______________________________________________
>vtk-developers mailing list
>vtk-developers at public.kitware.com
>http://public.kitware.com/mailman/listinfo/vtk-developers
>
>
>  
>

-- 
Jeff Lee
Software Engineer
Computational Dynamics North America Ltd
21 Lafayette Street, Suite 230
Lebanon NH 03766 USA
fax:   603 643 9994
phone: 603 643 9993 x109
http://www.cd-adapco.com