]> Creatis software - gdcm.git/blobdiff - src/gdcmHeader.cxx
add '\0' after ostringstream to avoid oddities on Solrais
[gdcm.git] / src / gdcmHeader.cxx
index 13da756dc1bde8eecaabd7bc93872e5051b7e24e..e9ae38fc4c71c315e2f68879429bb3a6fc330da7 100644 (file)
@@ -1,4 +1,4 @@
-// $Header: /cvs/public/gdcm/src/Attic/gdcmHeader.cxx,v 1.101 2003/10/15 15:01:16 jpr Exp $
+// $Header: /cvs/public/gdcm/src/Attic/gdcmHeader.cxx,v 1.117 2003/11/13 18:08:34 jpr Exp $
 
 #include "gdcmHeader.h"
 
@@ -38,7 +38,14 @@ void gdcmHeader::Initialise(void) {
  * @param   InFilename
  * @param   exception_on_error
  */
-gdcmHeader::gdcmHeader(const char *InFilename, bool exception_on_error) {
+gdcmHeader::gdcmHeader(const char *InFilename, 
+                       bool exception_on_error,
+                      bool  enable_sequences ) {
+   if (enable_sequences)
+      enableSequences = 1;
+   else
+      enableSequences = 0;
+   
    SetMaxSizeLoadElementValue(_MaxSizeLoadElementValue_);
    filename = InFilename;
    Initialise();
@@ -122,7 +129,7 @@ gdcmHeader::~gdcmHeader (void) {
 
 // Fourth semantics:
 //
-// ---> Warning : This fourth fiels is NOT part 
+// ---> Warning : This fourth field is NOT part 
 //                of the 'official' Dicom Dictionnary
 //                and should NOT be used.
 //                (Not defined for all the groups
@@ -394,7 +401,7 @@ void gdcmHeader::FindVR( gdcmElValue *ElVal) {
    
    fseek(fp, PositionOnEntry, SEEK_SET);
    // When this element is known in the dictionary we shall use, e.g. for
-   // the semantics (see  the usage of IsAnInteger), the vr proposed by the
+   // the semantics (see the usage of IsAnInteger), the vr proposed by the
    // dictionary entry. Still we have to flag the element as implicit since
    // we know now our assumption on expliciteness is not furfilled.
    // avoid  .
@@ -508,7 +515,7 @@ bool gdcmHeader::IsJPEGLossless(void) {
    const char * Transfert = Element->GetValue().c_str();
    if ( memcmp(Transfert+strlen(Transfert)-2 ,"70",2)==0) return true;
    if ( memcmp(Transfert+strlen(Transfert)-2 ,"55",2)==0) return true;
-   if (Element->GetValue() == "1.2.840.10008.1.2.4.57") return true;
+   if (Element->GetValue() == "1.2.840.10008.1.2.4.57")   return true;
 
    return false;
 }
@@ -627,18 +634,49 @@ bool gdcmHeader::IsDicomV3(void) {
  *          applying this heuristic.
  */
 void gdcmHeader::FixFoundLength(gdcmElValue * ElVal, guint32 FoundLength) {
-   if ( FoundLength == 0xffffffff)
+
+   ElVal->SetReadLength(FoundLength); // will be updated only if a bug is found
+                    
+   if ( FoundLength == 0xffffffff) {  
       FoundLength = 0;
+   }
+      
       // Sorry for the patch!  
       // XMedCom did the trick to read some nasty GE images ...
-    if (FoundLength == 13)
+   else if (FoundLength == 13) {
       // The following 'if' will be removed when there is no more
-      // images on Creatis HD with a 13 length for Manufacturer...
-      if ( (ElVal->GetGroup() != 0x0008) || (ElVal->GetElement() ) )
+      // images on Creatis HDs with a 13 length for Manufacturer...
+      if ( (ElVal->GetGroup() != 0x0008) &&  
+           ( (ElVal->GetElement() != 0x0070) || (ElVal->GetElement() != 0x0080) ) ) {
       // end of remove area
          FoundLength =10;
-
-   ElVal->SetLength(FoundLength);
+        ElVal->SetReadLength(10); // a bug is to be fixed
+      }
+   } 
+     // to fix some garbage 'Leonardo' Siemens images
+     // May be commented out to avoid overhead
+   else if ( (ElVal->GetGroup() == 0x0009) 
+       &&
+       ( (ElVal->GetElement() == 0x1113) || (ElVal->GetElement() == 0x1114) ) ){
+        FoundLength =4;
+       ElVal->SetReadLength(4); // a bug is to be fixed 
+   } 
+     // end of fix
+        
+    // to try to 'go inside' SeQuences (with length), and not to skip them        
+    else if ( ElVal->GetVR() == "SQ") { 
+       if (enableSequences)    // only if the user does want to !
+         FoundLength =0;        
+    } 
+    
+    // a SeQuence Element is beginning
+    // Let's forget it's length
+    // (we want to 'go inside')
+    else if(ElVal->GetGroup() == 0xfffe){
+         FoundLength =0;            
+    }
+                 
+   ElVal->SetUsableLength(FoundLength);
 }
 
 /**
@@ -658,23 +696,19 @@ void gdcmHeader::FixFoundLength(gdcmElValue * ElVal, guint32 FoundLength) {
 
    while ( ! FoundSequenceDelimiter) {
       g = ReadInt16();
-      n = ReadInt16();
-      
-      long l = ftell(fp);
-
+      n = ReadInt16();   
       if (errno == 1)
          return 0;
       TotalLength += 4;  // We even have to decount the group and element 
      
-      if ( g != 0xfffe           && g!=0xb00c ) /*for bogus header */ {
+      if ( g != 0xfffe && g!=0xb00c ) /*for bogus header */ {
          char msg[100]; // for sprintf. Sorry
          sprintf(msg,"wrong group (%04x) for an item sequence (%04x,%04x)\n",g, g,n);
          dbg.Verbose(1, "gdcmHeader::FindLengthOB: ",msg); 
-         long l = ftell(fp);
          errno = 1;
          return 0;
       }
-      if ( n == 0xe0dd       || ( g==0xb00c && n==0x0eb6 ) ) /* for bogus header  */ 
+      if ( n == 0xe0dd || ( g==0xb00c && n==0x0eb6 ) ) /* for bogus header  */ 
          FoundSequenceDelimiter = true;
       else if ( n != 0xe000 ){
          char msg[100];  // for sprintf. Sorry
@@ -686,8 +720,7 @@ void gdcmHeader::FixFoundLength(gdcmElValue * ElVal, guint32 FoundLength) {
       }
       ItemLength = ReadInt32();
       TotalLength += ItemLength + 4;  // We add 4 bytes since we just read
-                                      // the ItemLength with ReadInt32
-                                      
+                                      // the ItemLength with ReadInt32                                     
       SkipBytes(ItemLength);
    }
    fseek(fp, PositionOnEntry, SEEK_SET);
@@ -725,7 +758,7 @@ void gdcmHeader::FixFoundLength(gdcmElValue * ElVal, guint32 FoundLength) {
             ElVal->SetLength(FindLengthOB());
             return;
          }
-         FixFoundLength(ElVal, length32);        
+         FixFoundLength(ElVal, length32); 
          return;
       }
 
@@ -749,7 +782,7 @@ void gdcmHeader::FixFoundLength(gdcmElValue * ElVal, guint32 FoundLength) {
       //   appear when we find the first group with big endian encoding. This
       //   is easy to detect since the length of a "Group Length" tag (the
       //   ones with zero as element number) has to be of 4 (0x0004). When we
-      //   encouter 1024 (0x0400) chances are the encoding changed and we
+      //   encounter 1024 (0x0400) chances are the encoding changed and we
       //   found a group with big endian encoding.
       // We shall use this second strategy. In order to make sure that we
       // can interpret the presence of an apparently big endian encoded
@@ -802,14 +835,14 @@ void gdcmHeader::FixFoundLength(gdcmElValue * ElVal, guint32 FoundLength) {
    // not coexist in a Data Set and Data Sets nested within it".]
    // Length is on 4 bytes.
    FixFoundLength(ElVal, ReadInt32());
+   return;
 }
 
 /**
  * \ingroup gdcmHeader
  * \brief   Swaps back the bytes of 4-byte long integer accordingly to
  *          processor order.
- *
- * @return  The suggested integer.
+ * @return  The properly swaped 32 bits integer.
  */
 guint32 gdcmHeader::SwapLong(guint32 a) {
    switch (sw) {
@@ -863,7 +896,7 @@ guint16 gdcmHeader::SwapShort(guint16 a) {
  * @return 
  */
  void gdcmHeader::SkipElementValue(gdcmElValue * ElVal) {
-   SkipBytes(ElVal->GetLength());
+    SkipBytes(ElVal->GetLength());
 }
 
 /**
@@ -904,17 +937,17 @@ void gdcmHeader::LoadElementValue(gdcmElValue * ElVal) {
    // car commencer par les ignorer risque de conduire a qq chose
    // qui pourrait ne pas etre generalisable
    // Well, I'm expecting your code !!!
+   
+   // the test was commented out to 'go inside' the SeQuences
+   // we don't any longer skip them !
     
-   if( vr == "SQ" )
-      SkipLoad = true;
+  // if( vr == "SQ" )  
+  //    SkipLoad = true;
 
-   // Heuristic : a sequence "contains" a set of tags (called items). It looks
-   // like the last tag of a sequence (the one that terminates the sequence)
-   // has a group of 0xfffe (with a dummy length).
-   // Well ... 
-   // Actually (fffe e000) tells us an Element is beginning
+   // A SeQuence "contains" a set of Elements.  
+   //          (fffe e000) tells us an Element is beginning
    //          (fffe e00d) tells us an Element just ended
-   //          (fffe e0dd) tells us the current SEQuence just ended
+   //          (fffe e0dd) tells us the current SeQuence just ended
   
    if( group == 0xfffe )
       SkipLoad = true;
@@ -939,6 +972,7 @@ void gdcmHeader::LoadElementValue(gdcmElValue * ElVal) {
       s << "gdcm::NotLoaded.";
       s << " Address:" << (long)ElVal->GetOffset();
       s << " Length:"  << ElVal->GetLength();
+      s << " x(" << std::hex << ElVal->GetLength() << ")";
       ElVal->SetValue(s.str());
       return;
    }
@@ -946,9 +980,9 @@ void gdcmHeader::LoadElementValue(gdcmElValue * ElVal) {
    // When an integer is expected, read and convert the following two or
    // four bytes properly i.e. as an integer as opposed to a string.
        
-       // pour les elements de Value Multiplicity > 1
-       // on aura en fait une serie d'entiers  
-       // on devrait pouvoir faire + compact (?)
+       // Actually, elements with Value Multiplicity > 1
+       // contain a set of integers (not a single one)         
+       // Any compacter code suggested (?)
                
    if ( IsAnInteger(ElVal) ) {
       guint32 NewInt;
@@ -977,7 +1011,8 @@ void gdcmHeader::LoadElementValue(gdcmElValue * ElVal) {
                s << NewInt;
             }
          }
-      }                                        
+      }        
+      s << '\0'; // to avoid oddities on Solaris
       ElVal->SetValue(s.str());
       return;  
    }
@@ -1115,7 +1150,6 @@ gdcmElValue* gdcmHeader::NewElValueByNumber(guint16 Group, guint16 Elem) {
  */
 int gdcmHeader::ReplaceOrCreateByNumber(std::string Value, 
                                         guint16 Group, guint16 Elem ) {
-
        // TODO : FIXME JPRx
        // curieux, non ?
        // on (je) cree une Elvalue ne contenant pas de valeur
@@ -1155,6 +1189,7 @@ int gdcmHeader::ReplaceOrCreateByNumber(char* Value, guint16 Group, guint16 Elem
 /**
  * \ingroup gdcmHeader
  * \brief   Set a new value if the invoked element exists
+ *          Seems to be useless !!!
  * @param   Value
  * @param   Group
  * @param   Elem
@@ -1162,7 +1197,7 @@ int gdcmHeader::ReplaceOrCreateByNumber(char* Value, guint16 Group, guint16 Elem
  */
 int gdcmHeader::ReplaceIfExistByNumber(char* Value, guint16 Group, guint16 Elem ) {
 
-   gdcmElValue* elValue = PubElValSet.GetElementByNumber(Group, Elem);
+   //gdcmElValue* elValue = PubElValSet.GetElementByNumber(Group, Elem);
    std::string v = Value;      
    PubElValSet.SetElValueByNumber(v, Group, Elem);
    return 1;
@@ -1225,6 +1260,7 @@ gdcmElValue * gdcmHeader::ReadNextElement(void) {
    NewElVal = NewElValueByNumber(g, n);
    FindVR(NewElVal);
    FindLength(NewElVal);
+       
    if (errno == 1) {
       // Call it quits
       return (gdcmElValue *)0;
@@ -1242,9 +1278,8 @@ gdcmElValue * gdcmHeader::ReadNextElement(void) {
  * @return  The result of the heuristical predicate.
  */
 bool gdcmHeader::IsAnInteger(gdcmElValue * ElVal) {
-   guint16 group   = ElVal->GetGroup();
    guint16 element = ElVal->GetElement();
-   std::string  vr      = ElVal->GetVR();
+   std::string  vr = ElVal->GetVR();
    guint32 length  = ElVal->GetLength();
 
    // When we have some semantics on the element we just read, and if we
@@ -1298,6 +1333,40 @@ size_t gdcmHeader::GetPixelOffset(void) {
       return 0;
 }
 
+/**
+ * \ingroup gdcmHeader
+ * \brief   Recover the pixel area length (in Bytes) .
+ */
+size_t gdcmHeader::GetPixelAreaLength(void) {
+   // If this file complies with the norm we should encounter the
+   // "Image Location" tag (0x0028,  0x0200). This tag contains the
+   // the group that contains the pixel data (hence the "Pixel Data"
+   // is found by indirection through the "Image Location").
+   // 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.
+   guint16 grPixel;
+   guint16 numPixel;
+   std::string ImageLocation = GetPubElValByName("Image Location");
+   if ( ImageLocation == GDCM_UNFOUND ) {
+      grPixel = 0x7fe0;
+   } else {
+      grPixel = (guint16) atoi( ImageLocation.c_str() );
+   }
+   if (grPixel != 0x7fe0)
+      // This is a kludge for old dirty Philips imager.
+      numPixel = 0x1010;
+   else
+      numPixel = 0x0010;
+         
+   gdcmElValue* PixelElement = PubElValSet.GetElementByNumber(grPixel,
+                                                              numPixel);
+   if (PixelElement)
+      return PixelElement->GetLength();
+   else
+      return 0;
+}
+
 /**
  * \ingroup gdcmHeader
  * \brief   Searches both the public and the shadow dictionary (when they
@@ -1393,7 +1462,7 @@ std::string gdcmHeader::GetPubElValRepByNumber(guint16 group, guint16 element) {
  * \ingroup gdcmHeader
  * \brief   Searches within the public dictionary for element value of
  *          a given tag.
- * @param   TagName name of the researched element.
+ * @param   TagName name of the searched element.
  * @return  Corresponding element value when it exists, and the string
  *          GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
@@ -1410,7 +1479,7 @@ std::string gdcmHeader::GetPubElValByName(std::string TagName) {
  *          to convert the string typed content to caller's native type 
  *          (think of C++ vs Python). The VR is actually of a higher level
  *          of semantics than just the native C++ type.
- * @param   TagName name of the researched element.
+ * @param   TagName name of the searched element.
  * @return  Corresponding element value representation when it exists,
  *          and the string GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
@@ -1425,8 +1494,8 @@ std::string gdcmHeader::GetPubElValRepByName(std::string TagName) {
  * \ingroup gdcmHeader
  * \brief   Searches within elements parsed with the SHADOW dictionary 
  *          for the element value of a given tag.
- * @param   group Group of the researched tag.
- * @param   element Element of the researched tag.
+ * @param   group Group of the searched tag.
+ * @param   element Element of the searched tag.
  * @return  Corresponding element value representation when it exists,
  *          and the string GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
@@ -1443,8 +1512,8 @@ std::string gdcmHeader::GetShaElValByNumber(guint16 group, guint16 element) {
  *          to convert the string typed content to caller's native type 
  *          (think of C++ vs Python). The VR is actually of a higher level
  *          of semantics than just the native C++ type.
- * @param   group Group of the researched tag.
- * @param   element Element of the researched tag.
+ * @param   group Group of the searched tag.
+ * @param   element Element of the searched tag.
  * @return  Corresponding element value representation when it exists,
  *          and the string GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
@@ -1459,7 +1528,7 @@ std::string gdcmHeader::GetShaElValRepByNumber(guint16 group, guint16 element) {
  * \ingroup gdcmHeader
  * \brief   Searches within the elements parsed with the shadow dictionary
  *          for an element value of given tag.
- * @param   TagName name of the researched element.
+ * @param   TagName name of the searched element.
  * @return  Corresponding element value when it exists, and the string
  *          GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
@@ -1476,7 +1545,7 @@ std::string gdcmHeader::GetShaElValByName(std::string TagName) {
  *          to convert the string typed content to caller's native type 
  *          (think of C++ vs Python). The VR is actually of a higher level
  *          of semantics than just the native C++ type.
- * @param   TagName name of the researched element.
+ * @param   TagName name of the searched element.
  * @return  Corresponding element value representation when it exists,
  *          and the string GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
@@ -1492,8 +1561,8 @@ std::string gdcmHeader::GetShaElValRepByName(std::string TagName) {
  * \brief   Searches within elements parsed with the public dictionary 
  *          and then within the elements parsed with the shadow dictionary
  *          for the element value of a given tag.
- * @param   group Group of the researched tag.
- * @param   element Element of the researched tag.
+ * @param   group Group of the searched tag.
+ * @param   element Element of the searched tag.
  * @return  Corresponding element value representation when it exists,
  *          and the string GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
@@ -1514,8 +1583,8 @@ std::string gdcmHeader::GetElValByNumber(guint16 group, guint16 element) {
  *          to convert the string typed content to caller's native type 
  *          (think of C++ vs Python). The VR is actually of a higher level
  *          of semantics than just the native C++ type.
- * @param   group Group of the researched tag.
- * @param   element Element of the researched tag.
+ * @param   group Group of the searched tag.
+ * @param   element Element of the searched tag.
  * @return  Corresponding element value representation when it exists,
  *          and the string GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
@@ -1531,7 +1600,7 @@ std::string gdcmHeader::GetElValRepByNumber(guint16 group, guint16 element) {
  * \brief   Searches within elements parsed with the public dictionary 
  *          and then within the elements parsed with the shadow dictionary
  *          for the element value of a given tag.
- * @param   TagName name of the researched element.
+ * @param   TagName name of the searched element.
  * @return  Corresponding element value when it exists,
  *          and the string GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
@@ -1552,7 +1621,7 @@ std::string gdcmHeader::GetElValByName(std::string TagName) {
  *          to convert the string typed content to caller's native type 
  *          (think of C++ vs Python). The VR is actually of a higher level
  *          of semantics than just the native C++ type.
- * @param   TagName name of the researched element.
+ * @param   TagName name of the searched element.
  * @return  Corresponding element value representation when it exists,
  *          and the string GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
@@ -1646,7 +1715,7 @@ void gdcmHeader::ParseHeader(bool exception_on_error) throw(gdcmFormatError) {
    
    rewind(fp);
    CheckSwap();
-   while ( (newElValue = ReadNextElement()) ) {
+   while ( (newElValue = ReadNextElement()) ) { 
       SkipElementValue(newElValue);
       PubElValSet.Add(newElValue);
    }
@@ -1654,7 +1723,7 @@ void gdcmHeader::ParseHeader(bool exception_on_error) throw(gdcmFormatError) {
 
 /**
  * \ingroup gdcmHeader
- * \brief  This predicate, based on hopefully reasonnable heuristics,
+ * \brief  This predicate, based on hopefully reasonable heuristics,
  *         decides whether or not the current gdcmHeader was properly parsed
  *         and contains the mandatory information for being considered as
  *         a well formed and usable image.
@@ -1713,11 +1782,22 @@ gdcmElValue* gdcmHeader::NewManualElValToPubDict(std::string NewTagName,
  *          public tag based hash table.
  */
 void gdcmHeader::LoadElements(void) {
-   rewind(fp);   
-   TagElValueHT ht = PubElValSet.GetTagHt();
-   for (TagElValueHT::iterator tag = ht.begin(); tag != ht.end(); ++tag) {
-      LoadElementValue(tag->second);
-   }
+   rewind(fp);
+   
+   // We don't use any longer the HashTable, since a lot a stuff is missing
+   // when SeQuences were encountered 
+   //  
+   //TagElValueHT ht = PubElValSet.GetTagHt();
+   //for (TagElValueHT::iterator tag = ht.begin(); tag != ht.end(); ++tag) {
+   //     LoadElementValue(tag->second);
+   //}
+   
+     for (ListTag::iterator i = GetListElem().begin();  
+          i != GetListElem().end();
+          ++i){
+        LoadElementValue(*i);   
+     }    
+    
    rewind(fp);
 
    // Load 'non string' values   
@@ -1763,7 +1843,7 @@ void gdcmHeader::PrintPubElVal(std::ostream & os) {
 
 /**
   * \ingroup gdcmHeader
-  * \brief
+  * \brief 
   * @return
   */  
 void gdcmHeader::PrintPubDict(std::ostream & os) {
@@ -1773,12 +1853,11 @@ void gdcmHeader::PrintPubDict(std::ostream & os) {
 /**
   * \ingroup gdcmHeader
   * \brief
-  * @return
+  * @return integer, acts as a Boolean
   */ 
 int gdcmHeader::Write(FILE * fp, FileType type) {
 
-
-   // TODO : move the following lines (and a lot of others)
+   // TODO : move the following lines (and a lot of others, to be written)
    // to a future function CheckAndCorrectHeader
 
    if (type == ImplicitVR) {
@@ -1828,7 +1907,7 @@ void * gdcmHeader::LoadElementVoidArea(guint16 Group, guint16 Elem) {
                  << std::hex << Group << " " << Elem << std::endl;
        return NULL;
    }  
-   int res = PubElValSet.SetVoidAreaByNumber(a, Group, Elem);
+   /* int res = */ PubElValSet.SetVoidAreaByNumber(a, Group, Elem);
    // TODO check the result 
    size_t l2 = fread(a, 1, l ,fp);
    if(l != l2) {
@@ -1836,7 +1915,8 @@ void * gdcmHeader::LoadElementVoidArea(guint16 Group, guint16 Elem) {
                  << std::hex << Group << " " << Elem << std::endl;
        free(a);
        return NULL;
-   }  
+   }
+   return a;  
 }
 
 /**
@@ -1959,11 +2039,24 @@ int gdcmHeader::GetBitsStored(void) {
    return atoi(StrSize.c_str());
 }
 
+/**
+ * \ingroup gdcmHeader
+ * \brief   Retrieve the number of Bits Allocated
+ *          (8, 12 -compacted ACR-NEMA files, 16, ...)
+ * 
+ * @return  The encountered number of Bits Allocated, 0 by default.
+ */
+int gdcmHeader::GetBitsAllocated(void) { 
+   std::string StrSize = GetPubElValByNumber(0x0028,0x0100);
+   if (StrSize == GDCM_UNFOUND)
+      return 1;
+   return atoi(StrSize.c_str());
+}
 
 /**
  * \ingroup gdcmHeader
  * \brief   Retrieve the number of Samples Per Pixel
- *          (1 : gray level, 3 : RGB)
+ *          (1 : gray level, 3 : RGB -1 or 3 Planes-)
  * 
  * @return  The encountered number of Samples Per Pixel, 1 by default.
  */
@@ -2017,6 +2110,7 @@ int gdcmHeader::GetPixelSize(void) {
  *          - 32U unsigned 32 bit,
  *          - 32S   signed 32 bit,
  * \warning 12 bit images appear as 16 bit.
+ * \        24 bit images appear as 8 bit
  * @return  
  */
 std::string gdcmHeader::GetPixelType(void) {
@@ -2026,9 +2120,11 @@ std::string gdcmHeader::GetPixelType(void) {
       dbg.Verbose(0, "gdcmHeader::GetPixelType: unfound Bits Allocated");
       BitsAlloc = std::string("16");
    }
-   if (BitsAlloc == "12")
+   if (BitsAlloc == "12")            // It will be unpacked
       BitsAlloc = std::string("16");
-
+   else if (BitsAlloc == "24")       // (in order no to be messed up
+      BitsAlloc = std::string("8");  // by old RGB images)
+     
    std::string Signed;
    Signed = GetElValByName("Pixel Representation");
    if (Signed == GDCM_UNFOUND) {
@@ -2038,6 +2134,8 @@ std::string gdcmHeader::GetPixelType(void) {
    if (Signed == "0")
       Signed = std::string("U");
    else
+
+std::cout << "GetPixelType : " << BitsAlloc + Signed << std::endl;
       Signed = std::string("S");
 
    return( BitsAlloc + Signed);
@@ -2066,39 +2164,33 @@ std::string gdcmHeader::GetTransferSyntaxName(void) {
 
 /**
   * \ingroup gdcmHeader
-  * \brief gets the info from 0028,1101 : Lookup Table Desc-Red
-  * \           else 0
-  * @return Lookup Table Length 
-  * \       when (0028,0004),Photometric Interpretation = [PALETTE COLOR ] 
+  * \brief tells us if LUT are used
+  * \warning Right now, Segmented xxx Palette Color Lookup Table Data
+  * \        are NOT considered as LUT, since nobody knows
+  *\         how to deal with them
+  * @return int acts as a Boolean 
   */
   
-int gdcmHeader::GetLUTLength(void) {
-   std::vector<std::string> tokens;
-   int LutLength;
-   //int LutDepth;
-   //int LutNbits;
-   // Just hope Lookup Table Desc-Red = Lookup Table Desc-Red = Lookup Table Desc-Blue
-   std::string LutDescriptionR = GetPubElValByNumber(0x0028,0x1101);
-   if (LutDescriptionR == GDCM_UNFOUND)
+int gdcmHeader::HasLUT(void) {
+
+   // Check the presence of the LUT Descriptors 
+   if (GetPubElValByNumber(0x0028,0x1101) == GDCM_UNFOUND)
       return 0;
-   std::string LutDescriptionG = GetPubElValByNumber(0x0028,0x1102);
-   if (LutDescriptionG == GDCM_UNFOUND)
+   // LutDescriptorGreen 
+   if (GetPubElValByNumber(0x0028,0x1102) == GDCM_UNFOUND)
       return 0;
-   std::string LutDescriptionB = GetPubElValByNumber(0x0028,0x1103);
-   if (LutDescriptionB == GDCM_UNFOUND)
+   // LutDescriptorBlue 
+   if (GetPubElValByNumber(0x0028,0x1103) == GDCM_UNFOUND)
       return 0;
-   if( (LutDescriptionR != LutDescriptionG) || (LutDescriptionR != LutDescriptionB) ) {
-      dbg.Verbose(0, "gdcmHeader::GetLUTLength: The CLUT R,G,B are not equal");
+   //  It is not enough
+   // we check also 
+   if (GetPubElValByNumber(0x0028,0x1201) == GDCM_UNFOUND)
+      return 0;  
+   if (GetPubElValByNumber(0x0028,0x1202) == GDCM_UNFOUND)
+      return 0;
+   if (GetPubElValByNumber(0x0028,0x1203) == GDCM_UNFOUND)
       return 0;   
-   } 
-   std::cout << "Lut Description " << LutDescriptionR <<std::endl;
-   tokens.erase(tokens.begin(),tokens.end()); // clean any previous value
-   Tokenize (LutDescriptionR, tokens, "\\");
-   LutLength=atoi(tokens[0].c_str());
-   //LutDepth=atoi(tokens[1].c_str());
-   //LutNbits=atoi(tokens[2].c_str());
-   tokens.clear();
-   return LutLength;
+   return 1;
 }
 
 /**
@@ -2127,161 +2219,144 @@ int gdcmHeader::GetLUTNbits(void) {
    tokens.clear();
    return LutNbits;
 }
-  
-
-/**
-  * \ingroup gdcmHeader
-  * \brief gets the info from 0028,1201 : Lookup Table Red
-  * \           else 0
-  * @return Lookup Table Red 
-  * \       when (0028,0004),Photometric Interpretation = [PALETTE COLOR ] 
-  */ 
-void * gdcmHeader::GetLUTRed(void) {
-   return GetPubElValVoidAreaByNumber(0x0028,0x1201);  
-}
-
-/**
-  * \ingroup gdcmHeader
-  * \brief gets the info from 0028,1202 : Lookup Table Green
-  * \           else 0
-  * @return Lookup Table Red 
-  * \       when (0028,0004),Photometric Interpretation = [PALETTE COLOR ] 
-  */ 
-  void * gdcmHeader::GetLUTGreen(void) {
-   return GetPubElValVoidAreaByNumber(0x0028,0x1202);
-}
-
-/**
-  * \ingroup gdcmHeader
-  * \brief gets the info from 0028,1202 : Lookup Table Blue
-  * \           else 0
-  * @return Lookup Table Blue 
-  * \       when (0028,0004),Photometric Interpretation = [PALETTE COLOR ] 
-  */ 
-void * gdcmHeader::GetLUTBlue(void) {
-   return GetPubElValVoidAreaByNumber(0x0028,0x1203);
-}
 
 /**
   * \ingroup gdcmHeader
-  * \brief 
-  * @return Lookup Table RGB
+  * \brief builts Red/Green/Blue/Alpha LUT from Header
   * \       when (0028,0004),Photometric Interpretation = [PALETTE COLOR ]
-  * \        and (0028,1201),(0028,1202),(0028,1202) are found
-  * \warning : hazardous ! Use better GetPubElValVoidAreaByNumber
+  * \        and (0028,1101),(0028,1102),(0028,1102)  
+  * \          - xxx Palette Color Lookup Table Descriptor - are found
+  * \        and (0028,1201),(0028,1202),(0028,1202) 
+  * \          - xxx Palette Color Lookup Table Data - are found 
+  * \warning does NOT deal with :
+  * \ 0028 1100 Gray Lookup Table Descriptor (Retired)
+  * \ 0028 1221 Segmented Red Palette Color Lookup Table Data
+  * \ 0028 1222 Segmented Green Palette Color Lookup Table Data
+  * \ 0028 1223 Segmented Blue Palette Color Lookup Table Data 
+  * \ no known Dicom reader deails with them :-(
+  * @return Lookup Table RGBA
   */ 
-void * gdcmHeader::GetLUTRGB(void) {
+  
+unsigned char * gdcmHeader::GetLUTRGBA(void) {
 // Not so easy : see 
 // http://www.barre.nom.fr/medical/dicom2/limitations.html#Color%20Lookup%20Tables
 // and  OT-PAL-8-face.dcm
 
-   if (GetPubElValByNumber(0x0028,0x0004) == GDCM_UNFOUND) {
-   dbg.Verbose(0, "gdcmHeader::GetLUTRGB: unfound Photometric Interpretation");
+//  if Photometric Interpretation # PALETTE COLOR, no LUT to be done
+                
+   if (gdcmHeader::GetPubElValByNumber(0x0028,0x0004) != "PALETTE COLOR ") {
        return NULL;
    }  
-   void * LutR,*LutG,*LutB;
-   int l;
-     
-  // Maybe, some day we get an image 
-  // that respects the definition ...
-  // Let's consider no ones does.
-  
-   l= GetLUTLength();  
-   if(l==0) 
-     return (NULL);     
-   int nBits=GetLUTNbits();
-
-  // a virer quand on aura trouve UNE image 
-  // qui correspond VRAIMENT à la definition !
-    std::cout << "l " << l << " nBits " << nBits;
+
+   int lengthR, debR, nbitsR;
+   int lengthG, debG, nbitsG;
+   int lengthB, debB, nbitsB;
    
-   l= l/(nBits/8);
-    
-   LutR =GetPubElValVoidAreaByNumber(0x0028,0x1201);
-   LutG =GetPubElValVoidAreaByNumber(0x0028,0x1202);
-   LutB =GetPubElValVoidAreaByNumber(0x0028,0x1203);
+// Get info from Lut Descriptors
+// (the 3 LUT descriptors may be different)    
    
-   // Warning : Any value for nBits  as to be considered as 8
-   //           Any value for Length as to be considered as 256
-   // That's DICOM ...
+   std::string LutDescriptionR = GetPubElValByNumber(0x0028,0x1101);
+   if (LutDescriptionR == GDCM_UNFOUND)
+      return NULL;
+   std::string LutDescriptionG = GetPubElValByNumber(0x0028,0x1102);
+   if (LutDescriptionG == GDCM_UNFOUND)
+      return NULL;   
+   std::string LutDescriptionB = GetPubElValByNumber(0x0028,0x1103);
+   if (LutDescriptionB == GDCM_UNFOUND)
+      return NULL;
+      
+   std::vector<std::string> tokens;
+      
+   tokens.erase(tokens.begin(),tokens.end()); // clean any previous value
+   Tokenize (LutDescriptionR, tokens, "\\");
+   lengthR=atoi(tokens[0].c_str()); // Red LUT length in Bytes
+   debR   =atoi(tokens[1].c_str()); // subscript of the first Lut Value
+   nbitsR =atoi(tokens[2].c_str()); // Lut item size (in Bits)
+   tokens.clear();
    
-   // Just wait before removing the following code
-   /*
-   if (nBits == 16) {
-      guint16 * LUTRGB, *rgb;
-      LUTRGB = rgb = (guint16 *) malloc(3*l*sizeof( guint16));
-      guint16 * r = (guint16 *)LutR;
-      guint16 * g = (guint16 *)LutG;
-      guint16 * b = (guint16 *)LutB;
-      for(int i=0;i<l;i++) {
-         *rgb++ = *r++;
-         *rgb++ = *g++;
-         *rgb++ = *b++;
-      }
-      return(LUTRGB); 
-   } else
+   tokens.erase(tokens.begin(),tokens.end()); // clean any previous value
+   Tokenize (LutDescriptionG, tokens, "\\");
+   lengthG=atoi(tokens[0].c_str()); // Green LUT length in Bytes
+   debG   =atoi(tokens[1].c_str());
+   nbitsG =atoi(tokens[2].c_str());
+   tokens.clear();  
+   
+   tokens.erase(tokens.begin(),tokens.end()); // clean any previous value
+   Tokenize (LutDescriptionB, tokens, "\\");
+   lengthB=atoi(tokens[0].c_str()); // Blue LUT length in Bytes
+   debB   =atoi(tokens[1].c_str());
+   nbitsB =atoi(tokens[2].c_str());
+   tokens.clear();
+// Load LUTs into memory, (as they were stored on disk)
    
-   */ {      // we assume it's always 8 Bits
-      l=256; // we assume ...
-      unsigned char * LUTRGB, *rgb;
-      LUTRGB = rgb = (unsigned char *) malloc(3*l*sizeof( char));
-      unsigned char * r = (unsigned char *)LutR;
-      unsigned char * g = (unsigned char *)LutG;
-      unsigned char * b = (unsigned char *)LutB;
-      for(int i=0;i<l;i++) {
-      //std::cout << "lut16 " << i << " : " << *r << " " << *g << " " << *b 
-      //          << std::endl;
-      printf("lut 8 %d : %d %d %d \n",i,*r,*g,*b);
-         *rgb++ = *r++;
-         *rgb++ = *g++;
-         *rgb++ = *b++;
-      } 
-      free(LutR); free(LutB); free(LutG);
-      return(LUTRGB);   
+   unsigned char *lutR =(unsigned char *)
+                                   GetPubElValVoidAreaByNumber(0x0028,0x1201);
+   unsigned char *lutG =(unsigned char *)
+                                   GetPubElValVoidAreaByNumber(0x0028,0x1202);
+   unsigned char *lutB =(unsigned char *)
+                                   GetPubElValVoidAreaByNumber(0x0028,0x1203); 
+   
+   if (!lutR || !lutG || !lutB ) {
+       return NULL;
    } 
+ // forge the 4 * 8 Bits Red/Green/Blue/Alpha LUT 
+   
+  unsigned char *LUTRGBA = (unsigned char *)calloc(1024,1); // 256 * 4 (R, G, B, Alpha) 
+  if (!LUTRGBA) {
+     return NULL;
+  }
+  memset(LUTRGBA, 0, 1024);
+       // Bits Allocated
+   int nb;
+   std::string str_nb = GetPubElValByNumber(0x0028,0x0100);
+   if (str_nb == GDCM_UNFOUND ) {
+      nb = 16;
+   } else {
+      nb = atoi(str_nb.c_str() );
+   }  
+  int mult;
+  
+  if (nbitsR==16 && nb==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; 
+            // 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 overhead 
+           
+  unsigned char *a;      
+  int i;
+  a= LUTRGBA+0;
+  for(i=0;i<lengthR;i++) {
+     *a = lutR[i*mult+1]; 
+     a+=4;       
+  }        
+  a= LUTRGBA+1;
+  for(i=0;i<lengthG;i++) {
+     *a = lutG[i*mult+1]; 
+     a+=4;       
+  }  
+  a= LUTRGBA+2;
+  for(i=0;i<lengthB;i++) {
+     *a = lutB[i*mult+1]; 
+     a+=4;       
+  }  
+  a= LUTRGBA+3;
+  for(i=0;i<256;i++) {
+     *a = 1; // Alpha component
+     a+=4; 
+  } 
+      
+//How to free the now useless LUTs?
+//
+//free(LutR); free(LutB); free(LutG);
 
-/*  Sorry for the comments. The code will be moved in a fonction
-
-        std::string x=GetPubElValByNumber(0x0028,0x1201);
-        unsigned short int * lutR = (unsigned short int *)malloc((size_t)200*sizeof(short int));
-        unsigned short int * lutG = (unsigned short int *)malloc((size_t)200*sizeof(short int));
-        unsigned short int * lutB = (unsigned short int *)malloc((size_t)200*sizeof(short int));
-
-         std::vector<std::string> tokens;
-         tokens.erase(tokens.begin(),tokens.end()); // clean any previous value
-         Tokenize ((const std::string)x, tokens, "\\");
-         for (unsigned int i=0; i<tokens.size();i++) {
-            lutR[i] = atoi(tokens[i].c_str());
-            printf("%d (%x)\n",lutR[i],lutR[i]);   
-         }
-
-         std::string y=GetPubElValByNumber(0x0028,0x1202);
-         tokens.erase(tokens.begin(),tokens.end()); // clean any previous value
-         Tokenize ((const std::string)y, tokens, "\\");
-         for (unsigned int i=0; i<tokens.size();i++) {
-            lutG[i] = atoi(tokens[i].c_str());   
-         }
-
-         std::string z=GetPubElValByNumber(0x0028,0x1203);
-         tokens.erase(tokens.begin(),tokens.end()); // clean any previous value
-         Tokenize ((const std::string)z, tokens, "\\");
-         for (unsigned int i=0; i<tokens.size();i++) {
-            lutB[i] = atoi(tokens[i].c_str());   
-         }
-         tokens.erase(tokens.begin(),tokens.end()); // clean any previous value
-         Tokenize ((const std::string)x, tokens, "\\");
-         for (unsigned int i=0; i<tokens.size();i++) {
-            lutB[i] = atoi(tokens[i].c_str()); 
-         }
-
- */
-
-//int lgth=GetLUTLength();
-//cout << "lgth " << lgth << std::endl;;
-//for (int j=0;j<lgth;j++){
-//printf ("%d : %d (%x) %d (%x) %d (%x)\n",j,lutR[j],lutR[j],lutG[j],lutG[j],lutB[j],lutB[j]);
-//}
-
-
-}
+  return(LUTRGBA);   
+}