]> Creatis software - gdcm.git/blobdiff - src/gdcmPixelReadConvert.cxx
STYLE: Remove old comments
[gdcm.git] / src / gdcmPixelReadConvert.cxx
index 1d59ff9a31c53ca78e8fb095f3b23e21899b0f8f..3e4a1e346c8b0db4cc26618b18ec9bc0e4759363 100644 (file)
@@ -1,10 +1,10 @@
 /*=========================================================================
-
   Program:   gdcm
   Module:    $RCSfile: gdcmPixelReadConvert.cxx,v $
   Language:  C++
-  Date:      $Date: 2005/02/15 18:12:35 $
-  Version:   $Revision: 1.50 $
+  Date:      $Date: 2005/09/07 14:12:23 $
+  Version:   $Revision: 1.77 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
                                                                                 
 =========================================================================*/
 
+#include "gdcmPixelReadConvert.h"
 #include "gdcmDebug.h"
 #include "gdcmFile.h"
 #include "gdcmGlobal.h"
 #include "gdcmTS.h"
-#include "gdcmPixelReadConvert.h"
 #include "gdcmDocEntry.h"
 #include "gdcmRLEFramesInfo.h"
 #include "gdcmJPEGFragmentsInfo.h"
 
 namespace gdcm
 {
+
+//bool ReadMPEGFile (std::ifstream *fp, void *image_buffer, size_t lenght);
+bool gdcm_read_JPEG2000_file (void* raw, 
+                              char *inputdata, size_t inputlength);
 //-----------------------------------------------------------------------------
 #define str2num(str, typeNum) *((typeNum *)(str))
 
@@ -46,6 +50,10 @@ PixelReadConvert::PixelReadConvert()
    LutRedData   = 0;
    LutGreenData = 0;
    LutBlueData  = 0;
+   RLEInfo      = 0;
+   JPEGInfo     = 0;
+   UserFunction = 0;
+   FileInternal = 0;
 }
 
 /// Canonical Destructor
@@ -103,7 +111,7 @@ void PixelReadConvert::GrabInformationsFromFile( File *file )
    YSize           = file->GetYSize();
    ZSize           = file->GetZSize();
    SamplesPerPixel = file->GetSamplesPerPixel();
-   PixelSize       = file->GetPixelSize();
+   //PixelSize       = file->GetPixelSize();  Useless
    PixelSign       = file->IsSignedPixelData();
    SwapCode        = file->GetSwapCode();
    std::string ts  = file->GetTransferSyntax();
@@ -115,6 +123,7 @@ void PixelReadConvert::GrabInformationsFromFile( File *file )
      || Global::GetTS()->GetSpecialTransferSyntax(ts) == TS::ExplicitVRBigEndian
      || Global::GetTS()->GetSpecialTransferSyntax(ts) == TS::DeflatedExplicitVRLittleEndian;
 
+   IsMPEG          = Global::GetTS()->IsMPEG(ts);
    IsJPEG2000      = Global::GetTS()->IsJPEG2000(ts);
    IsJPEGLS        = Global::GetTS()->IsJPEGLS(ts);
    IsJPEGLossy     = Global::GetTS()->IsJPEGLossy(ts);
@@ -162,7 +171,7 @@ void PixelReadConvert::GrabInformationsFromFile( File *file )
       LutRedData = (uint8_t*)file->GetEntryBinArea( 0x0028, 0x1201 );
       if ( ! LutRedData )
       {
-         gdcmWarningMacro( "Unable to read Red LUT data" );
+         gdcmWarningMacro( "Unable to read Red Palette Color Lookup Table data" );
       }
 
       // //// Green round:
@@ -170,7 +179,7 @@ void PixelReadConvert::GrabInformationsFromFile( File *file )
       LutGreenData = (uint8_t*)file->GetEntryBinArea(0x0028, 0x1202 );
       if ( ! LutGreenData)
       {
-         gdcmWarningMacro( "Unable to read Green LUT data" );
+         gdcmWarningMacro( "Unable to read Green Palette Color Lookup Table data" );
       }
 
       // //// Blue round:
@@ -178,9 +187,10 @@ void PixelReadConvert::GrabInformationsFromFile( File *file )
       LutBlueData = (uint8_t*)file->GetEntryBinArea( 0x0028, 0x1203 );
       if ( ! LutBlueData )
       {
-         gdcmWarningMacro( "Unable to read Blue LUT data" );
+         gdcmWarningMacro( "Unable to read Blue Palette Color Lookup Table data" );
       }
    }
+   FileInternal = file;   
 
    ComputeRawAndRGBSizes();
 }
@@ -202,7 +212,7 @@ bool PixelReadConvert::ReadAndDecompressPixelData( std::ifstream *fp )
    }
 
    fp->seekg( PixelOffset, std::ios::beg );
-   if( fp->fail() || fp->eof())
+   if ( fp->fail() || fp->eof() )
    {
       gdcmWarningMacro( "Unable to find PixelOffset in file." );
       return false;
@@ -211,7 +221,7 @@ bool PixelReadConvert::ReadAndDecompressPixelData( std::ifstream *fp )
    AllocateRaw();
 
    //////////////////////////////////////////////////
-   //// Second stage: read from disk dans decompress.
+   //// Second stage: read from disk and decompress.
    if ( BitsAllocated == 12 )
    {
       ReadAndDecompress12BitsTo16Bits( fp);
@@ -222,12 +232,12 @@ bool PixelReadConvert::ReadAndDecompressPixelData( std::ifstream *fp )
       // after the field containing the image data. In this case, these
       // bad data are added to the size of the image (in the PixelDataLength
       // variable). But RawSize is the right size of the image !
-      if( PixelDataLength != RawSize)
+      if ( PixelDataLength != RawSize )
       {
          gdcmWarningMacro( "Mismatch between PixelReadConvert : "
                             << PixelDataLength << " and RawSize : " << RawSize );
       }
-      if( PixelDataLength > RawSize)
+      if ( PixelDataLength > RawSize )
       {
          fp->read( (char*)Raw, RawSize);
       }
@@ -250,6 +260,14 @@ bool PixelReadConvert::ReadAndDecompressPixelData( std::ifstream *fp )
          return false;
       }
    }
+   else if ( IsMPEG )
+   {
+      //gdcmWarningMacro( "Sorry, MPEG not yet taken into account" );
+      //return false;
+      // fp has already been seek to start of mpeg
+      //ReadMPEGFile(fp, Raw, PixelDataLength); 
+      return true;
+   }
    else
    {
       // Default case concerns JPEG family
@@ -265,6 +283,8 @@ bool PixelReadConvert::ReadAndDecompressPixelData( std::ifstream *fp )
    ConvertReorderEndianity();
    ConvertReArrangeBits();
    ConvertFixGreyLevels();
+   if (UserFunction) // user is allowed to Mirror, TopDown, Rotate,...the image
+      UserFunction( Raw, FileInternal);
    ConvertHandleColor();
 
    return true;
@@ -287,7 +307,7 @@ void PixelReadConvert::Squeeze()
 }
 
 /**
- * \brief Build the RGB image from the Raw imagage and the LUTs.
+ * \brief Build the RGB image from the Raw image and the LUTs.
  */
 bool PixelReadConvert::BuildRGBImage()
 {
@@ -309,17 +329,37 @@ bool PixelReadConvert::BuildRGBImage()
       // The job can't be done
       return false;
    }
+
+   gdcmWarningMacro( "--> BuildRGBImage" );
                                                                                 
    // Build RGB Pixels
    AllocateRGB();
-   uint8_t *localRGB = RGB;
-   for (size_t i = 0; i < RawSize; ++i )
+   
+   int j;
+   if ( BitsAllocated <= 8 )
    {
-      int j  = Raw[i] * 4;
-      *localRGB++ = LutRGBA[j];
-      *localRGB++ = LutRGBA[j+1];
-      *localRGB++ = LutRGBA[j+2];
-   }
+      uint8_t *localRGB = RGB;
+      for (size_t i = 0; i < RawSize; ++i )
+      {
+         j  = Raw[i] * 4;
+         *localRGB++ = LutRGBA[j];
+         *localRGB++ = LutRGBA[j+1];
+         *localRGB++ = LutRGBA[j+2];
+      }
+    }
+    else  // deal with 16 bits pixels and 16 bits Palette color
+    {
+      uint16_t *localRGB = (uint16_t *)RGB;
+      for (size_t i = 0; i < RawSize/2; ++i )
+      {
+         j  = ((uint16_t *)Raw)[i] * 4;
+         *localRGB++ = ((uint16_t *)LutRGBA)[j];
+         *localRGB++ = ((uint16_t *)LutRGBA)[j+1];
+         *localRGB++ = ((uint16_t *)LutRGBA)[j+2];
+      } 
+    }
    return true;
 }
 
@@ -386,27 +426,96 @@ bool PixelReadConvert::ReadAndDecompressJPEGFile( std::ifstream *fp )
 {
    if ( IsJPEG2000 )
    {
-      gdcmWarningMacro( "Sorry, JPEG2000 not yet taken into account" );
-      fp->seekg( JPEGInfo->GetFirstFragment()->GetOffset(), std::ios::beg);
-//    if ( ! gdcm_read_JPEG2000_file( fp,Raw ) )
-          return false;
+     // make sure this is the right JPEG compression
+     assert( !IsJPEGLossless || !IsJPEGLossy || !IsJPEGLS );
+     // FIXME this is really ugly but it seems I have to load the complete
+     // jpeg2000 stream to use jasper:
+     // I don't think we'll ever be able to deal with multiple fragments properly
+
+      unsigned long inputlength = 0;
+      JPEGFragment *jpegfrag = JPEGInfo->GetFirstFragment();
+      while( jpegfrag )
+      {
+         inputlength += jpegfrag->GetLength();
+         jpegfrag = JPEGInfo->GetNextFragment();
+      }
+      gdcmAssertMacro( inputlength != 0);
+      uint8_t *inputdata = new uint8_t[inputlength];
+      char *pinputdata = (char*)inputdata;
+      jpegfrag = JPEGInfo->GetFirstFragment();
+      while( jpegfrag )
+      {
+         fp->seekg( jpegfrag->GetOffset(), std::ios::beg);
+         fp->read(pinputdata, jpegfrag->GetLength());
+         pinputdata += jpegfrag->GetLength();
+         jpegfrag = JPEGInfo->GetNextFragment();
+      }
+      // Warning the inputdata buffer is delete in the function
+      if ( ! gdcm_read_JPEG2000_file( Raw, 
+          (char*)inputdata, inputlength ) )
+      {
+         return true;
+      }
+      // wow what happen, must be an error
+      return false;
    }
-
-   if ( IsJPEGLS )
+   else if ( IsJPEGLS )
    {
+     // make sure this is the right JPEG compression
+     assert( !IsJPEGLossless || !IsJPEGLossy || !IsJPEG2000 );
+   // WARNING : JPEG-LS is NOT the 'classical' Jpeg Lossless : 
+   // [JPEG-LS is the basis for new lossless/near-lossless compression
+   // standard for continuous-tone images intended for JPEG2000. The standard
+   // is based on the LOCO-I algorithm (LOw COmplexity LOssless COmpression
+   // for Images) developed at Hewlett-Packard Laboratories]
+   //
+   // see http://datacompression.info/JPEGLS.shtml
+   //
+#if 0
+   std::cerr << "count:" << JPEGInfo->GetFragmentCount() << std::endl;
+      unsigned long inputlength = 0;
+      JPEGFragment *jpegfrag = JPEGInfo->GetFirstFragment();
+      while( jpegfrag )
+      {
+         inputlength += jpegfrag->GetLength();
+         jpegfrag = JPEGInfo->GetNextFragment();
+      }
+      gdcmAssertMacro( inputlength != 0);
+      uint8_t *inputdata = new uint8_t[inputlength];
+      char *pinputdata = (char*)inputdata;
+      jpegfrag = JPEGInfo->GetFirstFragment();
+      while( jpegfrag )
+      {
+         fp->seekg( jpegfrag->GetOffset(), std::ios::beg);
+         fp->read(pinputdata, jpegfrag->GetLength());
+         pinputdata += jpegfrag->GetLength();
+         jpegfrag = JPEGInfo->GetNextFragment();
+      }  
+      
+  //fp->read((char*)Raw, PixelDataLength);
+
+  std::ofstream out("/tmp/jpegls.jpg");
+  out.write((char*)inputdata, inputlength);
+  out.close();
+  delete[] inputdata;
+#endif
+
       gdcmWarningMacro( "Sorry, JPEG-LS not yet taken into account" );
       fp->seekg( JPEGInfo->GetFirstFragment()->GetOffset(), std::ios::beg);
 //    if ( ! gdcm_read_JPEGLS_file( fp,Raw ) )
          return false;
    }
-
-   // else ??
-   // Precompute the offset localRaw will be shifted with
-   int length = XSize * YSize * SamplesPerPixel;
-   int numberBytes = BitsAllocated / 8;
-
-   JPEGInfo->DecompressFromFile(fp, Raw, BitsStored, numberBytes, length );
-   return true;
+   else
+   {
+     // make sure this is the right JPEG compression
+     assert( !IsJPEGLS || !IsJPEG2000 );
+     // Precompute the offset localRaw will be shifted with
+     int length = XSize * YSize * SamplesPerPixel;
+     int numberBytes = BitsAllocated / 8;
+
+     JPEGInfo->DecompressFromFile(fp, Raw, BitsStored, numberBytes, length );
+     return true;
+   }
 }
 
 /**
@@ -442,6 +551,7 @@ void PixelReadConvert::BuildLUTRGBA()
        || LutGreenDescriptor == GDCM_UNFOUND
        || LutBlueDescriptor  == GDCM_UNFOUND )
    {
+      gdcmWarningMacro( "(At least) a LUT Descriptor is missing" );
       return;
    }
 
@@ -450,21 +560,23 @@ void PixelReadConvert::BuildLUTRGBA()
    int lengthR;   // Red LUT length in Bytes
    int debR;      // Subscript of the first Lut Value
    int nbitsR;    // Lut item size (in Bits)
-   int nbRead = sscanf( LutRedDescriptor.c_str(),
+   int nbRead;    // nb of items in LUT descriptor (must be = 3)
+
+   nbRead = sscanf( LutRedDescriptor.c_str(),
                         "%d\\%d\\%d",
                         &lengthR, &debR, &nbitsR );
-   if( nbRead != 3 )
+   if ( nbRead != 3 )
    {
       gdcmWarningMacro( "Wrong Red LUT descriptor" );
-   }
-                                                                                
+   }                                                                                
    int lengthG;  // Green LUT length in Bytes
    int debG;     // Subscript of the first Lut Value
    int nbitsG;   // Lut item size (in Bits)
+
    nbRead = sscanf( LutGreenDescriptor.c_str(),
                     "%d\\%d\\%d",
-                    &lengthG, &debG, &nbitsG );
-   if( nbRead != 3 )
+                    &lengthG, &debG, &nbitsG );  
+   if ( nbRead != 3 )
    {
       gdcmWarningMacro( "Wrong Green LUT descriptor" );
    }
@@ -475,71 +587,162 @@ void PixelReadConvert::BuildLUTRGBA()
    nbRead = sscanf( LutRedDescriptor.c_str(),
                     "%d\\%d\\%d",
                     &lengthB, &debB, &nbitsB );
-   if( nbRead != 3 )
+   if ( nbRead != 3 )
    {
       gdcmWarningMacro( "Wrong Blue LUT descriptor" );
    }
+   gdcmWarningMacro(" lengthR " << lengthR << " debR " 
+                 << debR << " nbitsR " << nbitsR);
+   gdcmWarningMacro(" lengthG " << lengthG << " debG " 
+                 << debG << " nbitsG " << nbitsG);
+   gdcmWarningMacro(" lengthB " << lengthB << " debB " 
+                 << debB << " nbitsB " << nbitsB);
+
+   if ( !lengthR ) // if = 2^16, this shall be 0 see : CP-143
+      lengthR=65536;
+   if ( !lengthG ) // if = 2^16, this shall be 0
+      lengthG=65536;
+   if ( !lengthB ) // if = 2^16, this shall be 0
+      lengthB=65536; 
                                                                                 
    ////////////////////////////////////////////////////////
+
    if ( ( ! LutRedData ) || ( ! LutGreenData ) || ( ! LutBlueData ) )
    {
+      gdcmWarningMacro( "(At least) a LUT is missing" );
       return;
    }
 
-   ////////////////////////////////////////////////
-   // forge the 4 * 8 Bits Red/Green/Blue/Alpha LUT
-   LutRGBA = new uint8_t[ 1024 ]; // 256 * 4 (R, G, B, Alpha)
-   if ( !LutRGBA )
-      return;
-
-   memset( LutRGBA, 0, 1024 );
-                                                                                
-   int mult;
-   if ( ( nbitsR == 16 ) && ( BitsAllocated == 8 ) )
-   {
-      // when LUT item size is different than pixel size
-      mult = 2; // high byte must be = low byte
-   }
-   else
+   // -------------------------------------------------------------
+   
+   if ( BitsAllocated <= 8 )
    {
-      // See PS 3.3-2003 C.11.1.1.2 p 619
-      mult = 1;
-   }
+      // forge the 4 * 8 Bits Red/Green/Blue/Alpha LUT
+      LutRGBA = new uint8_t[ 1024 ]; // 256 * 4 (R, G, B, Alpha)
+      if ( !LutRGBA )
+         return;
+      LutItemNumber = 256;
+      LutItemSize   = 8;
+      memset( LutRGBA, 0, 1024 );
                                                                                 
-   // if we get a black image, let's just remove the '+1'
-   // from 'i*mult+1' and check again
-   // if it works, we shall have to check the 3 Palettes
-   // to see which byte is ==0 (first one, or second one)
-   // and fix the code
-   // We give up the checking to avoid some (useless ?) overhead
-   // (optimistic asumption)
-   int i;
-   uint8_t *a = LutRGBA + 0;
-   for( i=0; i < lengthR; ++i )
-   {
-      *a = LutRedData[i*mult+1];
-      a += 4;
-   }
+      int mult;
+      if ( ( nbitsR == 16 ) && ( BitsAllocated == 8 ) )
+      {
+         // when LUT item size is different than pixel size
+         mult = 2; // high byte must be = low byte
+      }
+      else
+      {
+         // See PS 3.3-2003 C.11.1.1.2 p 619
+         mult = 1;
+      }
                                                                                 
-   a = LutRGBA + 1;
-   for( i=0; i < lengthG; ++i)
-   {
-      *a = LutGreenData[i*mult+1];
-      a += 4;
-   }
+      // if we get a black image, let's just remove the '+1'
+      // from 'i*mult+1' and check again
+      // if it works, we shall have to check the 3 Palettes
+      // to see which byte is ==0 (first one, or second one)
+      // and fix the code
+      // We give up the checking to avoid some (useless ?) overhead
+      // (optimistic asumption)
+      int i;
+      uint8_t *a;
+
+      //take "Subscript of the first Lut Value" (debR,debG,debB) into account!
+
+      //FIXME :  +1 : to get 'low value' byte
+      //         Trouble expected on Big Endian Processors ?
+      //         16 BIts Per Pixel Palette Color to be swapped?
+
+      a = LutRGBA + 0 + debR;
+      for( i=0; i < lengthR; ++i )
+      {
+         *a = LutRedData[i*mult+1]; 
+         a += 4;
+      }
                                                                                 
-   a = LutRGBA + 2;
-   for(i=0; i < lengthB; ++i)
-   {
-      *a = LutBlueData[i*mult+1];
-      a += 4;
-   }
+      a = LutRGBA + 1 + debG;
+      for( i=0; i < lengthG; ++i)
+      {
+         *a = LutGreenData[i*mult+1];
+         a += 4;
+      }
                                                                                 
-   a = LutRGBA + 3;
-   for(i=0; i < 256; ++i)
+      a = LutRGBA + 2 + debB;
+      for(i=0; i < lengthB; ++i)
+      {
+         *a = LutBlueData[i*mult+1];
+         a += 4;
+      }
+                                    
+      a = LutRGBA + 3 ;
+      for(i=0; i < 256; ++i)
+      {
+         *a = 1; // Alpha component
+         a += 4;
+      }
+   }
+   else
    {
-      *a = 1; // Alpha component
-      a += 4;
+      // Probabely the same stuff is to be done for 16 Bits Pixels
+      // with 65536 entries LUT ?!?
+      // Still looking for accurate info on the web :-(
+
+      gdcmWarningMacro( "Sorry Palette Color Lookup Tables not yet dealt with"
+                         << " for 16 Bits Per Pixel images" );
+
+      // forge the 4 * 16 Bits Red/Green/Blue/Alpha LUT
+
+      LutRGBA = (uint8_t *)new uint16_t[ 65536*4 ]; // 2^16 * 4 (R, G, B, Alpha)
+      if ( !LutRGBA )
+         return;
+      memset( LutRGBA, 0, 65536*4*2 );  // 16 bits = 2 bytes ;-)
+
+      LutItemNumber = 65536;
+      LutItemSize   = 16;
+
+      int i;
+      uint16_t *a16;
+
+      //take "Subscript of the first Lut Value" (debR,debG,debB) into account!
+
+      a16 = (uint16_t*)LutRGBA + 0 + debR;
+      for( i=0; i < lengthR; ++i )
+      {
+         *a16 = ((uint16_t*)LutRedData)[i];
+         a16 += 4;
+      }
+                                                                              
+      a16 = (uint16_t*)LutRGBA + 1 + debG;
+      for( i=0; i < lengthG; ++i)
+      {
+         *a16 = ((uint16_t*)LutGreenData)[i];
+         a16 += 4;
+      }
+                                                                                
+      a16 = (uint16_t*)LutRGBA + 2 + debB;
+      for(i=0; i < lengthB; ++i)
+      {
+         *a16 = ((uint16_t*)LutBlueData)[i];
+         a16 += 4;
+      }
+                                                                             
+      a16 = (uint16_t*)LutRGBA + 3 ;
+      for(i=0; i < 65536; ++i)
+      {
+         *a16 = 1; // Alpha component
+         a16 += 4;
+      }
+/* Just to 'see' the LUT, at debug time
+
+      a16=(uint16_t*)LutRGBA;
+      for (int j=0;j<65536;j++)
+      {
+         std::cout << *a16     << " " << *(a16+1) << " "
+                   << *(a16+2) << " " << *(a16+3) << std::endl;
+         a16+=4;
+      }
+*/
    }
 }
 
@@ -550,7 +753,7 @@ void PixelReadConvert::ConvertSwapZone()
 {
    unsigned int i;
 
-   if( BitsAllocated == 16 )
+   if ( BitsAllocated == 16 )
    {
       uint16_t *im16 = (uint16_t*)Raw;
       switch( SwapCode )
@@ -569,7 +772,7 @@ void PixelReadConvert::ConvertSwapZone()
             gdcmWarningMacro("SwapCode value (16 bits) not allowed.");
       }
    }
-   else if( BitsAllocated == 32 )
+   else if ( BitsAllocated == 32 )
    {
       uint32_t s32;
       uint16_t high;
@@ -635,7 +838,7 @@ void PixelReadConvert::ConvertReorderEndianity()
       uint16_t *deb = (uint16_t *)Raw;
       for(int i = 0; i<l; i++)
       {
-         if( *deb == 0xffff )
+         if ( *deb == 0xffff )
          {
            *deb = 0;
          }
@@ -644,8 +847,6 @@ void PixelReadConvert::ConvertReorderEndianity()
    }
 }
 
-
-
 /**
  * \brief Deal with Grey levels i.e. re-arange them
  *        to have low values = dark, high values = bright
@@ -655,72 +856,155 @@ void PixelReadConvert::ConvertFixGreyLevels()
    if (!IsMonochrome1)
       return;
 
-   int i; // to please M$VC6
-   if ( BitsAllocated == 8 )
+   uint32_t i; // to please M$VC6
+   int16_t j;
+
+   if (!PixelSign)
    {
-      uint8_t *deb = (uint8_t *)Raw;
-      for (i=0; i<RawSize; i++)      
+      if ( BitsAllocated == 8 )
       {
-         *deb = 255 - *deb;
-         deb++;
+         uint8_t *deb = (uint8_t *)Raw;
+         for (i=0; i<RawSize; i++)      
+         {
+            *deb = 255 - *deb;
+            deb++;
+         }
+         return;
       }
-      return;
-   }
 
-   if ( BitsAllocated == 16 )
+      if ( BitsAllocated == 16 )
+      {
+         uint16_t mask =1;
+         for (j=0; j<BitsStored-1; j++)
+         {
+            mask = (mask << 1) +1; // will be fff when BitsStored=12
+         }
+
+         uint16_t *deb = (uint16_t *)Raw;
+         for (i=0; i<RawSize/2; i++)      
+         {
+            *deb = mask - *deb;
+            deb++;
+         }
+         return;
+       }
+   }
+   else
    {
-      uint16_t mask =1;
-      int bitStored = 12;   
-      for (i=0; i<BitsStored-1; i++)
+      if ( BitsAllocated == 8 )
       {
-         mask = (mask << 1) +1; // will be fff when BitStored=12
+         uint8_t smask8 = 255;
+         uint8_t *deb = (uint8_t *)Raw;
+         for (i=0; i<RawSize; i++)      
+         {
+            *deb = smask8 - *deb;
+            deb++;
+         }
+         return;
       }
-
-      uint16_t *deb = (uint16_t *)Raw;
-      for (i=0; i<RawSize/2; i++)      
+      if ( BitsAllocated == 16 )
       {
-         *deb = mask - *deb;
-         deb++;
+         uint16_t smask16 = 65535;
+         uint16_t *deb = (uint16_t *)Raw;
+         for (i=0; i<RawSize/2; i++)      
+         {
+            *deb = smask16 - *deb;
+            deb++;
+         }
+         return;
       }
-      return;
    }
 }
 
-
 /**
  * \brief  Re-arrange the bits within the bytes.
- * @return Boolean
+ * @return Boolean always true
  */
 bool PixelReadConvert::ConvertReArrangeBits() throw ( FormatError )
 {
+
    if ( BitsStored != BitsAllocated )
    {
       int l = (int)( RawSize / ( BitsAllocated / 8 ) );
       if ( BitsAllocated == 16 )
       {
-         uint16_t mask = 0xffff;
-         mask = mask >> ( BitsAllocated - BitsStored );
+         // pmask : to mask the 'unused bits' (may contain overlays)
+         uint16_t pmask = 0xffff;
+         pmask = pmask >> ( BitsAllocated - BitsStored );
+
          uint16_t *deb = (uint16_t*)Raw;
-         for(int i = 0; i<l; i++)
+
+         if ( !PixelSign )  // Pixels are unsigned
          {
-            *deb = (*deb >> (BitsStored - HighBitPosition - 1)) & mask;
-            deb++;
+            for(int i = 0; i<l; i++)
+            {   
+               *deb = (*deb >> (BitsStored - HighBitPosition - 1)) & pmask;
+               deb++;
+            }
+         }
+         else // Pixels are signed
+         {
+            // smask : to check the 'sign' when BitsStored != BitsAllocated
+            uint16_t smask = 0x0001;
+            smask = smask << ( 16 - (BitsAllocated - BitsStored + 1) );
+            // nmask : to propagate sign bit on negative values
+            int16_t nmask = (int16_t)0x8000;  
+            nmask = nmask >> ( BitsAllocated - BitsStored - 1 );
+            for(int i = 0; i<l; i++)
+            {
+               *deb = *deb >> (BitsStored - HighBitPosition - 1);
+               if ( *deb & smask )
+               {
+                  *deb = *deb | nmask;
+               }
+               else
+               {
+                  *deb = *deb & pmask;
+               }
+               deb++;
+            }
          }
       }
       else if ( BitsAllocated == 32 )
       {
-         uint32_t mask = 0xffffffff;
-         mask = mask >> ( BitsAllocated - BitsStored );
+         // pmask : to mask the 'unused bits' (may contain overlays)
+         uint32_t pmask = 0xffffffff;
+         pmask = pmask >> ( BitsAllocated - BitsStored );
+
          uint32_t *deb = (uint32_t*)Raw;
-         for(int i = 0; i<l; i++)
+
+         if ( !PixelSign )
          {
-            *deb = (*deb >> (BitsStored - HighBitPosition - 1)) & mask;
-            deb++;
+            for(int i = 0; i<l; i++)
+            {             
+               *deb = (*deb >> (BitsStored - HighBitPosition - 1)) & pmask;
+               deb++;
+            }
+         }
+         else
+         {
+            // smask : to check the 'sign' when BitsStored != BitsAllocated
+            uint32_t smask = 0x00000001;
+            smask = smask >> ( 32 - (BitsAllocated - BitsStored +1 ));
+            // nmask : to propagate sign bit on negative values
+            int32_t nmask = 0x80000000;   
+            nmask = nmask >> ( BitsAllocated - BitsStored -1 );
+
+            for(int i = 0; i<l; i++)
+            {
+               *deb = *deb >> (BitsStored - HighBitPosition - 1);
+               if ( *deb & smask )
+                  *deb = *deb | nmask;
+               else
+                  *deb = *deb & pmask;
+               deb++;
+            }
          }
       }
       else
       {
-         gdcmWarningMacro("Weird image");
+         gdcmWarningMacro("Weird image (BitsAllocated !=8, 12, 16, 32)");
          throw FormatError( "Weird image !?" );
       }
    }
@@ -733,6 +1017,8 @@ bool PixelReadConvert::ConvertReArrangeBits() throw ( FormatError )
  */
 void PixelReadConvert::ConvertRGBPlanesToRGBPixels()
 {
+   gdcmWarningMacro("--> ConvertRGBPlanesToRGBPixels");
+
    uint8_t *localRaw = Raw;
    uint8_t *copyRaw = new uint8_t[ RawSize ];
    memmove( copyRaw, localRaw, RawSize );
@@ -758,6 +1044,17 @@ void PixelReadConvert::ConvertRGBPlanesToRGBPixels()
  */
 void PixelReadConvert::ConvertYcBcRPlanesToRGBPixels()
 {
+  // Remarks for YBR newbees :
+  // YBR_FULL works very much like RGB, i.e. three samples per pixel, 
+  // just the color space is YCbCr instead of RGB. This is particularly useful
+  // for doppler ultrasound where most of the image is grayscale 
+  // (i.e. only populates the Y components) and Cb and Cr are mostly zero,
+  // except for the few patches of color on the image.
+  // On such images, RLE achieves a compression ratio that is much better 
+  // than the compression ratio on an equivalent RGB image. 
+   gdcmWarningMacro("--> ConvertYcBcRPlanesToRGBPixels");
+   
    uint8_t *localRaw = Raw;
    uint8_t *copyRaw = new uint8_t[ RawSize ];
    memmove( copyRaw, localRaw, RawSize );
@@ -770,30 +1067,35 @@ void PixelReadConvert::ConvertYcBcRPlanesToRGBPixels()
    int l        = XSize * YSize;
    int nbFrames = ZSize;
 
-   uint8_t *a = copyRaw;
+   uint8_t *a = copyRaw + 0;
    uint8_t *b = copyRaw + l;
-   uint8_t *c = copyRaw + l + l;
-   double R, G, B;
+   uint8_t *c = copyRaw + l+ l;
+   int32_t R, G, B;
 
-   /// \todo : Replace by the 'well known' integer computation
-   ///         counterpart. Refer to
+   ///  We replaced easy to understand but time consuming floating point
+   ///  computations by the 'well known' integer computation counterpart
+   ///  Refer to :
    ///            http://lestourtereaux.free.fr/papers/data/yuvrgb.pdf
-   ///         for code optimisation.
+   ///  for code optimisation.
 
    for ( int i = 0; i < nbFrames; i++ )
    {
       for ( int j = 0; j < l; j++ )
       {
-         R = 1.164 *(*a-16) + 1.596 *(*c -128) + 0.5;
-         G = 1.164 *(*a-16) - 0.813 *(*c -128) - 0.392 *(*b -128) + 0.5;
-         B = 1.164 *(*a-16) + 2.017 *(*b -128) + 0.5;
+         R = 38142 *(*a-16) + 52298 *(*c -128);
+         G = 38142 *(*a-16) - 26640 *(*c -128) - 12845 *(*b -128);
+         B = 38142 *(*a-16) + 66093 *(*b -128);
+
+         R = (R+16384)>>15;
+         G = (G+16384)>>15;
+         B = (B+16384)>>15;
 
-         if (R < 0.0)   R = 0.0;
-         if (G < 0.0)   G = 0.0;
-         if (B < 0.0)   B = 0.0;
-         if (R > 255.0) R = 255.0;
-         if (G > 255.0) G = 255.0;
-         if (B > 255.0) B = 255.0;
+         if (R < 0)   R = 0;
+         if (G < 0)   G = 0;
+         if (B < 0)   B = 0;
+         if (R > 255) R = 255;
+         if (G > 255) G = 255;
+         if (B > 255) B = 255;
 
          *(localRaw++) = (uint8_t)R;
          *(localRaw++) = (uint8_t)G;
@@ -839,7 +1141,7 @@ void PixelReadConvert::ConvertHandleColor()
    //     - "Planar Configuration" = 0,
    //     - "Photometric Interpretation" = "PALETTE COLOR".
    // Hence gdcm will use the folowing "heuristic" in order to be tolerant
-   // towards Dicom-non-conformance files:
+   // towards Dicom-non-conformant files:
    //   << whatever the "Planar Configuration" value might be, a
    //      "Photometric Interpretation" set to "PALETTE COLOR" forces
    //      a LUT intervention >>
@@ -851,9 +1153,13 @@ void PixelReadConvert::ConvertHandleColor()
    // - [Planar 1] AND [Photo C] handled with ConvertYcBcRPlanesToRGBPixels()
    // - [Planar 2] OR  [Photo D] requires LUT intervention.
 
+   gdcmWarningMacro("--> ConvertHandleColor"
+                    << "Planar Configuration " << PlanarConfiguration );
+
    if ( ! IsRawRGB() )
    {
       // [Planar 2] OR  [Photo D]: LUT intervention done outside
+      gdcmWarningMacro("--> RawRGB : LUT intervention done outside");
       return;
    }
                                                                                 
@@ -862,22 +1168,27 @@ void PixelReadConvert::ConvertHandleColor()
       if ( IsYBRFull )
       {
          // [Planar 1] AND [Photo C] (remember YBR_FULL_422 acts as RGB)
+         gdcmWarningMacro("--> YBRFull");
          ConvertYcBcRPlanesToRGBPixels();
       }
       else
       {
          // [Planar 1] AND [Photo C]
+         gdcmWarningMacro("--> YBRFull");
          ConvertRGBPlanesToRGBPixels();
       }
       return;
    }
                                                                                 
    // When planarConf is 0, and RLELossless (forbidden by Dicom norm)
-   // pixels need to be RGB-fied anyway
+   // pixels need to be RGB-fyied anyway
+
    if (IsRLELossless)
-   {
+   { 
+     gdcmWarningMacro("--> RLE Lossless");
      ConvertRGBPlanesToRGBPixels();
    }
+
    // In *normal *case, when planarConf is 0, pixels are already in RGB
 }
 
@@ -898,7 +1209,7 @@ void PixelReadConvert::ComputeRawAndRGBSizes()
                      * SamplesPerPixel;
    if ( HasLUT )
    {
-      RGBSize = 3 * RawSize;
+      RGBSize = 3 * RawSize; // works for 8 and 16 bits per Pixel
    }
    else
    {
@@ -968,14 +1279,3 @@ void PixelReadConvert::Print( std::ostream &os, std::string const &indent )
 
 //-----------------------------------------------------------------------------
 } // end namespace gdcm
-
-// NOTES on File internal calls
-// User
-// ---> GetImageData
-//     ---> GetImageDataIntoVector
-//        |---> GetImageDataIntoVectorRaw
-//        | lut intervention
-// User
-// ---> GetImageDataRaw
-//     ---> GetImageDataIntoVectorRaw
-