#include <string>

[gdcm.git] / TODO

diff --git a/TODO b/TODO
index 420485aefda6dc4d0e1a3ee16aef7cfcd292e6a0..0e7dc0211dae44f6ae6efc356acbb70f69248c45 100644 (file)
--- a/TODO
+++ b/TODO
@@ -1,3 +1,12 @@
+* gdcmElValSet::SetElValueLengthByNumber IMNSHO should be trashed.
+  It's only purpose is a onliner substitute to calling GetElValueByNumber
+  and then SetLength. This only obfuscates the caller code more than
+  clarifying it.
+  Besides the definition of gdcmElValSet::SetElValueLengthByNumber itself
+  it quite poor since it is a almost exact copy of
+  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]).
@@ -23,7 +32,7 @@
 * GetPubElValByNumber doit faire la difference entre chaine vide 
   et chaine pas touve''. Eventuellement raiser une exception ?
 * gdcmHeader::LoadElements only loads the element whose length is
-  below the specified size. When accessing the value of such a element
+  below the specified size. When accessing the value of such an element
   the content is unfound ! Find a decent way of loading the value on
   explicit demand.
 * fournir une method qui ne fait que lire les elements passes en arguments