X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=blobdiff_plain;f=src%2FgdcmDocEntryArchive.cxx;h=c82a1e96c2c779a8010788813bf56c45468b87b5;hb=8401b1407f4b482a8d9e8cf6759009d1076b6a7c;hp=ee4f5bf565c99506dff8df0977fe941ca4fe0a3e;hpb=d1c68c2c2ae9fadf927053150f7fbc625a7c7366;p=gdcm.git

diff --git a/src/gdcmDocEntryArchive.cxx b/src/gdcmDocEntryArchive.cxx
index ee4f5bf5..c82a1e96 100644
--- a/src/gdcmDocEntryArchive.cxx
+++ b/src/gdcmDocEntryArchive.cxx
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmDocEntryArchive.cxx,v $
   Language:  C++
-  Date:      $Date: 2005/02/01 10:29:55 $
-  Version:   $Revision: 1.11 $
+  Date:      $Date: 2005/10/24 16:00:47 $
+  Version:   $Revision: 1.17 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -45,28 +45,31 @@ DocEntryArchive::~DocEntryArchive()
 //-----------------------------------------------------------------------------
 // Public
 /**
- * \brief   Replace in the Header a DocEntry by the new DocEntry. The last
- *          DocEntry is kept in archive
+ * \brief   Replaces in the Header a DocEntry by the new DocEntry. 
+ *          The initial DocEntry is kept in archive.
  * @param   newEntry New entry to substitute to an other entry of the Header
- * @return  FALSE when an other DocEntry is already archived with the same
- *          generalized key, TRUE otherwise
+ * @return  FALSE when an other DocEntry is already archived with the same key
+ *          TRUE otherwise
  */
 bool DocEntryArchive::Push(DocEntry *newEntry)
 {
-   if(!newEntry)
+   if ( !newEntry )
       return false;
 
    uint16_t group = newEntry->GetDictEntry()->GetGroup();
-   uint16_t elem = newEntry->GetDictEntry()->GetElement();
-   std::string key = DictEntry::TranslateToKey(group,elem);
+   uint16_t elem  = newEntry->GetDictEntry()->GetElement();
+   TagKey key = DictEntry::TranslateToKey(group,elem);
 
-   if( Archive.find(key)==Archive.end() )
+   if ( Archive.find(key) == Archive.end() )
    {
       // Save the old DocEntry if any
-      DocEntry *old = ArchFile->GetDocEntry(group,elem);
-      Archive[key] = old;
-      if( old )
-         ArchFile->RemoveEntryNoDestroy(old);
+      DocEntry *old = ArchFile->GetDocEntry(group, elem);
+      Archive[key]  = old;
+      if ( old )
+      {
+         old->Register();
+         ArchFile->RemoveEntry(old);
+      }
 
       // Set the new DocEntry
       ArchFile->AddEntry(newEntry);
@@ -77,24 +80,27 @@ bool DocEntryArchive::Push(DocEntry *newEntry)
 }
 
 /**
- * \brief   Replace in the Header a DocEntry by the new DocEntry. The last
- *          DocEntry is kept in archive
- * @param   group   Group number of the Entry 
- * @param   elem  Element number of the Entry
- * @return  FALSE when an other DocEntry is already archived with the same
- *          generalized key, TRUE otherwise
+ * \brief   Removes out of the Header a DocEntry.
+ *          (it's kept in archive).
+ * @param   group   Group number of the Entry to remove
+ * @param   elem  Element number of the Entry to remove
+ * @return  FALSE when an other DocEntry is already archived with the same key
+ *          TRUE otherwise
  */
-bool DocEntryArchive::Push(uint16_t group,uint16_t elem)
+bool DocEntryArchive::Push(uint16_t group, uint16_t elem)
 {
-   std::string key = DictEntry::TranslateToKey(group,elem);
+   TagKey key = DictEntry::TranslateToKey(group, elem);
 
-   if( Archive.find(key)==Archive.end() )
+   if ( Archive.find(key)==Archive.end() )
    {
       // Save the old DocEntry if any
-      DocEntry *old = ArchFile->GetDocEntry(group,elem);
+      DocEntry *old = ArchFile->GetDocEntry(group, elem);
       Archive[key] = old;
-      if( old )
-         ArchFile->RemoveEntryNoDestroy(old);
+      if ( old )
+      {
+         old->Register();
+         ArchFile->RemoveEntry(old);
+      }
 
       return true;
    }
@@ -102,28 +108,33 @@ bool DocEntryArchive::Push(uint16_t group,uint16_t elem)
 }
 
 /**
- * \brief   Restore in the Header the DocEntry that have the generalized key. 
- *          The old entry is destroyed.
- * @param   group   Group number of the Entry 
- * @param   elem  Element number of the Entry
- * @return  FALSE when the generalized key isn't in the archive, 
+ * \brief   Restore in the Header the DocEntry specified by (group,element). 
+ *          The archive entry is destroyed.
+ * @param   group   Group number of the Entry to restore
+ * @param   elem  Element number of the Entry to restore
+ * @return  FALSE when the key isn't in the archive, 
  *          TRUE otherwise
  */
-bool DocEntryArchive::Restore(uint16_t group,uint16_t elem)
+bool DocEntryArchive::Restore(uint16_t group, uint16_t elem)
 {
-   std::string key=DictEntry::TranslateToKey(group,elem);
+   TagKey key=DictEntry::TranslateToKey(group, elem);
 
    TagDocEntryHT::iterator restoreIt=Archive.find(key);
-   if( restoreIt!=Archive.end() )
+   if ( restoreIt!=Archive.end() )
    {
       // Delete the new value
-      DocEntry *rem = ArchFile->GetDocEntry(group,elem);
-      if( rem )
+      DocEntry *rem = ArchFile->GetDocEntry(group, elem);
+      if ( rem )
+      {
          ArchFile->RemoveEntry(rem);
+      }
 
       // Restore the old value
-      if( Archive[key] )
-         ArchFile->AddEntry(Archive[key]);
+      if ( restoreIt->second )
+      {
+         ArchFile->AddEntry(restoreIt->second);
+         restoreIt->second->Unregister();
+      }
 
       Archive.erase(restoreIt);
 
@@ -133,8 +144,8 @@ bool DocEntryArchive::Restore(uint16_t group,uint16_t elem)
 }
 
 /**
- * \brief   Remove all DocEntry that are in the archive.  
- *          The entries aren't restored but only destroyed.
+ * \brief   Removes all DocEntry from the archive, and destroy them.  
+ *          The archives entries aren't restored.
  */
 void DocEntryArchive::ClearArchive( )
 {
@@ -142,7 +153,8 @@ void DocEntryArchive::ClearArchive( )
        it!=Archive.end();
        ++it)
    {
-      delete it->second;
+      if(it->second)
+         it->second->Unregister();
    }
    Archive.clear();
 }
@@ -166,7 +178,7 @@ void DocEntryArchive::Print(std::ostream &os)
        it!=Archive.end();
        ++it)
    {
-      if(it->second)
+      if ( it->second )
          it->second->Print(os);
    }
 }