Avoid warnings on C/C++ syle comments

[gdcm.git] / src / gdcmElementSet.cxx

diff --git a/src/gdcmElementSet.cxx b/src/gdcmElementSet.cxx
index 0c662dd070e5ffdb7290754aa60d6dec0691b1bd..a59c4e19d4f41fe8d53fa4403615d13af1b8d685 100644 (file)
--- a/src/gdcmElementSet.cxx
+++ b/src/gdcmElementSet.cxx
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmElementSet.cxx,v $
   Language:  C++
-  Date:      $Date: 2005/10/18 08:35:50 $
-  Version:   $Revision: 1.65 $
+  Date:      $Date: 2005/10/27 11:39:34 $
+  Version:   $Revision: 1.70 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -28,11 +28,9 @@ namespace gdcm
 /**
  * \brief   Constructor for a given ElementSet
  */
-//BOZ depthLevel is not usefull anymore
-ElementSet::ElementSet(int depthLevel) 
+ElementSet::ElementSet() 
           : DocEntrySet()
 {
-  (void)depthLevel;
 }
 
 /**
@@ -56,8 +54,20 @@ void ElementSet::WriteContent(std::ofstream *fp, FileType filetype)
    for (TagDocEntryHT::const_iterator i = TagHT.begin(); 
                                      i != TagHT.end(); 
                                     ++i)
-   {
-      i->second->WriteContent(fp, filetype);
+   { 
+       // depending on the gdcm::Document type 
+       // (gdcm::File; gdcm::DicomDir, (more to come ?)
+       // some groups *cannot* be present.
+       // We hereby protect gdcm for writting stupid things
+       // if they were found in the original document. 
+       if ( !MayIWrite( (i->second)->GetGroup() ) ) 
+          continue;
+      // Skip 'Group Length' element, since it may be wrong.
+      //       except for Group 0002 
+       if ( (i->second)->GetElement() == 0x0000 
+         && (i->second)->GetGroup() != 0x0002 )
+          continue; 
+       i->second->WriteContent(fp, filetype);
    } 
 }
 
@@ -77,6 +87,7 @@ bool ElementSet::AddEntry(DocEntry *newEntry)
    else
    {
       TagHT.insert(TagDocEntryHT::value_type(newEntry->GetKey(), newEntry));
+      newEntry->Register();
       return true;
    }
 }
@@ -91,8 +102,7 @@ bool ElementSet::RemoveEntry( DocEntry *entryToRemove)
    if ( TagHT.count(key) == 1 )
    {
       TagHT.erase(key);
-      //gdcmWarningMacro( "One element erased.");
-      delete entryToRemove;
+      entryToRemove->Unregister();
       return true;
    }
 
@@ -100,24 +110,6 @@ bool ElementSet::RemoveEntry( DocEntry *entryToRemove)
    return false ;
 }
 
-/**
- * \brief   Clear the hash table from given entry BUT keep the entry.
- * @param   entryToRemove Entry to remove.
- */
-bool ElementSet::RemoveEntryNoDestroy(DocEntry *entryToRemove)
-{
-   const TagKey &key = entryToRemove->GetKey();
-   if ( TagHT.count(key) == 1 )
-   {
-      TagHT.erase(key);
-      //gdcmWarningMacro( "One element erased.");
-      return true;
-   }
-
-   gdcmWarningMacro( "Key not present " << key);
-   return false ;
-}
-
 /**
  * \brief   delete all entries in the ElementSet
  */
@@ -127,7 +119,7 @@ void ElementSet::ClearEntry()
    {
       if ( cc->second )
       {
-         delete cc->second;
+         cc->second->Unregister();
       }
    }
    TagHT.clear();