]> Creatis software - gdcm.git/blobdiff - src/gdcmElementSet.cxx
STYLE: Remove old comments
[gdcm.git] / src / gdcmElementSet.cxx
index 7b306ee2b41b65d3b805cd38920e15726d9bbcf2..67ff023baf61206c95c6133a5f878439d028fc43 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmElementSet.cxx,v $
   Language:  C++
-  Date:      $Date: 2005/02/04 14:49:01 $
-  Version:   $Revision: 1.54 $
+  Date:      $Date: 2005/08/24 12:09:13 $
+  Version:   $Revision: 1.63 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -31,7 +31,7 @@ namespace gdcm
  */
 //BOZ depthLevel is not usefull anymore
 ElementSet::ElementSet(int depthLevel) 
-              : DocEntrySet()
+          : DocEntrySet()
 {
   (void)depthLevel;
 }
@@ -70,9 +70,9 @@ bool ElementSet::AddEntry(DocEntry *newEntry)
 {
    const TagKey &key = newEntry->GetKey();
 
-   if( TagHT.count(key) == 1 )
+   if ( TagHT.count(key) == 1 )
    {
-      gdcmVerboseMacro( "Key already present: " << key.c_str());
+      gdcmWarningMacro( "Key already present: " << key );
       return false;
    }
    else
@@ -89,15 +89,15 @@ bool ElementSet::AddEntry(DocEntry *newEntry)
 bool ElementSet::RemoveEntry( DocEntry *entryToRemove)
 {
    const TagKey &key = entryToRemove->GetKey();
-   if( TagHT.count(key) == 1 )
+   if ( TagHT.count(key) == 1 )
    {
       TagHT.erase(key);
-      //gdcmVerboseMacro( "One element erased.");
+      //gdcmWarningMacro( "One element erased.");
       delete entryToRemove;
       return true;
    }
 
-   gdcmVerboseMacro( "Key not present");
+   gdcmWarningMacro( "Key not present : " << key);
    return false ;
 }
 
@@ -108,14 +108,14 @@ bool ElementSet::RemoveEntry( DocEntry *entryToRemove)
 bool ElementSet::RemoveEntryNoDestroy(DocEntry *entryToRemove)
 {
    const TagKey &key = entryToRemove->GetKey();
-   if( TagHT.count(key) == 1 )
+   if ( TagHT.count(key) == 1 )
    {
       TagHT.erase(key);
-      //gdcmVerboseMacro( "One element erased.");
+      //gdcmWarningMacro( "One element erased.");
       return true;
    }
 
-   gdcmVerboseMacro( "Key not present");
+   gdcmWarningMacro( "Key not present " << key);
    return false ;
 }
 
@@ -163,45 +163,6 @@ DocEntry *ElementSet::GetNextEntry()
    return NULL;
 }
 
-/**
- * \brief   Get the first ValEntry while visiting *the* 'zero level' DocEntrySet
- *              (DocEntries out of any Sequence)
- *              This method is designed for Python users
- * \return  The first ValEntry if found, otherwhise NULL
- */
-ValEntry *ElementSet::GetFirstValEntry()
-{
-   gdcm::ValEntry *valEntry;
-   gdcm::DocEntry *d = GetFirstEntry();
-   // an other iterator is needed to allow user iterate 
-   // at the same time both on DocEntries and ValEntries 
-   ItValEntryTagHT = ItTagHT;
-   if ( valEntry = dynamic_cast<gdcm::ValEntry*>(d)) 
-      return valEntry;
-   return  GetNextValEntry();  
-}
-
-/**
- * \brief   Get the next ValEntry while visiting *the* 'zero level' DocEntrySet
- *              (DocEntries out of any Sequence) 
- * \note : meaningfull only if GetFirstValEntry already called 
- * \return  The next ValEntry if found, otherwhise NULL
- */
-ValEntry *ElementSet::GetNextValEntry()
-{
-   gdcm::ValEntry *valEntry;
-   gdcm::DocEntry *d = ItValEntryTagHT->second;
-   ++ItValEntryTagHT; 
-   while( d )
-   {
-      if ( valEntry = dynamic_cast<gdcm::ValEntry*>(d))
-         return valEntry;
-      else
-         return GetNextValEntry(); 
-   }
-   return 0;
-}
-
 /**
  * \brief  retrieves a Dicom Element using (group, element)
  * @param   group  Group number of the searched Dicom Element 
@@ -235,7 +196,7 @@ void ElementSet::Print(std::ostream &os, std::string const & )
 {
    for( TagDocEntryHT::const_iterator i = TagHT.begin(); i != TagHT.end(); ++i)
    {
-      DocEntryentry = i->second;
+      DocEntry *entry = i->second;
 
       entry->SetPrintLevel(PrintLevel);
       entry->Print(os);