X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=blobdiff_plain;f=TODO;h=0e7dc0211dae44f6ae6efc356acbb70f69248c45;hb=6a6058de5f928295b11c966fcebaa8fc0c9edf04;hp=420485aefda6dc4d0e1a3ee16aef7cfcd292e6a0;hpb=d74d6412b83d9f596597b54c3413d1a6ee9fb617;p=gdcm.git diff --git a/TODO b/TODO index 420485ae..0e7dc021 100644 --- 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