]> Creatis software - gdcm.git/commitdiff
* src/gdcmElValSet.[cxx/h], gdcmElValue.[cxx/h], gdcmFile.[cxx/h],
authorfrog <frog>
Wed, 12 Mar 2003 21:33:19 +0000 (21:33 +0000)
committerfrog <frog>
Wed, 12 Mar 2003 21:33:19 +0000 (21:33 +0000)
        gdcmHeader.[cxx/h]: ElValSet class renamed to gdcmElValSet, and
        ElValue class renamed to gdcmElValue for obvious consistency reasons.

ChangeLog
TODO
src/gdcmElValSet.cxx
src/gdcmElValSet.h
src/gdcmElValue.cxx
src/gdcmElValue.h
src/gdcmFile.cxx
src/gdcmHeader.cxx
src/gdcmHeader.h

index 8fc0034d7dfa8c3edbf7340f3e84a303cc7268bc..ed60d8949e1ab9bc60f7ee7bc25f3a9e248b3607 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2003-03-12 Eric Boix <Eric.Boix@creatis.insa-lyon.fr>
+      * gdcmPython/__init__.py fix of GDCM_DICT_PATH which was not
+        to exported to the environement (see gdcmPython/testSuite.py
+        for usage).
+      * src/gdcmElValSet.[cxx/h], gdcmElValue.[cxx/h], gdcmFile.[cxx/h],
+        gdcmHeader.[cxx/h]: ElValSet class renamed to gdcmElValSet, and
+        ElValue class renamed to gdcmElValue for obvious consistency reasons.
+
 2003-03-10 Eric Boix <Eric.Boix@creatis.insa-lyon.fr>
       * gdcmPython/__init__.py doesn't crash anymore when running in
         in InstallMode or PreInstallMode but with an environement given
diff --git a/TODO b/TODO
index 1ba8deee56dabe95719501d506dcfbcf512110c7..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]).
index cfdae3cfe9b9fb5b4a29fc59fa79ba7e0955cd46..6c140ae11eac05b9ac122a172d0497d1ee198bbb 100644 (file)
@@ -1,4 +1,4 @@
-// gdcmElValSet.cxx
+// $Header: /cvs/public/gdcm/src/Attic/gdcmElValSet.cxx,v 1.17 2003/03/12 21:33:20 frog Exp $
 
 #include "gdcmUtil.h"
 #include "gdcmElValSet.h"
@@ -8,16 +8,16 @@
 static void Tokenize (const string& str, vector<string>& tokens, const string& delimiters = " ");
 
 
-TagElValueHT & ElValSet::GetTagHt(void) {
+TagElValueHT & gdcmElValSet::GetTagHt(void) {
        return tagHt;
 }
 
-void ElValSet::Add(ElValue * newElValue) {
+void gdcmElValSet::Add(gdcmElValue * newElValue) {
        tagHt [newElValue->GetKey()]  = newElValue;
        NameHt[newElValue->GetName()] = newElValue;
 }
 
-void ElValSet::Print(ostream & os) {
+void gdcmElValSet::Print(ostream & os) {
        for (TagElValueHT::iterator tag = tagHt.begin();
                  tag != tagHt.end();
                  ++tag){
@@ -28,7 +28,7 @@ void ElValSet::Print(ostream & os) {
        }
 } 
 
-void ElValSet::PrintByName(ostream & os) {
+void gdcmElValSet::PrintByName(ostream & os) {
        for (TagElValueNameHT::iterator tag = NameHt.begin();
                  tag != NameHt.end();
                  ++tag){
@@ -39,50 +39,51 @@ void ElValSet::PrintByName(ostream & os) {
        }
 }
 
-ElValue* ElValSet::GetElementByNumber(guint32 group, guint32 element) {
+gdcmElValue* gdcmElValSet::GetElementByNumber(guint32 group, guint32 element) {
        TagKey key = gdcmDictEntry::TranslateToKey(group, element);
        if ( ! tagHt.count(key))
-               return (ElValue*)0;
+               return (gdcmElValue*)0;
        if (tagHt.count(key) > 1)
-               dbg.Verbose(0, "ElValSet::GetElementByNumber",
+               dbg.Verbose(0, "gdcmElValSet::GetElementByNumber",
                            "multiple entries for this key (FIXME) !");
        return tagHt.find(key)->second;
 }
 
-ElValue* ElValSet::GetElementByName(string TagName) {
+gdcmElValue* gdcmElValSet::GetElementByName(string TagName) {
    if ( ! NameHt.count(TagName))
-      return (ElValue*)0;
+      return (gdcmElValue*)0;
    if (NameHt.count(TagName) > 1)
-      dbg.Verbose(0, "ElValSet::GetElement",
+      dbg.Verbose(0, "gdcmElValSet::GetElement",
                   "multipe entries for this key (FIXME) !");
    return NameHt.find(TagName)->second;
 }
 
-string ElValSet::GetElValueByNumber(guint32 group, guint32 element) {
+string gdcmElValSet::GetElValueByNumber(guint32 group, guint32 element) {
        TagKey key = gdcmDictEntry::TranslateToKey(group, element);
        if ( ! tagHt.count(key))
                return "gdcm::Unfound";
        if (tagHt.count(key) > 1)
-               dbg.Verbose(0, "ElValSet::GetElValueByNumber",
+               dbg.Verbose(0, "gdcmElValSet::GetElValueByNumber",
                            "multiple entries for this key (FIXME) !");
        return tagHt.find(key)->second->GetValue();
 }
 
-string ElValSet::GetElValueByName(string TagName) {
+string gdcmElValSet::GetElValueByName(string TagName) {
        if ( ! NameHt.count(TagName))
                return "gdcm::Unfound";
        if (NameHt.count(TagName) > 1)
-               dbg.Verbose(0, "ElValSet::GetElValue",
+               dbg.Verbose(0, "gdcmElValSet::GetElValue",
                            "multipe entries for this key (FIXME) !");
        return NameHt.find(TagName)->second->GetValue();
 }
 
-int ElValSet::SetElValueByNumber(string content, guint32 group, guint32 element) {
+int gdcmElValSet::SetElValueByNumber(string content,
+                                     guint32 group, guint32 element) {
        TagKey key = gdcmDictEntry::TranslateToKey(group, element);
        if ( ! tagHt.count(key))
                return 0;
        if (tagHt.count(key) > 1) {
-               dbg.Verbose(0, "ElValSet::SetElValueByNumber",
+               dbg.Verbose(0, "gdcmElValSet::SetElValueByNumber",
                            "multiple entries for this key (FIXME) !");
                return (0); 
        }                                      
@@ -97,11 +98,11 @@ int ElValSet::SetElValueByNumber(string content, guint32 group, guint32 element)
        return(1);              
 }
 
-int ElValSet::SetElValueByName(string content, string TagName) {
+int gdcmElValSet::SetElValueByName(string content, string TagName) {
        if ( ! NameHt.count(TagName))
                return 0;
        if (NameHt.count(TagName) > 1) {
-               dbg.Verbose(0, "ElValSet::SetElValueByName",
+               dbg.Verbose(0, "gdcmElValSet::SetElValueByName",
                            "multipe entries for this key (FIXME) !");
                return 0;
        }
@@ -111,26 +112,28 @@ int ElValSet::SetElValueByName(string content, string TagName) {
 }
 
 
-int ElValSet::SetElValueLengthByNumber(guint32 l, guint32 group, guint32 element) {
+int gdcmElValSet::SetElValueLengthByNumber(guint32 l,
+                                           guint32 group, guint32 element) {
        TagKey key = gdcmDictEntry::TranslateToKey(group, element);
        if ( ! tagHt.count(key))
                return 0;
        if (tagHt.count(key) > 1) {
-               dbg.Verbose(0, "ElValSet::SetElValueLengthByNumber",
+               dbg.Verbose(0, "gdcmElValSet::SetElValueLengthByNumber",
                            "multiple entries for this key (FIXME) !");
                return (0); 
        }                                      
+   // FIXME JPR: comments in English please !
        // m à j LgrElem 
        tagHt[key]->SetLength(l);        
        return(1);              
 }
 
 
-int ElValSet::SetElValueLengthByName(guint32 l, string TagName) {
+int gdcmElValSet::SetElValueLengthByName(guint32 l, string TagName) {
        if ( ! NameHt.count(TagName))
                return 0;
        if (NameHt.count(TagName) > 1) {
-               dbg.Verbose(0, "ElValSet::SetElValueByName",
+               dbg.Verbose(0, "gdcmElValSet::SetElValueByName",
                            "multipe entries for this key (FIXME) !");
                return 0;
        }
@@ -139,7 +142,7 @@ int ElValSet::SetElValueLengthByName(guint32 l, string TagName) {
 }
 
 
-int ElValSet::Write(FILE * _fp) {
+int gdcmElValSet::Write(FILE * _fp) {
 
 // ATTENTION : fonction non terminée (commitée a titre de precaution)
 
@@ -162,7 +165,7 @@ int ElValSet::Write(FILE * _fp) {
        // Utilisées pour le calcul Group Length
        int deja = 0;
        guint32 lgrCalcGroupe=0;
-       ElValue *elem, *elemZ, *elemZPrec;
+       gdcmElValue *elem, *elemZ, *elemZPrec;
        guint16 grCourant = 0;
        
        // Question :
@@ -195,7 +198,7 @@ int ElValSet::Write(FILE * _fp) {
                        if(elemZ->GetElement() != 0x0000) {     // pas d'element 'Lgr groupe'
                                // On crée
                                gdcmDictEntry * tagZ = new gdcmDictEntry(grCourant, 0x0000, "UL");
-                               elemZ = new ElValue(tagZ); // on le cree
+                               elemZ = new gdcmElValue(tagZ); // on le cree
                                elemZ->SetLength(4);
                                Add(elemZ);                                      // On l'accroche à sa place    
                        }       
@@ -304,7 +307,7 @@ int ElValSet::Write(FILE * _fp) {
 
 
 
-int ElValSet::WriteAcr(FILE * _fp) {
+int gdcmElValSet::WriteAcr(FILE * _fp) {
 
 
 // ATTENTION : fonction non terminée (commitée a titre de precaution)
@@ -330,7 +333,7 @@ int ElValSet::WriteAcr(FILE * _fp) {
        // Utilisées pour le calcul Group Length
        int deja = 0;
        guint32 lgrCalcGroupe=0;
-       ElValue *elem, *elemZ, *elemZPrec;
+       gdcmElValue *elem, *elemZ, *elemZPrec;
        guint16 grCourant = 0;
        
        // Question :
@@ -363,7 +366,7 @@ int ElValSet::WriteAcr(FILE * _fp) {
                        if(elemZ->GetElement() != 0x0000) {     // pas d'element 'Lgr groupe'
                                // On crée
                                gdcmDictEntry * tagZ = new gdcmDictEntry(grCourant, 0x0000, "UL");
-                               elemZ = new ElValue(tagZ); // on le cree
+                               elemZ = new gdcmElValue(tagZ); // on le cree
                                elemZ->SetLength(4);
                                Add(elemZ);                                      // On l'accroche à sa place    
                        }       
index eccd5584ab8e15cc35cf6bdc8f4b210f2650fdb5..ba351707f53fd19f218993c2bfe2c658c6029744 100644 (file)
@@ -1,4 +1,4 @@
-// gdcmElValSet.h
+// $Header: /cvs/public/gdcm/src/Attic/gdcmElValSet.h,v 1.3 2003/03/12 21:33:20 frog Exp $
 
 #ifndef GDCMELVALSET_H
 #define GDCMELVALSET_H
 
 ////////////////////////////////////////////////////////////////////////////
 // Container for a set of successfully parsed ElValues.
-typedef map<TagKey, ElValue*> TagElValueHT;
-typedef map<string, ElValue*> TagElValueNameHT;
+typedef map<TagKey, gdcmElValue*> TagElValueHT;
+typedef map<string, gdcmElValue*> TagElValueNameHT;
 
-class GDCM_EXPORT ElValSet {
+class GDCM_EXPORT gdcmElValSet {
        TagElValueHT tagHt;             // Both accesses with a TagKey or with a
        TagElValueNameHT NameHt;        // the DictEntry.Name are required.
 public:        
-       void Add(ElValue*);             
+       void Add(gdcmElValue*);         
        void Print(ostream &);
        void PrintByName(ostream &);
        int  Write(FILE *fp);
        int  WriteAcr(FILE *fp);
-       ElValue* GetElementByNumber(guint32 group, guint32 element);
-       ElValue* GetElementByName  (string);
+       gdcmElValue* GetElementByNumber(guint32 group, guint32 element);
+       gdcmElValue* GetElementByName  (string);
        string   GetElValueByNumber(guint32 group, guint32 element);
        string   GetElValueByName  (string);
        
index ecd680f10c0f8b69095ceda3e3e5ae2995297cad..61b216ba75a53a8eec25c24828c685209f631028 100644 (file)
@@ -9,7 +9,7 @@
  * @param   in Pointer to existing dictionary entry
  */
 
-ElValue::ElValue(gdcmDictEntry* in) {
+gdcmElValue::gdcmElValue(gdcmDictEntry* in) {
        ImplicitVr = false;
        entry = in;
 }
index b1305bc6b1590c347f5df32a88984de54af716c2..a2dd3c2c6e594cb6c8d2e540459adcb8df21144e 100644 (file)
@@ -1,15 +1,16 @@
-// gdcmElValue.h
+// $Header: /cvs/public/gdcm/src/Attic/gdcmElValue.h,v 1.2 2003/03/12 21:33:20 frog Exp $
 
 #ifndef GDCMELVALUE_H
 #define GDCMELVALUE_H
 
 #include "gdcmDictEntry.h"
+class gdcmHeader;
 
 
 ///////////////////////////////////////////////////////////////////////////
 // The dicom header of a Dicom file contains a set of such ELement VALUES
 // (when successfuly parsed against a given Dicom dictionary)
-class GDCM_EXPORT ElValue {
+class GDCM_EXPORT gdcmElValue {
 private:
        gdcmDictEntry *entry;
        guint32 LgrElem;
@@ -17,11 +18,15 @@ private:
                               // elements happen to be implicit. Flag them here
                               // since we can't use the entry->vr without breaking
                               // the underlying dictionary.
+       void SetOffset(size_t of){ Offset = of; };
+   // FIXME: In fact we should be more specific and use :
+   //friend gdcmElValue * gdcmHeader::ReadNextElement(void);
+   friend class gdcmHeader;
 public:
        string  value;
        size_t Offset;     // Offset from the begining of file for direct user access
        
-       ElValue(gdcmDictEntry*);
+       gdcmElValue(gdcmDictEntry*);
        void SetDictEntry(gdcmDictEntry *NewEntry) { entry = NewEntry; };
        bool   IsVrUnknown(void) { return entry->IsVrUnknown(); };
        void SetImplicitVr(void) { ImplicitVr = true; };
@@ -36,18 +41,11 @@ public:
        void SetLength(guint32 l){ LgrElem = l;               };
        guint32 GetLength(void)  { return LgrElem;            };
        
-       // Question : SetLength est public 
-       // (sinon, on ne pourrait pas l'appeler dans ElValSet)
-       // alors que *personne* ne devrait s'en servir !
-       // c'est *forcément* la lgr de la string 'value', non?
-
        void SetValue(string val){ value = val; };
        string  GetValue(void)   { return value;};
 
-       void SetOffset(size_t of){ Offset = of; };
        size_t  GetOffset(void)  { return Offset;};
-       // Question : SetOffset est public ...
-       // Quel utilisateur serait ammené à modifier l'Offset ?
 };
 
+
 #endif
index 0a71c03e104d1099805d51b33570bd7dc62618ad..a586dc7dc935aa1bd34481b27ff84984c052cde7 100644 (file)
@@ -8,16 +8,16 @@ static void _Swap(void* im, int swap, int lgr, int nb);
 /**
  * \ingroup   gdcmFile
  * \brief Constructor dedicated to writing a new DICOMV3 part10 compliant
- * \file (see SetFileName, SetDcmTag and Write)
- * \Opens (in read only and when possible) an existing file and checks
- * \for DICOM compliance. Returns NULL on failure.
- * \Note: the in-memory representation of all available tags found in
- * \the DICOM header is post-poned to first header information access.
- * \This avoid a double parsing of public part of the header when
- * \one sets an a posteriori shadow dictionary (efficiency can be
- * \seen as a side effect).   
+ *        file (see SetFileName, SetDcmTag and Write)
+ *        Opens (in read only and when possible) an existing file and checks
+ *        for DICOM compliance. Returns NULL on failure.
+ * \Note  the in-memory representation of all available tags found in
+ *        the DICOM header is post-poned to first header information access.
+ *        This avoid a double parsing of public part of the header when
+ *        one sets an a posteriori shadow dictionary (efficiency can be
+ *        seen as a side effect).   
  *
- * @param 
+ * @param filename file to be opened for parsing
  *
  * @return     
  */
@@ -29,6 +29,7 @@ gdcmFile::gdcmFile(string & filename)
 
 
 /////////////////////////////////////////////////////////////////
+// FIXME JPR: comments in English please !
 /**
  * \ingroup   gdcmFile
  * \brief     Renvoie la longueur A ALLOUER pour recevoir les pixels de l'image
@@ -439,7 +440,7 @@ int gdcmFile::WriteDcm (string nomFichier) {
        fwrite("DICM",4,1,fp1);
 
        // un accesseur de + est obligatoire ???
-       // pourtant le ElValSet contenu dans le gdcmHeader 
+       // pourtant le gdcmElValSet contenu dans le gdcmHeader 
        // ne devrait pas être visible par l'utilisateur final (?)
        
        GetPubElVals().Write(fp1);
@@ -479,7 +480,7 @@ int gdcmFile::WriteAcr (string nomFichier) {
        } 
 
        // un accesseur de + est obligatoire ???
-       // pourtant le ElValSet contenu dans le gdcmHeader 
+       // pourtant le gdcmElValSet contenu dans le gdcmHeader 
        // ne devrait pas être visible par l'utilisateur final (?)
        
        GetPubElVals().WriteAcr(fp1);
index 37b2e1ecbb4b57c88f20e7555ac2def7829a1263..10f5700dac5c3445f5e847d8d1f4f801d823b978 100644 (file)
@@ -259,7 +259,7 @@ void gdcmHeader::GetPixels(size_t lgrTotale, void* _Pixels) {
  * \ingroup   gdcmHeader
  * \brief     Find the value representation of the current tag.
  */
-void gdcmHeader::FindVR( ElValue *ElVal) {
+void gdcmHeader::FindVR( gdcmElValue *ElVal) {
    if (filetype != ExplicitVR)
       return;
 
@@ -343,7 +343,7 @@ void gdcmHeader::FindVR( ElValue *ElVal) {
  * @return  True when ImplicitVRLittleEndian found. False in all other cases.
  */
 bool gdcmHeader::IsImplicitVRLittleEndianTransferSyntax(void) {
-   ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010);
+   gdcmElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010);
    if ( !Element )
       return false;
    LoadElementValueSafe(Element);
@@ -361,7 +361,7 @@ bool gdcmHeader::IsImplicitVRLittleEndianTransferSyntax(void) {
  * @return  True when ExplicitVRLittleEndian found. False in all other cases.
  */
 bool gdcmHeader::IsExplicitVRLittleEndianTransferSyntax(void) {
-   ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010);
+   gdcmElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010);
    if ( !Element )
       return false;
    LoadElementValueSafe(Element);
@@ -379,7 +379,7 @@ bool gdcmHeader::IsExplicitVRLittleEndianTransferSyntax(void) {
  * @return  True when DeflatedExplicitVRLittleEndian found. False in all other cases.
  */
 bool gdcmHeader::IsDeflatedExplicitVRLittleEndianTransferSyntax(void) {
-   ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010);
+   gdcmElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010);
    if ( !Element )
       return false;
    LoadElementValueSafe(Element);
@@ -397,7 +397,7 @@ bool gdcmHeader::IsDeflatedExplicitVRLittleEndianTransferSyntax(void) {
  * @return  True when big endian found. False in all other cases.
  */
 bool gdcmHeader::IsExplicitVRBigEndianTransferSyntax(void) {
-   ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010);
+   gdcmElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010);
    if ( !Element )
       return false;
    LoadElementValueSafe(Element);
@@ -415,7 +415,7 @@ bool gdcmHeader::IsExplicitVRBigEndianTransferSyntax(void) {
  * @return  True when JPEGBaseLineProcess1found. False in all other cases.
  */
 bool gdcmHeader::IsJPEGBaseLineProcess1TransferSyntax(void) {
-   ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010);
+   gdcmElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010);
    if ( !Element )
       return false;
    LoadElementValueSafe(Element);
@@ -428,7 +428,7 @@ bool gdcmHeader::IsJPEGBaseLineProcess1TransferSyntax(void) {
 // faire qq chose d'intelligent a la place de ça
 
 bool gdcmHeader::IsJPEGLossless(void) {
-   ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010);
+   gdcmElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010);
    if ( !Element )
       return false;
    LoadElementValueSafe(Element);
@@ -447,7 +447,7 @@ bool gdcmHeader::IsJPEGLossless(void) {
  * @return  True when JPEGExtendedProcess2-4 found. False in all other cases.
  */
 bool gdcmHeader::IsJPEGExtendedProcess2_4TransferSyntax(void) {
-   ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010);
+   gdcmElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010);
    if ( !Element )
       return false;
    LoadElementValueSafe(Element);
@@ -465,7 +465,7 @@ bool gdcmHeader::IsJPEGExtendedProcess2_4TransferSyntax(void) {
  * @return  True when JPEGExtendedProcess3-5 found. False in all other cases.
  */
 bool gdcmHeader::IsJPEGExtendedProcess3_5TransferSyntax(void) {
-   ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010);
+   gdcmElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010);
    if ( !Element )
       return false;
    LoadElementValueSafe(Element);
@@ -484,7 +484,7 @@ bool gdcmHeader::IsJPEGExtendedProcess3_5TransferSyntax(void) {
  *          other cases.
  */
 bool gdcmHeader::IsJPEGSpectralSelectionProcess6_8TransferSyntax(void) {
-   ElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010);
+   gdcmElValue* Element = PubElVals.GetElementByNumber(0x0002, 0x0010);
    if ( !Element )
       return false;
    LoadElementValueSafe(Element);
@@ -500,7 +500,7 @@ bool gdcmHeader::IsJPEGSpectralSelectionProcess6_8TransferSyntax(void) {
  *          the parser went Jabberwocky) one can hope improving things by
  *          applying this heuristic.
  */
-void gdcmHeader::FixFoundLength(ElValue * ElVal, guint32 FoundLength) {
+void gdcmHeader::FixFoundLength(gdcmElValue * ElVal, guint32 FoundLength) {
    if ( FoundLength == 0xffffffff)
       FoundLength = 0;
    ElVal->SetLength(FoundLength);
@@ -544,7 +544,7 @@ guint32 gdcmHeader::FindLengthOB(void) {
    return TotalLength;
 }
 
-void gdcmHeader::FindLength(ElValue * ElVal) {
+void gdcmHeader::FindLength(gdcmElValue * ElVal) {
    guint16 element = ElVal->GetElement();
    string  vr      = ElVal->GetVR();
    guint16 length16;
@@ -683,7 +683,7 @@ void gdcmHeader::SkipBytes(guint32 NBytes) {
    (void)fseek(fp, (long)NBytes, SEEK_CUR);
 }
 
-void gdcmHeader::SkipElementValue(ElValue * ElVal) {
+void gdcmHeader::SkipElementValue(gdcmElValue * ElVal) {
    SkipBytes(ElVal->GetLength());
 }
 
@@ -703,7 +703,7 @@ void gdcmHeader::SetMaxSizeLoadElementValue(long NewSize) {
  *                than the value specified with
  *                gdcmHeader::SetMaxSizeLoadElementValue()
  */
-void gdcmHeader::LoadElementValue(ElValue * ElVal) {
+void gdcmHeader::LoadElementValue(gdcmElValue * ElVal) {
    size_t item_read;
    guint16 group  = ElVal->GetGroup();
    guint16 elem   = ElVal->GetElement();
@@ -851,7 +851,7 @@ void gdcmHeader::LoadElementValue(ElValue * ElVal) {
  * @param ElVal   Element whose value shall be loaded. 
  * @return  
  */
-void gdcmHeader::LoadElementValueSafe(ElValue * ElVal) {
+void gdcmHeader::LoadElementValueSafe(gdcmElValue * ElVal) {
    long PositionOnEntry = ftell(fp);
    LoadElementValue(ElVal);
    fseek(fp, PositionOnEntry, SEEK_SET);
@@ -894,17 +894,17 @@ guint32 gdcmHeader::ReadInt32(void) {
  * @param   Group group   of the underlying DictEntry
  * @param   Elem  element of the underlying DictEntry
  */
-ElValue* gdcmHeader::NewElValueByKey(guint16 Group, guint16 Elem) {
+gdcmElValue* gdcmHeader::NewElValueByKey(guint16 Group, guint16 Elem) {
    // Find out if the tag we encountered is in the dictionaries:
    gdcmDictEntry * NewTag = GetDictEntryByKey(Group, Elem);
    if (!NewTag)
       NewTag = new gdcmDictEntry(Group, Elem);
 
-   ElValue* NewElVal = new ElValue(NewTag);
+   gdcmElValue* NewElVal = new gdcmElValue(NewTag);
    if (!NewElVal) {
       dbg.Verbose(1, "gdcmHeader::NewElValueByKey",
-                  "failed to allocate ElValue");
-      return (ElValue*)0;
+                  "failed to allocate gdcmElValue");
+      return (gdcmElValue*)0;
    }
    return NewElVal;
 }
@@ -916,17 +916,17 @@ ElValue* gdcmHeader::NewElValueByKey(guint16 Group, guint16 Elem) {
  *          a default one when absent.
  * @param   Name    Name of the underlying DictEntry
  */
-ElValue* gdcmHeader::NewElValueByName(string Name) {
+gdcmElValue* gdcmHeader::NewElValueByName(string Name) {
 
    gdcmDictEntry * NewTag = GetDictEntryByName(Name);
    if (!NewTag)
       NewTag = new gdcmDictEntry(0xffff, 0xffff, "LO", "Unknown", Name);
 
-   ElValue* NewElVal = new ElValue(NewTag);
+   gdcmElValue* NewElVal = new gdcmElValue(NewTag);
    if (!NewElVal) {
       dbg.Verbose(1, "gdcmHeader::ObtainElValueByName",
-                  "failed to allocate ElValue");
-      return (ElValue*)0;
+                  "failed to allocate gdcmElValue");
+      return (gdcmElValue*)0;
    }
    return NewElVal;
 }  
@@ -936,24 +936,24 @@ ElValue* gdcmHeader::NewElValueByName(string Name) {
  * \brief   Read the next tag but WITHOUT loading it's value
  * @return  On succes the newly created ElValue, NULL on failure.      
  */
-ElValue * gdcmHeader::ReadNextElement(void) {
+gdcmElValue * gdcmHeader::ReadNextElement(void) {
    guint16 g;
    guint16 n;
-   ElValue * NewElVal;
+   gdcmElValue * NewElVal;
    
    g = ReadInt16();
    n = ReadInt16();
    if (errno == 1)
       // We reached the EOF (or an error occured) and header parsing
       // has to be considered as finished.
-      return (ElValue *)0;
+      return (gdcmElValue *)0;
    
    NewElVal = NewElValueByKey(g, n);
    FindVR(NewElVal);
    FindLength(NewElVal);
    if (errno == 1)
       // Call it quits
-      return (ElValue *)0;
+      return (gdcmElValue *)0;
    NewElVal->SetOffset(ftell(fp));
    return NewElVal;
 }
@@ -965,7 +965,7 @@ ElValue * gdcmHeader::ReadNextElement(void) {
  * @param   ElVal The element value on which to apply the predicate.
  * @return  The result of the heuristical predicate.
  */
-bool gdcmHeader::IsAnInteger(ElValue * ElVal) {
+bool gdcmHeader::IsAnInteger(gdcmElValue * ElVal) {
    guint16 group   = ElVal->GetGroup();
    guint16 element = ElVal->GetElement();
    string  vr      = ElVal->GetVR();
@@ -1048,7 +1048,7 @@ size_t gdcmHeader::GetPixelOffset(void) {
       numPixel = 0x1010;
    else
       numPixel = 0x0010;
-   ElValue* PixelElement = PubElVals.GetElementByNumber(grPixel, numPixel);
+   gdcmElValue* PixelElement = PubElVals.GetElementByNumber(grPixel, numPixel);
    if (PixelElement)
       return PixelElement->GetOffset();
    else
@@ -1139,7 +1139,7 @@ string gdcmHeader::GetPubElValByNumber(guint16 group, guint16 element) {
  *          and the string "gdcm::Unfound" otherwise.
  */
 string gdcmHeader::GetPubElValRepByNumber(guint16 group, guint16 element) {
-   ElValue* elem =  PubElVals.GetElementByNumber(group, element);
+   gdcmElValue* elem =  PubElVals.GetElementByNumber(group, element);
    if ( !elem )
       return "gdcm::Unfound";
    return elem->GetVR();
@@ -1171,7 +1171,7 @@ string gdcmHeader::GetPubElValByName(string TagName) {
  *          and the string "gdcm::Unfound" otherwise.
  */
 string gdcmHeader::GetPubElValRepByName(string TagName) {
-   ElValue* elem =  PubElVals.GetElementByName(TagName);
+   gdcmElValue* elem =  PubElVals.GetElementByName(TagName);
    if ( !elem )
       return "gdcm::Unfound";
    return elem->GetVR();
@@ -1205,7 +1205,7 @@ string gdcmHeader::GetShaElValByNumber(guint16 group, guint16 element) {
  *          and the string "gdcm::Unfound" otherwise.
  */
 string gdcmHeader::GetShaElValRepByNumber(guint16 group, guint16 element) {
-   ElValue* elem =  ShaElVals.GetElementByNumber(group, element);
+   gdcmElValue* elem =  ShaElVals.GetElementByNumber(group, element);
    if ( !elem )
       return "gdcm::Unfound";
    return elem->GetVR();
@@ -1237,7 +1237,7 @@ string gdcmHeader::GetShaElValByName(string TagName) {
  *          and the string "gdcm::Unfound" otherwise.
  */
 string gdcmHeader::GetShaElValRepByName(string TagName) {
-   ElValue* elem =  ShaElVals.GetElementByName(TagName);
+   gdcmElValue* elem =  ShaElVals.GetElementByName(TagName);
    if ( !elem )
       return "gdcm::Unfound";
    return elem->GetVR();
@@ -1321,7 +1321,7 @@ string gdcmHeader::GetElValRepByName(string TagName) {
 
 /**
  * \ingroup gdcmHeader
- * \brief   Accesses an existing ElValue in the PubElVals of this instance
+ * \brief   Accesses an existing gdcmElValue in the PubElVals of this instance
  *          through it's (group, element) and modifies it's content with
  *          the given value.
  * @param   content new value to substitute with
@@ -1336,7 +1336,7 @@ int gdcmHeader::SetPubElValByNumber(string content, guint16 group,
 
 /**
  * \ingroup gdcmHeader
- * \brief   Accesses an existing ElValue in the PubElVals of this instance
+ * \brief   Accesses an existing gdcmElValue in the PubElVals of this instance
  *          through tag name and modifies it's content with the given value.
  * @param   content new value to substitute with
  * @param   TagName name of the tag to be modified
@@ -1347,7 +1347,7 @@ int gdcmHeader::SetPubElValByName(string content, string TagName) {
 
 /**
  * \ingroup gdcmHeader
- * \brief   Accesses an existing ElValue in the PubElVals of this instance
+ * \brief   Accesses an existing gdcmElValue in the PubElVals of this instance
  *          through it's (group, element) and modifies it's length with
  *          the given value.
  *             NOT FOR BOZOs !
@@ -1363,7 +1363,7 @@ int gdcmHeader::SetPubElValLengthByNumber(guint32 lgr, guint16 group,
 
 /**
  * \ingroup gdcmHeader
- * \brief   Accesses an existing ElValue in the ShaElVals of this instance
+ * \brief   Accesses an existing gdcmElValue in the ShaElVals of this instance
  *          through it's (group, element) and modifies it's content with
  *          the given value.
  * @param   content new value to substitute with
@@ -1376,12 +1376,9 @@ int gdcmHeader::SetShaElValByNumber(string content,
    return (  ShaElVals.SetElValueByNumber (content, group, element) );
 }
 
-
-
-
 /**
  * \ingroup gdcmHeader
- * \brief   Accesses an existing ElValue in the ShaElVals of this instance
+ * \brief   Accesses an existing gdcmElValue in the ShaElVals of this instance
  *          through tag name and modifies it's content with the given value.
  * @param   content new value to substitute with
  * @param   TagName name of the tag to be modified
@@ -1395,7 +1392,7 @@ int gdcmHeader::SetShaElValByName(string content, string TagName) {
  * \brief   Parses the header of the file but WITHOUT loading element values.
  */
 void gdcmHeader::ParseHeader(bool exception_on_error) throw(gdcmFormatError) {
-   ElValue * newElValue = (ElValue *)0;
+   gdcmElValue * newElValue = (gdcmElValue *)0;
    
    rewind(fp);
    CheckSwap();
@@ -1408,7 +1405,7 @@ void gdcmHeader::ParseHeader(bool exception_on_error) throw(gdcmFormatError) {
 /**
  * \ingroup gdcmHeader
  * \brief   Once the header is parsed add some gdcm convenience/helper elements
- *          in the ElValSet. For example add:
+ *          in the gdcmElValSet. For example add:
  *          - gdcmImageType which is an entry containing a short for the
  *            type of image and whose value ranges in 
  *               I8   (unsigned 8 bit image)
@@ -1418,7 +1415,7 @@ void gdcmHeader::ParseHeader(bool exception_on_error) throw(gdcmFormatError) {
  *            the ones of the official DICOM fields Rows, Columns and Planes.
  */
 void gdcmHeader::AddAndDefaultElements(void) {
-   ElValue* NewEntry = (ElValue*)0;
+   gdcmElValue* NewEntry = (gdcmElValue*)0;
 
    NewEntry = NewElValueByName("gdcmXSize");
    NewEntry->SetValue(GetElValByName("Rows"));
index fa33ebc8d039e96605688aedf330fcac8b02088b..3e675c47310430c39e65fb7a8684f67a6407c4bf 100644 (file)
@@ -47,9 +47,9 @@ private:
    gdcmDict* RefShaDict;
 
    /// ELement VALueS parsed with the PUBlic dictionary.
-   ElValSet PubElVals;
+   gdcmElValSet PubElVals;
    /// ELement VALueS parsed with the SHAdow dictionary.
-   ElValSet ShaElVals;
+   gdcmElValSet ShaElVals;
    /// Refering underlying filename.
    string filename; 
    FILE * fp;
@@ -86,16 +86,16 @@ private:
    gdcmDictEntry * GetDictEntryByName(string name);
 
    // ElValue related utilities
-   ElValue * ReadNextElement(void);
-   ElValue * NewElValueByKey(guint16 group, guint16 element);
-   ElValue * NewElValueByName(string name);
-   void FindLength(ElValue *);
-   void FindVR(ElValue *);
-   void LoadElementValue(ElValue *);
-   void LoadElementValueSafe(ElValue *);
-   void SkipElementValue(ElValue *);
-   void FixFoundLength(ElValue*, guint32);
-   bool IsAnInteger(ElValue *);
+   gdcmElValue * ReadNextElement(void);
+   gdcmElValue * NewElValueByKey(guint16 group, guint16 element);
+   gdcmElValue * NewElValueByName(string name);
+   void FindLength(gdcmElValue *);
+   void FindVR(gdcmElValue *);
+   void LoadElementValue(gdcmElValue *);
+   void LoadElementValueSafe(gdcmElValue *);
+   void SkipElementValue(gdcmElValue *);
+   void FixFoundLength(gdcmElValue*, guint32);
+   bool IsAnInteger(gdcmElValue *);
    
    bool IsImplicitVRLittleEndianTransferSyntax(void);
    bool IsExplicitVRLittleEndianTransferSyntax(void);
@@ -165,7 +165,7 @@ public:
    int gdcmHeader::SetPubElValLengthByNumber(guint32 lgr, guint16 group,
                                     guint16 element);
 
-   ElValSet GetPubElVals() { return(PubElVals); }
+   gdcmElValSet GetPubElVals() { return(PubElVals); }
 };
 
 #endif