]> Creatis software - gdcm.git/blobdiff - src/gdcmFile.cxx
* Amelioration of code, remove some code copy. For the loading of
[gdcm.git] / src / gdcmFile.cxx
index b565a3a498604de9b834ae91a247ae9f15d7ed4f..666a8e96f5fdd0245ddfb14ec1d15cb82937732e 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmFile.cxx,v $
   Language:  C++
-  Date:      $Date: 2004/10/20 14:30:40 $
-  Version:   $Revision: 1.146 $
+  Date:      $Date: 2004/11/24 11:17:47 $
+  Version:   $Revision: 1.161 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -18,6 +18,9 @@
 
 #include "gdcmFile.h"
 #include "gdcmDebug.h"
+#include "gdcmUtil.h"
+#include "gdcmBinEntry.h"
+#include <fstream>
 
 namespace gdcm 
 {
@@ -72,6 +75,12 @@ File::File(std::string const & filename )
  */
 void File::Initialise()
 {
+   WriteMode = WMODE_RGB;
+   WriteType = WTYPE_IMPL_VR;
+
+   PixelConverter = new PixelConvert;
+   Archive = new DocEntryArchive( HeaderInternal );
+
    if ( HeaderInternal->IsReadable() )
    {
       ImageDataSizeRaw = ComputeDecompressedPixelDataSizeFromHeader();
@@ -84,9 +93,9 @@ void File::Initialise()
          ImageDataSize = ImageDataSizeRaw;
       }
 
-      PixelConverter = new PixelConvert;
       PixelConverter->GrabInformationsFromHeader( HeaderInternal );
    }
+
    SaveInitialValues();
 }
 
@@ -97,13 +106,20 @@ void File::Initialise()
  */
 File::~File()
 { 
+   if( PixelConverter )
+   {
+      delete PixelConverter;
+   }
+   if( Archive )
+   {
+      delete Archive;
+   }
+
    if( SelfHeader )
    {
       delete HeaderInternal;
    }
    HeaderInternal = 0;
-
-   DeleteInitialValues();
 }
 
 /**
@@ -112,120 +128,8 @@ File::~File()
  */
 void File::SaveInitialValues()
 { 
-
    PixelRead  = -1; // no ImageData read yet.
-   LastAllocatedPixelDataLength = 0;
    Pixel_Data = 0;
-
-   InitialSpp = "";     
-   InitialPhotInt = "";
-   InitialPlanConfig = "";
-   InitialBitsAllocated = "";
-   InitialHighBit = "";
-  
-   InitialRedLUTDescr   = 0;
-   InitialGreenLUTDescr = 0;
-   InitialBlueLUTDescr  = 0;
-   InitialRedLUTData    = 0;
-   InitialGreenLUTData  = 0;
-   InitialBlueLUTData   = 0; 
-                
-   if ( HeaderInternal->IsReadable() )
-   {
-      // the following values *may* be modified 
-      // by File::GetImageDataIntoVectorRaw
-      // we save their initial value.
-      InitialSpp           = HeaderInternal->GetEntryByNumber(0x0028,0x0002);
-      InitialPhotInt       = HeaderInternal->GetEntryByNumber(0x0028,0x0004);
-      InitialPlanConfig    = HeaderInternal->GetEntryByNumber(0x0028,0x0006);
-      
-      InitialBitsAllocated = HeaderInternal->GetEntryByNumber(0x0028,0x0100);
-      InitialHighBit       = HeaderInternal->GetEntryByNumber(0x0028,0x0102);
-
-      // the following entries *may* be removed from the H table
-      // (NOT deleted ...) by File::GetImageDataIntoVectorRaw  
-      // we keep a pointer on them.
-      InitialRedLUTDescr   = HeaderInternal->GetDocEntryByNumber(0x0028,0x1101);
-      InitialGreenLUTDescr = HeaderInternal->GetDocEntryByNumber(0x0028,0x1102);
-      InitialBlueLUTDescr  = HeaderInternal->GetDocEntryByNumber(0x0028,0x1103);
-
-      InitialRedLUTData    = HeaderInternal->GetDocEntryByNumber(0x0028,0x1201);
-      InitialGreenLUTData  = HeaderInternal->GetDocEntryByNumber(0x0028,0x1202);
-      InitialBlueLUTData   = HeaderInternal->GetDocEntryByNumber(0x0028,0x1203); 
-   }
-}
-
-/**
- * \brief restores some initial values
- * \warning not end user intended
- */
-void File::RestoreInitialValues()
-{   
-   if ( HeaderInternal->IsReadable() )
-   {      
-      // the following values *may* have been modified 
-      // by File::GetImageDataIntoVectorRaw
-      // we restore their initial value.
-      if ( InitialSpp != "")
-         HeaderInternal->SetEntryByNumber(InitialSpp,0x0028,0x0002);
-      if ( InitialPhotInt != "")
-         HeaderInternal->SetEntryByNumber(InitialPhotInt,0x0028,0x0004);
-      if ( InitialPlanConfig != "")
-
-         HeaderInternal->SetEntryByNumber(InitialPlanConfig,0x0028,0x0006);
-      if ( InitialBitsAllocated != "")
-          HeaderInternal->SetEntryByNumber(InitialBitsAllocated,0x0028,0x0100);
-      if ( InitialHighBit != "")
-          HeaderInternal->SetEntryByNumber(InitialHighBit,0x0028,0x0102);
-               
-      // the following entries *may* be have been removed from the H table
-      // (NOT deleted ...) by File::GetImageDataIntoVectorRaw  
-      // we restore them.
-
-      if (InitialRedLUTDescr)
-         HeaderInternal->AddEntry(InitialRedLUTDescr);
-      if (InitialGreenLUTDescr)
-         HeaderInternal->AddEntry(InitialGreenLUTDescr);
-      if (InitialBlueLUTDescr)
-         HeaderInternal->AddEntry(InitialBlueLUTDescr);
-
-      if (InitialRedLUTData)
-         HeaderInternal->AddEntry(InitialBlueLUTDescr);
-      if (InitialGreenLUTData)
-         HeaderInternal->AddEntry(InitialGreenLUTData);
-      if (InitialBlueLUTData)
-         HeaderInternal->AddEntry(InitialBlueLUTData);
-   }
-}
-
-/**
- * \brief delete initial values (il they were saved)
- *        of InitialLutDescriptors and InitialLutData
- */
-void File::DeleteInitialValues()
-{ 
-
-// InitialLutDescriptors and InitialLutData
-// will have to be deleted if the don't belong any longer
-// to the Header H table when the header is deleted...
-
-   if ( InitialRedLUTDescr )           
-      delete InitialRedLUTDescr;
-  
-   if ( InitialGreenLUTDescr )
-      delete InitialGreenLUTDescr;
-      
-   if ( InitialBlueLUTDescr )      
-      delete InitialBlueLUTDescr; 
-       
-   if ( InitialRedLUTData )      
-      delete InitialRedLUTData;
-   
-   if ( InitialGreenLUTData != NULL)
-      delete InitialGreenLUTData;
-      
-   if ( InitialBlueLUTData != NULL)      
-      delete InitialBlueLUTData;      
 }
 
 //-----------------------------------------------------------------------------
@@ -273,7 +177,7 @@ int File::ComputeDecompressedPixelDataSizeFromHeader()
    //  - it is not defined (i.e. it's value is 0)
    //  - it's 12, since we will expand the image to 16 bits (see
    //    PixelConvert::ConvertDecompress12BitsTo16Bits() )
-   if ( ( numberBitsAllocated == 0 ) || ( numberBitsAllocated == 12 ) )
+   if ( numberBitsAllocated == 0 || numberBitsAllocated == 12 )
    {
       numberBitsAllocated = 16;
    } 
@@ -299,33 +203,97 @@ int File::ComputeDecompressedPixelDataSizeFromHeader()
  */
 uint8_t* File::GetImageData()
 {
-   // FIXME (Mathieu)
-   // I need to deallocate Pixel_Data before doing any allocation:
+   if ( ! GetDecompressed() )
+   {
+      // If the decompression failed nothing can be done.
+      return 0;
+   }
+                                                                                
+   if ( HeaderInternal->HasLUT() && PixelConverter->BuildRGBImage() )
+   {
+      return PixelConverter->GetRGB();
+   }
+   else
+   {
+      // When no LUT or LUT conversion fails, return the decompressed
+      return PixelConverter->GetDecompressed();
+   }
    
-   if ( Pixel_Data )
-     if ( LastAllocatedPixelDataLength != ImageDataSize ) 
-        free(Pixel_Data);
-   if ( !Pixel_Data )
-      Pixel_Data = new uint8_t[ImageDataSize];
-    
-   if ( Pixel_Data )
-   {
-      LastAllocatedPixelDataLength = ImageDataSize;
-
-      // we load the pixels (and transform grey level + LUT into RGB)
-      GetImageDataIntoVector(Pixel_Data, ImageDataSize);
-
-      // We say the value *is* loaded.
-      GetHeader()->SetEntryByNumber( GDCM_BINLOADED,
-         GetHeader()->GetGrPixel(), GetHeader()->GetNumPixel());
-
-      // Will be 7fe0, 0010 in standard case
-      GetHeader()->SetEntryBinAreaByNumber( Pixel_Data, 
-         GetHeader()->GetGrPixel(), GetHeader()->GetNumPixel()); 
-   }      
-   PixelRead = 0; // no PixelRaw
-
-   return Pixel_Data;
+/*   if ( ! GetDecompressed() )
+   {
+      // If the decompression failed nothing can be done.
+      return 0;
+   }
+                                                                                
+   uint8_t* pixelData;
+   if ( HeaderInternal->HasLUT() && PixelConverter->BuildRGBImage() )
+   {
+      pixelData = PixelConverter->GetRGB();
+   }
+   else
+   {
+      // When no LUT or LUT conversion fails, return the decompressed
+      pixelData = PixelConverter->GetDecompressed();
+   }
+
+// PIXELCONVERT CLEANME
+   // Restore the header in a disk-consistent state
+   // (if user asks twice to get the pixels from disk)
+   if ( PixelRead != -1 ) // File was "read" before
+   {
+      RestoreInitialValues();
+   }
+   if ( PixelConverter->GetRGB() )
+   {
+      // now, it's an RGB image
+      // Lets's write it in the Header
+      std::string spp = "3";        // Samples Per Pixel
+      HeaderInternal->SetEntryByNumber(spp,0x0028,0x0002);
+      std::string rgb = "RGB ";     // Photometric Interpretation
+      HeaderInternal->SetEntryByNumber(rgb,0x0028,0x0004);
+      std::string planConfig = "0"; // Planar Configuration
+      HeaderInternal->SetEntryByNumber(planConfig,0x0028,0x0006);
+      PixelRead = 0; // no PixelRaw
+   }
+   else
+   {
+      if ( HeaderInternal->HasLUT() )
+      {
+         // The LUT interpretation failed
+         std::string photometricInterpretation = Util::DicomString("MONOCHROME1");
+         HeaderInternal->SetEntryByNumber( photometricInterpretation,
+                                           0x0028, 0x0004 );
+         PixelRead = 0; // no PixelRaw
+      }
+      else
+      {
+         if ( PixelConverter->IsDecompressedRGB() )
+         {
+            ///////////////////////////////////////////////////
+            // now, it's an RGB image
+            // Lets's write it in the Header
+            // Droping Palette Color out of the Header
+            // has been moved to the Write process.
+            // TODO : move 'values' modification to the write process
+            //      : save also (in order to be able to restore)
+            //      : 'high bit' -when not equal to 'bits stored' + 1
+            //      : 'bits allocated', when it's equal to 12 ?!
+            std::string spp = "3";            // Samples Per Pixel
+            std::string photInt = "RGB ";     // Photometric Interpretation
+            std::string planConfig = "0";     // Planar Configuration
+            HeaderInternal->SetEntryByNumber(spp,0x0028,0x0002);
+            HeaderInternal->SetEntryByNumber(photInt,0x0028,0x0004);
+            HeaderInternal->SetEntryByNumber(planConfig,0x0028,0x0006);
+         }
+         PixelRead = 1; // PixelRaw
+      } 
+   }
+
+   // We say the value *is* loaded.
+   SetPixelData(pixelData);
+// END PIXELCONVERT CLEANME
+
+   return pixelData;*/
 }
 
 /**
@@ -355,50 +323,37 @@ uint8_t* File::GetImageData()
  */
 size_t File::GetImageDataIntoVector (void* destination, size_t maxSize)
 {
-   GetImageDataIntoVectorRaw (destination, maxSize);
-   PixelRead = 0 ; // =0 : no ImageDataRaw 
-   if ( !HeaderInternal->HasLUT() )
+   if ( ! GetDecompressed() )
    {
-      return ImageDataSize;
+      // If the decompression failed nothing can be done.
+      return 0;
    }
-                            
-   FILE* fp = HeaderInternal->OpenFile();
-   if ( PixelConverter->BuildRGBImage( fp ) )
-   {
-      memmove( destination,
-               (void*)PixelConverter->GetRGB(),
-               PixelConverter->GetRGBSize() );
-    
-      // now, it's an RGB image
-      // Lets's write it in the Header
-
-      // FIXME : Better use CreateOrReplaceIfExist ?
-
-      std::string spp = "3";        // Samples Per Pixel
-      HeaderInternal->SetEntryByNumber(spp,0x0028,0x0002);
-      std::string rgb = "RGB ";     // Photometric Interpretation
-      HeaderInternal->SetEntryByNumber(rgb,0x0028,0x0004);
-      std::string planConfig = "0"; // Planar Configuration
-      HeaderInternal->SetEntryByNumber(planConfig,0x0028,0x0006);
 
+   if ( HeaderInternal->HasLUT() && PixelConverter->BuildRGBImage() )
+   {
+      if ( PixelConverter->GetRGBSize() > maxSize )
+      {
+         dbg.Verbose(0, "File::GetImageDataIntoVector: pixel data bigger"
+                        "than caller's expected MaxSize");
+         return 0;
+      }
+      memcpy( destination,
+              (void*)PixelConverter->GetRGB(),
+              PixelConverter->GetRGBSize() );
+      return PixelConverter->GetRGBSize();
    }
-   else
-   { 
-      // PixelConverter->BuildRGBImage() failed probably because
-      // PixelConverter->GetLUTRGBA() failed:
-      // (gdcm-US-ALOKA-16.dcm), contains Segmented xxx Palette Color 
-      // that are *more* than 65535 long ?!? 
-      // No idea how to manage such an image !
-      // Need to make RGB Pixels (?) from grey Pixels (?!) and Gray Lut  (!?!)
-      // It seems that *no Dicom Viewer* has any idea :-(
-        
-      std::string photomInterp = "MONOCHROME1 ";  // Photometric Interpretation
-      HeaderInternal->SetEntryByNumber(photomInterp,0x0028,0x0004);
-   } 
-   HeaderInternal->CloseFile();
 
-   /// \todo Drop Palette Color out of the Header?
-   return ImageDataSize; 
+   // Either no LUT conversion necessary or LUT conversion failed
+   if ( PixelConverter->GetDecompressedSize() > maxSize )
+   {
+      dbg.Verbose(0, "File::GetImageDataIntoVector: pixel data bigger"
+                     "than caller's expected MaxSize");
+      return 0;
+   }
+   memcpy( destination,
+           (void*)PixelConverter->GetDecompressed(),
+           PixelConverter->GetDecompressedSize() );
+   return PixelConverter->GetDecompressedSize();
 }
 
 /**
@@ -412,118 +367,67 @@ size_t File::GetImageDataIntoVector (void* destination, size_t maxSize)
  */
 uint8_t* File::GetImageDataRaw ()
 {
-   size_t imgDataSize;
-   if ( HeaderInternal->HasLUT() )
-      /// \todo Let Header user a chance to get the right value
-      imgDataSize = ImageDataSizeRaw;
-   else 
-      imgDataSize = ImageDataSize;
-    
-   // FIXME (Mathieu)
-   // I need to deallocate Pixel_Data before doing any allocation:
-   
-   if ( Pixel_Data )
-      if ( LastAllocatedPixelDataLength != imgDataSize )
-         free(Pixel_Data);
-   if ( !Pixel_Data ) 
-      Pixel_Data = new uint8_t[imgDataSize];
+   return GetDecompressed();
+/*   uint8_t* decompressed = GetDecompressed();
+   if ( ! decompressed )
+   {
+      return 0;
+   }
 
-   if ( Pixel_Data )
+// PIXELCONVERT CLEANME
+   // Restore the header in a disk-consistent state
+   // (if user asks twice to get the pixels from disk)
+   if ( PixelRead != -1 ) // File was "read" before
    {
-      LastAllocatedPixelDataLength = imgDataSize;
-      
-      // we load the pixels ( grey level or RGB, but NO transformation)
-       GetImageDataIntoVectorRaw(Pixel_Data, imgDataSize);
+      RestoreInitialValues();
+   }
+   if ( PixelConverter->IsDecompressedRGB() )
+   {
+      ///////////////////////////////////////////////////
+      // now, it's an RGB image
+      // Lets's write it in the Header
+      // Droping Palette Color out of the Header
+      // has been moved to the Write process.
+      // TODO : move 'values' modification to the write process
+      //      : save also (in order to be able to restore)
+      //      : 'high bit' -when not equal to 'bits stored' + 1
+      //      : 'bits allocated', when it's equal to 12 ?!
+      std::string spp = "3";            // Samples Per Pixel
+      std::string photInt = "RGB ";     // Photometric Interpretation
+      std::string planConfig = "0";     // Planar Configuration
+      HeaderInternal->SetEntryByNumber(spp,0x0028,0x0002);
+      HeaderInternal->SetEntryByNumber(photInt,0x0028,0x0004);
+      HeaderInternal->SetEntryByNumber(planConfig,0x0028,0x0006);
+   }
 
-      // We say the value *is* loaded.
-      GetHeader()->SetEntryByNumber( GDCM_BINLOADED,
-         GetHeader()->GetGrPixel(), GetHeader()->GetNumPixel());
+   // We say the value *is* loaded.
+   SetPixelData(decompressed);
  
-      // will be 7fe0, 0010 in standard cases
-      GetHeader()->SetEntryBinAreaByNumber(Pixel_Data, 
-         GetHeader()->GetGrPixel(), GetHeader()->GetNumPixel());
-   } 
    PixelRead = 1; // PixelRaw
+// END PIXELCONVERT CLEANME
 
-   return Pixel_Data;
+   return decompressed;*/
 }
 
-/**
- * \brief   Copies at most MaxSize bytes of pixel data to caller's
- *          memory space.
- * \warning This function was designed to avoid people that want to build
- *          a volume from an image stack to need first to get the image pixels 
- *          and then move them to the volume area.
- *          It's absolutely useless for any VTK user since vtk chooses 
- *          to invert the lines of an image, that is the last line comes first
- *          (for some axis related reasons?). Hence he will have 
- *          to load the image line by line, starting from the end.
- *          VTK users hace to call GetImageData
- * \warning DOES NOT transform the Grey Plane + Palette Color (if any) 
- *                   into a single RGB Pixels Plane
- *          the (VTK) user will manage the palettes
- *     
- * @param   destination Address (in caller's memory space) at which the
- *          pixel data should be copied
- * @param   maxSize Maximum number of bytes to be copied. When MaxSize
- *          is not sufficient to hold the pixel data the copy is not
- *          executed (i.e. no partial copy).
- * @return  On success, the number of bytes actually copied. Zero on
- *          failure e.g. MaxSize is lower than necessary.
- */
-void File::GetImageDataIntoVectorRaw (void* destination, size_t maxSize)
+uint8_t* File::GetDecompressed()
 {
-  // we save the initial values of the following
-  // in order to be able to restore the header in a disk-consistent state
-  // (if user asks twice to get the pixels from disk)
-
-   if ( PixelRead != -1 ) // File was "read" before
-   {  
-      RestoreInitialValues(); 
-   }
-   
-   PixelRead = 1 ; // PixelRaw
-    
-   if ( ImageDataSize > maxSize )
-   {
-      dbg.Verbose(0, "File::GetImageDataIntoVector: pixel data bigger"
-                     "than caller's expected MaxSize");
-      return;
-   }
-
-   FILE* fp = HeaderInternal->OpenFile();
-   PixelConverter->ReadAndDecompressPixelData( fp );
-   HeaderInternal->CloseFile();
-   memmove( destination,
-            (void*)PixelConverter->GetDecompressed(),
-            PixelConverter->GetDecompressedSize() );
-
-   if ( ! PixelConverter->IsDecompressedRGB() )
+   uint8_t* decompressed = PixelConverter->GetDecompressed();
+   if ( ! decompressed )
    {
-      return;
+      // The decompressed image migth not be loaded yet:
+      std::ifstream* fp = HeaderInternal->OpenFile();
+      PixelConverter->ReadAndDecompressPixelData( fp );
+      if(fp) HeaderInternal->CloseFile();
+      decompressed = PixelConverter->GetDecompressed();
+      if ( ! decompressed )
+      {
+        dbg.Verbose(0, "File::GetDecompressed: read/decompress of "
+                       "pixel data apparently went wrong.");
+         return 0;
+      }
    }
 
-///////////////////////////////////////////////////
-   // now, it's an RGB image
-   // Lets's write it in the Header
-   // Droping Palette Color out of the Header
-   // has been moved to the Write process.
-
-   // TODO : move 'values' modification to the write process
-   //      : save also (in order to be able to restore)
-   //      : 'high bit' -when not equal to 'bits stored' + 1
-   //      : 'bits allocated', when it's equal to 12 ?!
-
-   std::string spp = "3";            // Samples Per Pixel
-   std::string photInt = "RGB ";     // Photometric Interpretation
-   std::string planConfig = "0";     // Planar Configuration
-     
-   HeaderInternal->SetEntryByNumber(spp,0x0028,0x0002);
-   HeaderInternal->SetEntryByNumber(photInt,0x0028,0x0004);
-   HeaderInternal->SetEntryByNumber(planConfig,0x0028,0x0006);
-   return; 
+   return decompressed;
 }
 
 /**
@@ -561,14 +465,14 @@ bool File::SetImageData(uint8_t* inData, size_t expectedSize)
 
 bool File::WriteRawData(std::string const & fileName)
 {
-   FILE* fp1 = fopen(fileName.c_str(), "wb");
-   if (fp1 == NULL)
+  std::ofstream fp1(fileName.c_str(), std::ios::out | std::ios::binary );
+   if (!fp1)
    {
-      printf("Fail to open (write) file [%s] \n", fileName.c_str());
+      dbg.Verbose(2, "Fail to open (write) file:", fileName.c_str());
       return false;
    }
-   fwrite (Pixel_Data, ImageDataSize, 1, fp1);
-   fclose (fp1);
+   fp1.write((char*)Pixel_Data, ImageDataSize);
+   fp1.close();
 
    return true;
 }
@@ -584,7 +488,8 @@ bool File::WriteRawData(std::string const & fileName)
 
 bool File::WriteDcmImplVR (std::string const & fileName)
 {
-   return WriteBase(fileName, ImplicitVR);
+   SetWriteTypeToDcmImplVR();
+   return Write(fileName);
 }
 
 /**
@@ -597,7 +502,8 @@ bool File::WriteDcmImplVR (std::string const & fileName)
 
 bool File::WriteDcmExplVR (std::string const & fileName)
 {
-   return WriteBase(fileName, ExplicitVR);
+   SetWriteTypeToDcmExplVR();
+   return Write(fileName);
 }
 
 /**
@@ -616,7 +522,22 @@ bool File::WriteDcmExplVR (std::string const & fileName)
 
 bool File::WriteAcr (std::string const & fileName)
 {
-   return WriteBase(fileName, ACR);
+   SetWriteTypeToAcr();
+   return Write(fileName);
+}
+
+bool File::Write(std::string const& fileName)
+{
+   switch(WriteType)
+   {
+      case WTYPE_IMPL_VR:
+         return WriteBase(fileName,ImplicitVR);
+      case WTYPE_EXPL_VR:
+         return WriteBase(fileName,ExplicitVR);
+      case WTYPE_ACR:
+         return WriteBase(fileName,ACR);
+   }
+   return(false);
 }
 
 //-----------------------------------------------------------------------------
@@ -631,27 +552,39 @@ bool File::WriteAcr (std::string const & fileName)
  */
 bool File::WriteBase (std::string const & fileName, FileType type)
 {
-   if ( PixelRead == -1 && type != ExplicitVR)
+/*   if ( PixelRead == -1 && type != ExplicitVR)
    {
       return false;
-   }
+   }*/
 
-   FILE* fp1 = fopen(fileName.c_str(), "wb");
+   std::ofstream* fp1 = new std::ofstream(fileName.c_str(), 
+                              std::ios::out | std::ios::binary);
    if (fp1 == NULL)
    {
-      printf("Failed to open (write) File [%s] \n", fileName.c_str());
+      dbg.Verbose(2, "Failed to open (write) File: " , fileName.c_str());
       return false;
    }
 
    if ( type == ImplicitVR || type == ExplicitVR )
    {
       // writing Dicom File Preamble
-      uint8_t* filePreamble = new uint8_t[128];
+      char filePreamble[128];
       memset(filePreamble, 0, 128);
-      fwrite(filePreamble, 128, 1, fp1);
-      fwrite("DICM", 4, 1, fp1);
+      fp1->write(filePreamble, 128);
+      fp1->write("DICM", 4);
+   }
 
-      delete[] filePreamble;
+   switch(WriteMode)
+   {
+      case WMODE_NATIVE :
+         SetWriteToNative();
+         break;
+      case WMODE_DECOMPRESSED :
+         SetWriteToDecompressed();
+         break;
+      case WMODE_RGB :
+         SetWriteToRGB();
+         break;
    }
 
    // --------------------------------------------------------------
@@ -667,15 +600,11 @@ bool File::WriteBase (std::string const & fileName, FileType type)
    std::string rows, columns; 
    if ( HeaderInternal->GetFileType() == ACR_LIBIDO)
    {
-      rows    = HeaderInternal->GetEntryByNumber(0x0028, 0x0010);
-      columns = HeaderInternal->GetEntryByNumber(0x0028, 0x0011);
-
-      HeaderInternal->SetEntryByNumber(columns,  0x0028, 0x0010);
-      HeaderInternal->SetEntryByNumber(rows   ,  0x0028, 0x0011);
+      SetWriteToLibido();
    }
    // ----------------- End of Special Patch ----------------
       
-   uint16_t grPixel  = HeaderInternal->GetGrPixel();
+/*   uint16_t grPixel  = HeaderInternal->GetGrPixel();
    uint16_t numPixel = HeaderInternal->GetNumPixel();;
           
    DocEntry* PixelElement = 
@@ -690,7 +619,7 @@ bool File::WriteBase (std::string const & fileName, FileType type)
    {
       // we tranformed GrayLevel pixels + LUT into RGB Pixel
       PixelElement->SetLength( ImageDataSize );
-   }
+   }*/
  
    HeaderInternal->Write(fp1, type);
 
@@ -702,13 +631,15 @@ bool File::WriteBase (std::string const & fileName, FileType type)
 
    if ( HeaderInternal->GetFileType() == ACR_LIBIDO )
    {
-      HeaderInternal->SetEntryByNumber(rows   , 0x0028, 0x0010);
-      HeaderInternal->SetEntryByNumber(columns, 0x0028, 0x0011);
+      RestoreWriteFromLibido();
    }
    // ----------------- End of Special Patch ----------------
-   
-   // fwrite(Pixel_Data, ImageDataSize, 1, fp1);  // should be useless, now
-   fclose (fp1);
+
+   RestoreWrite();
+
+
+   fp1->close ();
+   delete fp1;
 
    return true;
 }
@@ -721,5 +652,164 @@ uint8_t* File::GetLutRGBA()
    return PixelConverter->GetLutRGBA();
 }
 
+//-----------------------------------------------------------------------------
+// Private
+/**
+ * \brief Set the pixel datas in the good entry of the Header
+ */
+void File::SetPixelData(uint8_t* data)
+{
+   GetHeader()->SetEntryByNumber( GDCM_BINLOADED,
+      GetHeader()->GetGrPixel(), GetHeader()->GetNumPixel());
+
+   // Will be 7fe0, 0010 in standard case
+   DocEntry* currentEntry = GetHeader()->GetDocEntryByNumber(GetHeader()->GetGrPixel(), GetHeader()->GetNumPixel());
+   if ( currentEntry )
+   {
+      if ( BinEntry* binEntry = dynamic_cast<BinEntry *>(currentEntry) )
+         // Flag is to false because datas are kept in the gdcmPixelConvert
+         binEntry->SetBinArea( data, false );
+   }
+}
+
+void File::SetWriteToNative()
+{
+// Nothing to do
+}
+
+void File::SetWriteToDecompressed()
+{
+//   if (( !HeaderInternal->HasLUT() ) || (!PixelConverter->BuildRGBImage()))
+   if(HeaderInternal->HasLUT() && PixelConverter->BuildRGBImage())
+   {
+      SetWriteToRGB();
+   } 
+   else
+   {
+      ValEntry* photInt = CopyValEntry(0x0028,0x0004);
+      photInt->SetValue("MONOCHROME1 ");
+      photInt->SetLength(12);
+
+      BinEntry* pixel = CopyBinEntry(GetHeader()->GetGrPixel(),GetHeader()->GetNumPixel());
+      pixel->SetValue(GDCM_BINLOADED);
+      pixel->SetBinArea(PixelConverter->GetDecompressed(),false);
+      pixel->SetLength(PixelConverter->GetDecompressedSize());
+
+      Archive->Push(photInt);
+      Archive->Push(pixel);
+   }
+/*   else 
+   {
+      SetWriteToRGB();
+   } */
+}
+
+void File::SetWriteToRGB()
+{
+   if(PixelConverter->BuildRGBImage())
+   {
+      ValEntry* spp = CopyValEntry(0x0028,0x0002);
+      spp->SetValue("3 ");
+      spp->SetLength(2);
+
+      ValEntry* photInt = CopyValEntry(0x0028,0x0004);
+      photInt->SetValue("RGB ");
+      photInt->SetLength(4);
+
+      ValEntry* planConfig = CopyValEntry(0x0028,0x0006);
+      planConfig->SetValue("0 ");
+      planConfig->SetLength(2);
+
+      BinEntry* pixel = CopyBinEntry(GetHeader()->GetGrPixel(),GetHeader()->GetNumPixel());
+      pixel->SetValue(GDCM_BINLOADED);
+      pixel->SetBinArea(PixelConverter->GetRGB(),false);
+      pixel->SetLength(PixelConverter->GetRGBSize());
+
+      Archive->Push(spp);
+      Archive->Push(photInt);
+      Archive->Push(planConfig);
+      Archive->Push(pixel);
+   }
+   else
+   {
+      SetWriteToDecompressed();
+   }
+}
+
+void File::RestoreWrite()
+{
+   Archive->Restore(0x0028,0x0002);
+   Archive->Restore(0x0028,0x0004);
+   Archive->Restore(0x0028,0x0006);
+   Archive->Restore(GetHeader()->GetGrPixel(),GetHeader()->GetNumPixel());
+}
+
+void File::SetWriteToLibido()
+{
+   ValEntry *oldRow = dynamic_cast<ValEntry *>(HeaderInternal->GetDocEntryByNumber(0x0028, 0x0010));
+   ValEntry *oldCol = dynamic_cast<ValEntry *>(HeaderInternal->GetDocEntryByNumber(0x0028, 0x0011));
+   
+   if( oldRow && oldCol )
+   {
+      std::string rows, columns; 
+
+      ValEntry *newRow=new ValEntry(oldRow->GetDictEntry());
+      ValEntry *newCol=new ValEntry(oldCol->GetDictEntry());
+
+      newRow->Copy(oldCol);
+      newCol->Copy(oldRow);
+
+      newRow->SetValue(oldCol->GetValue());
+      newCol->SetValue(oldRow->GetValue());
+
+      Archive->Push(newRow);
+      Archive->Push(newCol);
+   }
+}
+
+void File::RestoreWriteFromLibido()
+{
+   Archive->Restore(0x0028,0x0010);
+   Archive->Restore(0x0028,0x0011);
+}
+
+ValEntry* File::CopyValEntry(uint16_t group,uint16_t element)
+{
+   DocEntry* oldE = HeaderInternal->GetDocEntryByNumber(group, element);
+   ValEntry* newE;
+
+   if(oldE)
+   {
+      newE = new ValEntry(oldE->GetDictEntry());
+      newE->Copy(oldE);
+   }
+   else
+   {
+      newE = GetHeader()->NewValEntryByNumber(group,element);
+   }
+
+   return(newE);
+}
+
+BinEntry* File::CopyBinEntry(uint16_t group,uint16_t element)
+{
+   DocEntry* oldE = HeaderInternal->GetDocEntryByNumber(group, element);
+   BinEntry* newE;
+
+   if(oldE)
+   {
+      newE = new BinEntry(oldE->GetDictEntry());
+      newE->Copy(oldE);
+   }
+   else
+   {
+      newE = GetHeader()->NewBinEntryByNumber(group,element);
+   }
+
+
+   return(newE);
+}
+
+//-----------------------------------------------------------------------------
 } // end namespace gdcm