X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=blobdiff_plain;f=TODO;h=4673dc1063c90d5278d3adcf3dd985b463d3a451;hb=293b091c68447d30c8bda413cff1d6d76d9326a4;hp=82bba741364f276ee76834a7e8dcd81b96cdb0eb;hpb=10c1c646ed20f022c10647f4b3a829a67d70f7e0;p=gdcm.git diff --git a/TODO b/TODO index 82bba741..4673dc10 100644 --- a/TODO +++ b/TODO @@ -1,7 +1,22 @@ -* Convert the C-like IO to C++ IO (i.e. remove FILE*, fread...). Some docs - are available at - http://www.angelfire.com/country/aldev0/cpphowto/cpp_BinaryFileIO.html -* Remove enable_sequences and ignore_shadow from gdcmDocument(). +----------------------------------------------------------------------------- +Use namespace gdcm: + Problem: using enum with name like 'Unknow' on .Net, or LP on cygwin + causes problems. + Question: when introducing the namespace, should we remove the gdcm + prefix from classes or keep it ? +----------------------------------------------------------------------------- +Convert the C-like IO to C++ IO: + Goal: remove all the C-oriented IO references like FILE*, fread... + with the C++ fstream notation. Provide overload of operators + << and >> for any gdcm class using file IO. + Question: the underlying jpeg libraries (written in C) use the FILE* + notation. Is there a way to still use fstream in gdcm, and + build or pass the proper FILE* to jpeg libs ? + References: + binary IO are available at + http://www.angelfire.com/country/aldev0/cpphowto/cpp_BinaryFileIO.html +----------------------------------------------------------------------------- +* Clean up src/gdcmValEntry.[h|cxx] from VoidArea * vtk/vtkGdcmHeader.cxx: if speed becomes a concern some changes can be made at the cost of memory consumption (refer to header of vtk/vtkGdcmHeader.cxx) @@ -15,14 +30,6 @@ gdcmElValSet::GetElValueByNumber except for the returned code. gdcmHeader::SetPubElValLengthByNumber (which is based on gdcmElValSet::SetElValueLengthByNumber) is used nowhere... -* The declarations commented out and starting with "TODO Swig" (try - grep "TODO Swig" *.h) needed to be temporarily removed for swig to - proceed correctly (in fact problems appears at loading of _gdcm.[so/dll]). - So, simply uncomment the declaration once you provided the definition of - the method... -* As stated by the first lines of Test/ExceptionAndPython/README, it looks - like we can move back to the exceptions and remove the errno stuff from - src/gdcm* ! * Fix the bug in Test/bug1.cxx (see first comment line): Win32 only. * All (or at least many of) the methods of gdcmHeader whose only arguments are an ElValue* (e.g. FindLength, FindVR, LoadElementValue...) can @@ -68,5 +75,3 @@ a.write(output); } -* use namespace for gdcm, to avoid problem when using enum with name like - 'Unknow' on .Net, and LP on cygwin that cause problems