]> Creatis software - gdcm.git/commitdiff
* src/gdcmDocument.[h|cxx], gdcmFile.[h|cxx], gdcmHeader.[h|cxx]:
authorfrog <frog>
Fri, 30 Jul 2004 16:09:25 +0000 (16:09 +0000)
committerfrog <frog>
Fri, 30 Jul 2004 16:09:25 +0000 (16:09 +0000)
    skip_shadow removed from gdcmHeader constructor and dependencies.
  * Example/*.cxx and Test/*.cxx changed accordingly.

18 files changed:
ChangeLog
Example/PrintDocument.cxx
Example/PrintFile.cxx
Example/PrintHeader.cxx
Example/TestWrite.cxx
Example/Write.cxx
Example/WriteRead.cxx
Testing/TestAllEntryVerify.cxx
Testing/TestAllReadCompareDicom.cxx
Testing/TestReadWriteReadCompare.cxx
src/gdcmDicomDir.cxx
src/gdcmDocument.cxx
src/gdcmDocument.h
src/gdcmFile.cxx
src/gdcmFile.h
src/gdcmHeader.cxx
src/gdcmHeader.h
vtk/vtkGdcmReader.cxx

index 3f67163a3cce81db6f3d34223fa10aaca8e2a991..3a66c1681c2a0b5f8ace74ec8d55946a86080c59 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -4,6 +4,9 @@
     enable_sequences removed from gdcmHeader constructor and dependencies.
   * Example/PrintHeader.cxx: fixed accordingly (enable_sequences removal).
   * gdcmPython/demo/PrintHeader.py: dummy fix.
+  * src/gdcmDocument.[h|cxx], gdcmFile.[h|cxx], gdcmHeader.[h|cxx]:
+    skip_shadow removed from gdcmHeader constructor and dependencies.
+  * Example/*.cxx and Test/*.cxx changed accordingly.
 
 2004-07-27 Eric Boix <Eric.Boix@creatis.insa-lyon.fr>
   * Test/CMakeLists.txt: newly re-introduced SIEMENS_Sonata-12-MONO2-SQ.dcm
index d3ceeb25dafc9157d92c24d848f5bcec7dc7914a..85e4b2971b5f9a63478fcaa61fbe548cdc672bd1 100644 (file)
@@ -8,8 +8,10 @@ int main(int argc, char* argv[])
    std::string fileName;   
 
    if (argc != 2) {
-      std::cout << " Usage : " << argv[0] 
-                << " filename." << std::endl;
+      std::cout << " Usage : "
+                << argv[0] 
+                << " filename."
+                << std::endl;
    }
 
    if (argc > 1) {
@@ -19,8 +21,7 @@ int main(int argc, char* argv[])
       fileName += "/test.acr";
    }
    
-   e1= new gdcmHeader 
-      (fileName.c_str(),false, true);
+   e1= new gdcmHeader( fileName.c_str(), false );
 
    e1->SetPrintLevel(2);
    
index edc2957f83b8cdfba0665b2c501da0a9c0b51bcf..0beeda3f4934603a86dad8eefa550fc6eba8c93a 100644 (file)
@@ -18,8 +18,7 @@ int main(int argc, char* argv[])
       fileName += "/test.acr";
    }
    
-   e1= new gdcmHeader 
-          (fileName.c_str(),false, true);
+   e1= new gdcmHeader( fileName.c_str(), false );
 
    f1 = new gdcmFile(e1);
 
index c8ec5d6ca433e5eefe57971bc2a9eeb7159fc5f5..be38cc4f1c82299405a0b4199645dcd50621e757 100644 (file)
@@ -3,28 +3,24 @@
 
 int main(int argc, char* argv[])
 { 
-   bool dropPriv = false;
    std::string fileName;   
 
-   if (argc == 1) {
-      std::cout << argv[0] <<
-      " fileName" << std::endl <<
-      "    [nopriv]  if you don't want to print Shadow groups"  << std::endl;
+   if (argc == 1)
+   {
+      std::cout << argv[0] << " fileName" << std::endl;
    }
 
-   if (argc > 1) {
+   if (argc > 1)
+   {
       fileName=argv[1];
-   } else {
+   }
+   else
+   {
       fileName += GDCM_DATA_ROOT;
       fileName += "/test.acr";
    }
    
-   for (int j=0;j<argc;j++) {
-      if (strcmp(argv[j],"nopriv")==0)
-         dropPriv=true;
-   }
-   
-   gdcmFile *e2 = new gdcmFile(fileName.c_str(),false, dropPriv);
+   gdcmFile *e2 = new gdcmFile( fileName.c_str(), false );
    gdcmHeader *e1 = e2->GetHeader();  
         
    if (argc > 2) {
index cb9b9e697bdde0093188e1bf6c54218644dcbdea..402f7c684298e0e5e71e2be4753407dd3ea8f1cd 100644 (file)
@@ -46,8 +46,9 @@ int main(int argc, char* argv[])
      
    toto = argv[1]; 
 
-   e1 = new gdcmHeader(toto.c_str(), false, true);
-   if (!e1->IsReadable()) {
+   e1 = new gdcmHeader( toto.c_str(), false );
+   if (!e1->IsReadable())
+   {
        std::cerr << "Sorry, not a Readable DICOM / ACR File"  <<std::endl;
        return 0;
    }
index dbc5303882ba7ee4ba9ad6d1ce29c10d3d8a6424..5f716bdbea58326166e8d764e8080e024602bc50 100644 (file)
@@ -45,7 +45,7 @@ int main(int argc, char* argv[])
      
    toto = argv[1]; 
 
-   e1 = new gdcmHeader(toto.c_str(), false, true);
+   e1 = new gdcmHeader( toto.c_str(), false );
    if (!e1->IsReadable()) {
        std::cerr << "Sorry, not a Readable DICOM / ACR File"  <<std::endl;
        return 0;
index 4c8c0c14bcaad69db897ee7b80e56ac044dd08bd..48be008419142b52240abb1a56fa942a34670661 100644 (file)
@@ -25,7 +25,7 @@ int main(int argc, char* argv[])
 
    std::cout << argv[1] << std::endl;
 
-   e1 = new gdcmHeader(toto, false, true);
+   e1 = new gdcmHeader( toto, false );
    if (!e1->IsReadable()) {
        std::cerr << "Sorry, " << toto <<"  not a Readable DICOM / ACR File"
                  <<std::endl;
@@ -44,7 +44,7 @@ int main(int argc, char* argv[])
 
 // --------------------- we read the written image
       
-   e2 = new gdcmHeader(zozo, false, true);
+   e2 = new gdcmHeader( zozo, false );
    if (!e2->IsReadable()) {
        std::cerr << "Sorry, " << zozo << " not a Readable DICOM / ACR File"  
                  <<std::endl;
index 222d2f354fc149f184b085e9357383fcefcd4c44..6c86f298250bb089e32298ce4bc24be7dff25166 100644 (file)
@@ -163,7 +163,7 @@ bool ReferenceFileParser::Check()
    {
       string fileName = DataPath + i->first;
       cout << Indent << "FileName: " << fileName << endl;
-      gdcmHeader* tested = new gdcmHeader( fileName.c_str(), false, true );
+      gdcmHeader* tested = new gdcmHeader( fileName.c_str(), false );
       if( !tested->IsReadable() )
       {
         cerr << Indent << "Image not gdcm compatible:"
index 0c0595c80123b4d4d13039b8459e09182280d3ef..386a6ffb923d001891f0628cb5267924c75adf0a 100644 (file)
@@ -73,7 +73,7 @@ int TestAllReadCompareDicom(int argc, char* argv[])
    
       std::cout << "   Testing: " << filename << std::endl;
 
-      gdcmFile* tested = new gdcmFile( filename, false, true );
+      gdcmFile* tested = new gdcmFile( filename, false );
       if( !tested->GetHeader()->IsReadable() )
       {
         std::cout << "      Image not gdcm compatible:"
@@ -116,8 +116,7 @@ int TestAllReadCompareDicom(int argc, char* argv[])
 
       ////// When reference file is not gdcm readable test is failed:
   
-      gdcmFile* reference = new gdcmFile( referenceFileName.c_str(),
-                                          false, true );
+      gdcmFile* reference = new gdcmFile( referenceFileName.c_str(), false );
       if( !reference->GetHeader()->IsReadable() )
       {
          std::cout << "      Reference image " << std::endl
index 3df0cbdfd6f3142a9d67cf8df22d2c6a7716ce40..ed14c08a0d9c53575e80e8033ab2b1b4744bb348 100644 (file)
@@ -39,7 +39,7 @@ int TestReadWriteReadCompare(int argc, char* argv[])
 
       //////////////// Step 1 (see above description):
 
-      gdcmHeader *header = new gdcmHeader( filename, false, true );
+      gdcmHeader *header = new gdcmHeader( filename, false );
       if( !header->IsReadable() )
       {
          std::cerr << "Test::TestReadWriteReadCompare: Image not gdcm compatible:"
@@ -74,8 +74,7 @@ int TestReadWriteReadCompare(int argc, char* argv[])
     
       //////////////// Step 3:
 
-      gdcmFile* reread = new gdcmFile( "TestReadWriteReadCompare.dcm",
-                                      false, true );
+      gdcmFile* reread = new gdcmFile( "TestReadWriteReadCompare.dcm", false );
       if( !reread->GetHeader()->IsReadable() )
       {
         std::cerr << "Test::TestReadWriteReadCompare: Could not reread image "
index b80d6bac08e6096ff0ddc006819d1308656b0bfa..3dcefe86d7306119f0f2c412a0d4a08686619526 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmDicomDir.cxx,v $
   Language:  C++
-  Date:      $Date: 2004/07/30 11:40:13 $
-  Version:   $Revision: 1.60 $
+  Date:      $Date: 2004/07/30 16:09:27 $
+  Version:   $Revision: 1.61 $
   
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -75,7 +75,7 @@ gdcmDicomDir::gdcmDicomDir(bool exception_on_error):
  */
 gdcmDicomDir::gdcmDicomDir(std::string const & fileName, bool parseDir,
                            bool exception_on_error):
-   gdcmDocument( fileName, exception_on_error, true) // true : enable SeQuences
+   gdcmDocument( fileName, exception_on_error)
 {
    // que l'on ai passe un root directory ou un DICOMDIR
    // et quelle que soit la valeur de parseDir,
@@ -451,21 +451,29 @@ void gdcmDicomDir::CreateDicomDirChainedList(std::string const & path)
          break;
       }
 
-      header = new gdcmHeader(it->c_str(),false,true);
-      if(!header) {
-         std::cout << "failure in new Header " << it->c_str() << std::endl; // JPR
-      }
-      if(header->IsReadable()) {
-         list.push_back(header);  // adds the file header to the chained list
-         std::cout << "readable : " <<it->c_str() << std::endl; // JPR
-       }
-      else
+      header = new gdcmHeader( it->c_str(),false );
+      if( !header )
       {
-         delete header;
+         dbg.Verbose( 1,
+                      "gdcmDicomDir::CreateDicomDirChainedList: "
+                      "failure in new Header ",
+                      it->c_str() );
       }
-      count++;
-
+      if( header->IsReadable() )
+      {
+         // Add the file header to the chained list:
+         list.push_back(header);
+         dbg.Verbose( 1,
+                      "gdcmDicomDir::CreateDicomDirChainedList: readable ",
+                      it->c_str() );
+       }
+       else
+       {
+          delete header;
+       }
+       count++;
    }
+
    // sorts Patient/Study/Serie/
    std::sort(list.begin(), list.end(), gdcmDicomDir::HeaderLessThan );
 
index 4a204595b79f98e71e134101f568d3a22a66ac89..ccd0b644be13c468d5c272c8c0223d3eff4a0451 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmDocument.cxx,v $
   Language:  C++
-  Date:      $Date: 2004/07/30 11:40:13 $
-  Version:   $Revision: 1.59 $
+  Date:      $Date: 2004/07/30 16:09:27 $
+  Version:   $Revision: 1.60 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -84,15 +84,11 @@ const unsigned int gdcmDocument::MAX_SIZE_PRINT_ELEMENT_VALUE = 0x7fffffff;
  * \brief   constructor  
  * @param   inFilename file to be opened for parsing
  * @param   exception_on_error whether we throw an exception or not
- * @param   ignore_shadow to allow skipping the shadow elements, 
- *          to save memory space.
  */
 gdcmDocument::gdcmDocument( std::string const & filename, 
-                            bool exception_on_error,
-                            bool ignore_shadow) 
+                            bool exception_on_error) 
               : gdcmElementSet(-1)
 {
-   IgnoreShadow = ignore_shadow;
    SetMaxSizeLoadEntry(MAX_SIZE_LOAD_ELEMENT_VALUE); 
    Filename = filename;
    Initialise();
index c999b59134f928d3fa671124660cfd6631987f18..12563947d9007b08f2a55556dd7f8b022a53b002 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmDocument.h,v $
   Language:  C++
-  Date:      $Date: 2004/07/30 11:40:13 $
-  Version:   $Revision: 1.24 $
+  Date:      $Date: 2004/07/30 16:09:27 $
+  Version:   $Revision: 1.25 $
  
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -158,8 +158,7 @@ protected:
    // gdcmDicomDir are meaningfull).
    gdcmDocument(bool exception_on_error  = false);
    gdcmDocument(std::string const & filename, 
-                bool  exception_on_error = false, 
-                bool  ignore_shadow      = false);
+                bool  exception_on_error = false);
    virtual ~gdcmDocument();
    
    void gdcmDocument::Parse7FE0 ();   
index c4672b6e9d3f9a021fa9817936a9e575aec2de40..248fc34daeaac29a9aa2f3d0579b166f88af0199 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmFile.cxx,v $
   Language:  C++
-  Date:      $Date: 2004/07/30 11:40:13 $
-  Version:   $Revision: 1.117 $
+  Date:      $Date: 2004/07/30 16:09:27 $
+  Version:   $Revision: 1.118 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -63,16 +63,11 @@ gdcmFile::gdcmFile(gdcmHeader *header)
  *        seen as a side effect).   
  * @param filename file to be opened for parsing
  * @param   exception_on_error whether we throw an exception or not
- * @param   ignore_shadow to allow skipping the shadow elements, 
- *          to save memory space.
  */
 gdcmFile::gdcmFile(std::string const & filename, 
-                   bool exception_on_error,
-                   bool ignore_shadow)
+                   bool exception_on_error )
 {
-   Header = new gdcmHeader( filename,
-                            exception_on_error,
-                            ignore_shadow );
+   Header = new gdcmHeader( filename, exception_on_error );
    SelfHeader = true;
    PixelRead  = -1; // no ImageData read yet.
 
index dc1836226367ba2890ef6aaf9ab06cee594936f7..b7a395827c8a4488c26e522afad87fc4de0f6118 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmFile.h,v $
   Language:  C++
-  Date:      $Date: 2004/07/30 11:40:13 $
-  Version:   $Revision: 1.39 $
+  Date:      $Date: 2004/07/30 16:09:27 $
+  Version:   $Revision: 1.40 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -33,8 +33,7 @@ class GDCM_EXPORT gdcmFile
 public:
    gdcmFile(gdcmHeader *header);
    gdcmFile(std::string const & filename, 
-            bool  exception_on_error = false, 
-            bool  skip_shadow        = false);
+            bool  exception_on_error = false);
  
    virtual ~gdcmFile();
 
index 5e01a8f37ecdc30f4a940207709bfb3ac9fd108c..5023fad1f5d3598b7061e5f69c3acd7747a645c2 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmHeader.cxx,v $
   Language:  C++
-  Date:      $Date: 2004/07/30 11:40:13 $
-  Version:   $Revision: 1.179 $
+  Date:      $Date: 2004/07/30 16:09:27 $
+  Version:   $Revision: 1.180 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
  * \brief  Constructor 
  * @param  filename name of the file whose header we want to analyze
  * @param  exception_on_error whether we want to throw an exception or not
- * @param  ignore_shadow = true if user wants to skip shadow groups 
- *         during parsing, to save memory space
  */
 gdcmHeader::gdcmHeader( std::string const & filename, 
-                        bool exception_on_error,
-                        bool ignore_shadow ):
+                        bool exception_on_error ):
             gdcmDocument( filename,
-                          exception_on_error,
-                          ignore_shadow )
+                          exception_on_error )
 {    
    // for some ACR-NEMA images GrPixel, NumPixel is *not* 7fe0,0010
    // We may encounter the 'RETired' (0x0028, 0x0200) tag
@@ -49,8 +45,7 @@ gdcmHeader::gdcmHeader( std::string const & filename,
    // Inside the group pointed by "Image Location" the searched element
    // is conventionally the element 0x0010 (when the norm is respected).
    // When the "Image Location" is absent we default to group 0x7fe0.
-   
-   // This IS the right place for the code
+   // Note: this IS the right place for the code
  
    // Image Location
    std::string imgLocation = GetEntryByNumber(0x0028, 0x0200);
index c5afb10b8b484a08c1ea425d5b416b74a61dcfe3..4835fb818480a0f2a5c0bcf54d27d20225c42c98 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmHeader.h,v $
   Language:  C++
-  Date:      $Date: 2004/07/30 11:40:13 $
-  Version:   $Revision: 1.82 $
+  Date:      $Date: 2004/07/30 16:09:27 $
+  Version:   $Revision: 1.83 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -109,8 +109,7 @@ protected:
 public:
    gdcmHeader(bool exception_on_error = false);
    gdcmHeader(std::string const & filename, 
-              bool  exception_on_error = false, 
-              bool  skip_shadow        = false);
+              bool  exception_on_error = false);
  
    virtual ~gdcmHeader();
 
index 11d9f68076f835e9ec4a2a770822ae1ffe973d20..7953ac8e7255315e754e3731c5bee389db2d889d 100644 (file)
@@ -58,7 +58,7 @@
 #include <vtkPointData.h>
 #include <vtkLookupTable.h>
 
-vtkCxxRevisionMacro(vtkGdcmReader, "$Revision: 1.46 $");
+vtkCxxRevisionMacro(vtkGdcmReader, "$Revision: 1.47 $");
 vtkStandardNewMacro(vtkGdcmReader);
 
 //-----------------------------------------------------------------------------
@@ -444,8 +444,7 @@ int vtkGdcmReader::CheckFileCoherence()
       fclose(fp);
 
       // Stage 1.2: check for Gdcm parsability
-      gdcmHeader GdcmHeader(FileName->c_str(), false, true);
-      //                             true : for enableSequences
+      gdcmHeader GdcmHeader(FileName->c_str(), false );
       if (!GdcmHeader.IsReadable())
       {
          vtkErrorMacro("Gdcm cannot parse file " << FileName->c_str());
@@ -611,8 +610,7 @@ size_t vtkGdcmReader::LoadImageInMemory(
              unsigned long & UpdateProgressCount)
 {
    vtkDebugMacro("Copying to memory image [" << FileName.c_str() << "]");
-   gdcmFile GdcmFile(FileName.c_str(),false,true);
-   // true : to enable SeQuences
+   gdcmFile GdcmFile( FileName.c_str(), false );
    size_t size;
 
    // If the data structure of vtk for image/volume representation