X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=blobdiff_plain;f=Testing%2FTestReadWriteReadCompare.cxx;h=ac868dcdc254e529f576cf3c843fa9578f3f99fc;hb=05a8ef5406eb1d78b0a43a714b41656a297a7f3c;hp=939ab869b76107cad4116bf0cce3c781283f0942;hpb=8b695d4a325b768e1bec82f41d74c49ebb906cc8;p=gdcm.git

diff --git a/Testing/TestReadWriteReadCompare.cxx b/Testing/TestReadWriteReadCompare.cxx
index 939ab869..ac868dcd 100644
--- a/Testing/TestReadWriteReadCompare.cxx
+++ b/Testing/TestReadWriteReadCompare.cxx
@@ -6,9 +6,10 @@
 
 int TestReadWriteReadCompare(int argc, char* argv[]) 
 {
-   if (argc) {
-    std::cerr << "Test::TestReadWriteReadCompare: Usage: " << argv[0]
-              << " (no arguments needed)." << std::endl;
+   if (argc)
+   {
+      std::cerr << "Test::TestReadWriteReadCompare: Usage: " << argv[0]
+                << " (no arguments needed)." << std::endl;
    }
    
    std::cout<< "Test::TestReadWriteReadCompare: description " << std::endl;
@@ -27,103 +28,110 @@ int TestReadWriteReadCompare(int argc, char* argv[])
              << "images " << std::endl
              << "           match (as expanded by gdcm)." << std::endl;
 
-  int i = 0;
-  while( gdcmDataImages[i] != 0 )
-    {
-    std::string filename = GDCM_DATA_ROOT;
-    filename += "/";  //doh!
-    filename += gdcmDataImages[i++];
+   int i = 0;
+   while( gdcmDataImages[i] != 0 )
+   {
+      std::string filename = GDCM_DATA_ROOT;
+      filename += "/";
+      filename += gdcmDataImages[i++];
    
-    std::cout << "   Testing: " << filename << std::endl;
+      std::cout << "   Testing: " << filename << std::endl;
 
-    //////////////// Step 1 (see above description):
+      //////////////// Step 1 (see above description):
 
-    gdcmHeader *header = new gdcmHeader( filename.c_str(), false, true );
-    if( !header->IsReadable() )
+      gdcmHeader *header = new gdcmHeader( filename );
+      if( !header->IsReadable() )
       {
-      std::cerr << "Test::TestReadWriteReadCompare: Image not gdcm compatible:"
-                << filename << std::endl;
-      delete header;
-      return 0;
+         std::cerr << "Test::TestReadWriteReadCompare: Image not gdcm compatible:"
+                   << filename << std::endl;
+         delete header;
+         return 1;
       }
-    std::cout << "           step 1 ...";
+      std::cout << "           step 1 ...";
 
-    //////////////// Step 2:
+      //////////////// Step 2:
 
-    gdcmFile*  file = new gdcmFile( header );
-    int dataSize    = file->GetImageDataSize();
-    void* imageData = file->GetImageData(); //EXTREMELY IMPORTANT
+      gdcmFile*  file = new gdcmFile( header );
+      int dataSize    = file->GetImageDataSize();
+      void* imageData = file->GetImageData(); //EXTREMELY IMPORTANT
+             // Sure, it is : It's up to the user to decide if he wants to
+             // GetImageData or if he wants to GetImageDataRaw
+             // (even if we do it by setting a flag, he will have to decide) 
 
-    /// \todo Following line commented out because gdcmFile::SetImageData() is
-    /// brain dead: it sets ImageDataSize to its argument and PixelRead to a.
-    /// Later on, when writing gdcmFile::WriteBase() and because PixelRead == 1
-    /// we call
-    ///    PixelElement->SetLength( ImageDataSizeRaw );
-    /// where we use ImageDataSizeRAW instead of ImageDataSize !
-    /// But when the original image made the transformation LUT -> RGB, 
-    /// ImageDataSizeRaw is the third of ImageDataSize, and there is no
-    /// reason (since we called gdcmFile::SetImageData) to use the Raw image
-    /// size... This "bug" in gdcmFile made that we had to black list
-    /// images 8BitsUncompressedColor.dcm, OT-PAL-8-face.dcm and 
-    /// US-PAL-8-10x-echo.dcm...
-    /// In conclusion fix gdcmFile, and then uncomment the following line.
-    /// file->SetImageData(imageData, dataSize);
-    file->WriteDcmExplVR( "TestReadWriteReadCompare.dcm" );
-    std::cout << "2...";
+      /// \todo Following line commented out because gdcmFile::SetImageData() is
+      /// brain dead: it sets ImageDataSize to its argument and PixelRead to a.
+      /// Later on, when writing gdcmFile::WriteBase() 
+      /// and because PixelRead == 1 we call
+      ///    PixelElement->SetLength( ImageDataSizeRaw );
+      /// where we use ImageDataSizeRAW instead of ImageDataSize !
+      /// But when the original image made the transformation LUT -> RGB, 
+      /// ImageDataSizeRaw is the third of ImageDataSize, and there is no
+      /// reason (since we called gdcmFile::SetImageData) to use the Raw image
+      /// size... This "bug" in gdcmFile made that we had to black list
+      /// images 8BitsUncompressedColor.dcm, OT-PAL-8-face.dcm and 
+      /// US-PAL-8-10x-echo.dcm...
+      /// In conclusion fix gdcmFile, and then uncomment the following line.
+      
+      // --> I did. ctest doesn't break. But ... is it enought to say it's OK ?
+      
+      file->SetImageData(imageData, dataSize);
+      
+      file->WriteDcmExplVR( "TestReadWriteReadCompare.dcm" );
+      std::cout << "2...";
     
-    //////////////// Step 3:
+      //////////////// Step 3:
 
-    gdcmFile* reread = new gdcmFile( "TestReadWriteReadCompare.dcm",
-                                      false, true );
-    if( !reread->GetHeader()->IsReadable() )
-    {
-      std::cerr << "Test::TestReadWriteReadCompare: Could not reread image "
-                << "written:" << filename << std::endl;
-      delete header;
-      delete file;
-      delete reread;
-      return 1;
-    }
-    std::cout << "3...";
-    // For the next step:
-    int    dataSizeWritten = reread->GetImageDataSize();
-    void* imageDataWritten = reread->GetImageData();
+      gdcmFile* reread = new gdcmFile( "TestReadWriteReadCompare.dcm" );
+      if( !reread->GetHeader()->IsReadable() )
+      {
+        std::cerr << "Test::TestReadWriteReadCompare: Could not reread image "
+                  << "written:" << filename << std::endl;
+        delete header;
+        delete file;
+        delete reread;
+        return 1;
+      }
+      std::cout << "3...";
+      // For the next step:
+      int    dataSizeWritten = reread->GetImageDataSize();
+      void* imageDataWritten = reread->GetImageData();
 
-    //////////////// Step 4:
+      //////////////// Step 4:
  
-    if (dataSize != dataSizeWritten)
-    {
-       std::cout << std::endl
-          << "        Pixel areas lengths differ: "
-          << dataSize << " # " << dataSizeWritten << std::endl;
-       delete (char*)imageData;
-       delete (char*)imageDataWritten;
-       delete header;
-       delete file;
-       delete reread;
-       return 1;
-    }
+      if (dataSize != dataSizeWritten)
+      {
+         std::cout << std::endl
+            << "        Pixel areas lengths differ: "
+            << dataSize << " # " << dataSizeWritten << std::endl;
+         delete (char*)imageData;
+         delete (char*)imageDataWritten;
+         delete header;
+         delete file;
+         delete reread;
+         return 1;
+      }
 
-    if (int res=memcmp(imageData, imageDataWritten, dataSize) !=0)
-    {
-       std::cout << std::endl
-          << "        Pixel differ (as expanded in memory)." << std::endl;
-       delete (char*)imageData;
-       delete (char*)imageDataWritten;
-       delete header;
-       delete file;
-       delete reread;
-       return 1;
-    }
-    std::cout << "4...OK." << std::endl ;
+      if (int res = memcmp(imageData, imageDataWritten, dataSize) !=0)
+      {
+         (void)res;
+         std::cout << std::endl
+            << "        Pixel differ (as expanded in memory)." << std::endl;
+         delete (char*)imageData;
+         delete (char*)imageDataWritten;
+         delete header;
+         delete file;
+         delete reread;
+         return 1;
+      }
+      std::cout << "4...OK." << std::endl ;
 
-    //////////////// Clean up:
-    delete (char*)imageData;
-    delete (char*)imageDataWritten;
-    delete header;
-    delete file;
-    delete reread;
-  }
+      //////////////// Clean up:
+      delete (char*)imageData;
+      delete (char*)imageDataWritten;
+      delete header;
+      delete file;
+      delete reread;
+   }
 
   return 0;
 }