]> Creatis software - gdcm.git/blobdiff - src/gdcmHeader.cxx
Remaining \n in std::cout and std::cerr replaced by std::endl
[gdcm.git] / src / gdcmHeader.cxx
index c03e3419df4b815d2302646f4ed56f5673c29c70..52ff002cd456d0f2f315edc2c21a15ea7aea7243 100644 (file)
@@ -1,4 +1,16 @@
-// $Header: /cvs/public/gdcm/src/Attic/gdcmHeader.cxx,v 1.73 2003/07/01 10:04:36 frog Exp $
+// $Header: /cvs/public/gdcm/src/Attic/gdcmHeader.cxx,v 1.86 2003/09/24 13:00:59 jpr Exp $
+
+//This is needed when compiling in debug mode
+#ifdef _MSC_VER
+//'identifier' : not all control paths return a value
+//#pragma warning ( disable : 4715 )
+// 'identifier' : class 'type' needs to have dll-interface to be used by
+// clients of class 'type2'
+#pragma warning ( disable : 4251 )
+// 'identifier' : identifier was truncated to 'number' characters in the
+// debug information
+#pragma warning ( disable : 4786 )
+#endif //_MSC_VER
 
 #include <stdio.h>
 #include <cerrno>
 #include <sstream>
 #include "gdcmUtil.h"
 #include "gdcmHeader.h"
-using namespace std;
-
-
-// TODO : remove DEBUG
-#define DEBUG 0
+#include "gdcmTS.h"
 
 // Refer to gdcmHeader::CheckSwap()
 #define HEADER_LENGTH_TO_READ       256
 // Refer to gdcmHeader::SetMaxSizeLoadElementValue()
-#define _MaxSizeLoadElementValue_   1024
-
+//#define _MaxSizeLoadElementValue_   1024
+#define _MaxSizeLoadElementValue_   4096
 /**
  * \ingroup gdcmHeader
  * \brief   
@@ -41,15 +49,15 @@ void gdcmHeader::Initialise(void) {
  * @param   InFilename
  * @param   exception_on_error
  */
- gdcmHeader::gdcmHeader(const char *InFilename, bool exception_on_error) {
-  SetMaxSizeLoadElementValue(_MaxSizeLoadElementValue_);
-  filename = InFilename;
-  Initialise();
-  if ( !OpenFile(exception_on_error))
-     return;
-  ParseHeader();
-  LoadElements();
-  CloseFile();
+gdcmHeader::gdcmHeader(const char *InFilename, bool exception_on_error) {
+   SetMaxSizeLoadElementValue(_MaxSizeLoadElementValue_);
+   filename = InFilename;
+   Initialise();
+   if ( !OpenFile(exception_on_error))
+      return;
+   ParseHeader();
+   LoadElements();
+   CloseFile();
 }
 
 /**
@@ -84,7 +92,7 @@ gdcmHeader::gdcmHeader(bool exception_on_error) {
 /**
  * \ingroup gdcmHeader
  * \brief   
- * @return  
+ * @return  TRUE if the close was successfull 
  */
 bool gdcmHeader::CloseFile(void) {
   int closed = fclose(fp);
@@ -99,11 +107,11 @@ bool gdcmHeader::CloseFile(void) {
  * \brief   Canonical destructor.
  */
 gdcmHeader::~gdcmHeader (void) {
-   dicom_vr =   (gdcmVR*)0; 
-   Dicts    =   (gdcmDictSet*)0;
-   RefPubDict = (gdcmDict*)0;
-   RefShaDict = (gdcmDict*)0;
-   return;
+  dicom_vr =   (gdcmVR*)0; 
+  Dicts    =   (gdcmDictSet*)0;
+  RefPubDict = (gdcmDict*)0;
+  RefShaDict = (gdcmDict*)0;
+  return;
 }
 
 // Fourth semantics:
@@ -136,6 +144,7 @@ gdcmHeader::~gdcmHeader (void) {
 // CRV      Curve
 // OLY      Overlays
 // PXL      Pixels
+// DL       Delimiters
 //
 
 /**
@@ -151,7 +160,7 @@ void gdcmHeader::CheckSwap()
    // 0x00000004. Finding the swap code in then straigthforward. Trouble
    // occurs when we can't find such group...
    guint32  s;
-   guint32  x=4;  // x : pour ntohs
+   guint32  x=4;  // x : for ntohs
    bool net2host; // true when HostByteOrder is the same as NetworkByteOrder
     
    int lgrLue;
@@ -192,9 +201,9 @@ void gdcmHeader::CheckSwap()
       // Use gdcmHeader::dicom_vr to test all the possibilities
       // instead of just checking for UL, OB and UI !?
       if(  (memcmp(entCur, "UL", (size_t)2) == 0) ||
-         (memcmp(entCur, "OB", (size_t)2) == 0) ||
-         (memcmp(entCur, "UI", (size_t)2) == 0) )
-       {
+          (memcmp(entCur, "OB", (size_t)2) == 0) ||
+          (memcmp(entCur, "UI", (size_t)2) == 0) )   
+      {
          filetype = ExplicitVR;
          dbg.Verbose(1, "gdcmHeader::CheckSwap:",
                      "explicit Value Representation");
@@ -203,7 +212,6 @@ void gdcmHeader::CheckSwap()
          dbg.Verbose(1, "gdcmHeader::CheckSwap:",
                      "not an explicit Value Representation");
       }
-
       if (net2host) {
          sw = 4321;
          dbg.Verbose(1, "gdcmHeader::CheckSwap:",
@@ -298,13 +306,14 @@ void gdcmHeader::SwitchSwapToBigEndian(void) {
 /**
  * \ingroup   gdcmHeader
  * \brief     Find the value representation of the current tag.
+ * @param ElVal
  */
 void gdcmHeader::FindVR( gdcmElValue *ElVal) {
    if (filetype != ExplicitVR)
       return;
 
    char VR[3];
-   string vr;
+   std::string vr;
    int lgrLue;
    char msg[100]; // for sprintf. Sorry
 
@@ -320,7 +329,7 @@ void gdcmHeader::FindVR( gdcmElValue *ElVal) {
    
    lgrLue=fread (&VR, (size_t)2,(size_t)1, fp);
    VR[2]=0;
-   vr = string(VR);
+   vr = std::string(VR);
       
    // Assume we are reading a falsely explicit VR file i.e. we reached
    // a tag where we expect reading a VR but are in fact we read the
@@ -382,7 +391,7 @@ void gdcmHeader::FindVR( gdcmElValue *ElVal) {
 
 /**
  * \ingroup gdcmHeader
- * \brief   Determines if the Transfer Syntax was allready encountered
+ * \brief   Determines if the Transfer Syntax was already encountered
  *          and if it corresponds to a ImplicitVRLittleEndian one.
  *
  * @return  True when ImplicitVRLittleEndian found. False in all other cases.
@@ -392,7 +401,7 @@ bool gdcmHeader::IsImplicitVRLittleEndianTransferSyntax(void) {
    if ( !Element )
       return false;
    LoadElementValueSafe(Element);
-   string Transfer = Element->GetValue();
+   std::string Transfer = Element->GetValue();
    if ( Transfer == "1.2.840.10008.1.2" )
       return true;
    return false;
@@ -400,7 +409,7 @@ bool gdcmHeader::IsImplicitVRLittleEndianTransferSyntax(void) {
 
 /**
  * \ingroup gdcmHeader
- * \brief   Determines if the Transfer Syntax was allready encountered
+ * \brief   Determines if the Transfer Syntax was already encountered
  *          and if it corresponds to a ExplicitVRLittleEndian one.
  *
  * @return  True when ExplicitVRLittleEndian found. False in all other cases.
@@ -410,7 +419,7 @@ bool gdcmHeader::IsExplicitVRLittleEndianTransferSyntax(void) {
    if ( !Element )
       return false;
    LoadElementValueSafe(Element);
-   string Transfer = Element->GetValue();
+   std::string Transfer = Element->GetValue();
    if ( Transfer == "1.2.840.10008.1.2.1" )
       return true;
    return false;
@@ -418,7 +427,7 @@ bool gdcmHeader::IsExplicitVRLittleEndianTransferSyntax(void) {
 
 /**
  * \ingroup gdcmHeader
- * \brief   Determines if the Transfer Syntax was allready encountered
+ * \brief   Determines if the Transfer Syntax was already encountered
  *          and if it corresponds to a DeflatedExplicitVRLittleEndian one.
  *
  * @return  True when DeflatedExplicitVRLittleEndian found. False in all other cases.
@@ -428,7 +437,7 @@ bool gdcmHeader::IsDeflatedExplicitVRLittleEndianTransferSyntax(void) {
    if ( !Element )
       return false;
    LoadElementValueSafe(Element);
-   string Transfer = Element->GetValue();
+   std::string Transfer = Element->GetValue();
    if ( Transfer == "1.2.840.10008.1.2.1.99" )
       return true;
    return false;
@@ -436,7 +445,7 @@ bool gdcmHeader::IsDeflatedExplicitVRLittleEndianTransferSyntax(void) {
 
 /**
  * \ingroup gdcmHeader
- * \brief   Determines if the Transfer Syntax was allready encountered
+ * \brief   Determines if the Transfer Syntax was already encountered
  *          and if it corresponds to a Explicit VR Big Endian one.
  *
  * @return  True when big endian found. False in all other cases.
@@ -446,7 +455,7 @@ bool gdcmHeader::IsExplicitVRBigEndianTransferSyntax(void) {
    if ( !Element )
       return false;
    LoadElementValueSafe(Element);
-   string Transfer = Element->GetValue();
+   std::string Transfer = Element->GetValue();
    if ( Transfer == "1.2.840.10008.1.2.2" )  //1.2.2 ??? A verifier !
       return true;
    return false;
@@ -454,7 +463,7 @@ bool gdcmHeader::IsExplicitVRBigEndianTransferSyntax(void) {
 
 /**
  * \ingroup gdcmHeader
- * \brief   Determines if the Transfer Syntax was allready encountered
+ * \brief   Determines if the Transfer Syntax was already encountered
  *          and if it corresponds to a JPEGBaseLineProcess1 one.
  *
  * @return  True when JPEGBaseLineProcess1found. False in all other cases.
@@ -464,7 +473,7 @@ bool gdcmHeader::IsJPEGBaseLineProcess1TransferSyntax(void) {
    if ( !Element )
       return false;
    LoadElementValueSafe(Element);
-   string Transfer = Element->GetValue();
+   std::string Transfer = Element->GetValue();
    if ( Transfer == "1.2.840.10008.1.2.4.50" )
       return true;
    return false;
@@ -491,7 +500,7 @@ bool gdcmHeader::IsJPEGLossless(void) {
 
 /**
  * \ingroup gdcmHeader
- * \brief   Determines if the Transfer Syntax was allready encountered
+ * \brief   Determines if the Transfer Syntax was already encountered
  *          and if it corresponds to a JPEGExtendedProcess2-4 one.
  *
  * @return  True when JPEGExtendedProcess2-4 found. False in all other cases.
@@ -501,7 +510,7 @@ bool gdcmHeader::IsJPEGExtendedProcess2_4TransferSyntax(void) {
    if ( !Element )
       return false;
    LoadElementValueSafe(Element);
-   string Transfer = Element->GetValue();
+   std::string Transfer = Element->GetValue();
    if ( Transfer == "1.2.840.10008.1.2.4.51" )
       return true;
    return false;
@@ -509,7 +518,7 @@ bool gdcmHeader::IsJPEGExtendedProcess2_4TransferSyntax(void) {
 
 /**
  * \ingroup gdcmHeader
- * \brief   Determines if the Transfer Syntax was allready encountered
+ * \brief   Determines if the Transfer Syntax was already encountered
  *          and if it corresponds to a JPEGExtendeProcess3-5 one.
  *
  * @return  True when JPEGExtendedProcess3-5 found. False in all other cases.
@@ -519,7 +528,7 @@ bool gdcmHeader::IsJPEGExtendedProcess3_5TransferSyntax(void) {
    if ( !Element )
       return false;
    LoadElementValueSafe(Element);
-   string Transfer = Element->GetValue();
+   std::string Transfer = Element->GetValue();
    if ( Transfer == "1.2.840.10008.1.2.4.52" )
       return true;
    return false;
@@ -527,7 +536,7 @@ bool gdcmHeader::IsJPEGExtendedProcess3_5TransferSyntax(void) {
 
 /**
  * \ingroup gdcmHeader
- * \brief   Determines if the Transfer Syntax was allready encountered
+ * \brief   Determines if the Transfer Syntax was already encountered
  *          and if it corresponds to a JPEGSpectralSelectionProcess6-8 one.
  *
  * @return  True when JPEGSpectralSelectionProcess6-8 found. False in all
@@ -538,12 +547,51 @@ bool gdcmHeader::IsJPEGSpectralSelectionProcess6_8TransferSyntax(void) {
    if ( !Element )
       return false;
    LoadElementValueSafe(Element);
-   string Transfer = Element->GetValue();
+   std::string Transfer = Element->GetValue();
    if ( Transfer == "1.2.840.10008.1.2.4.53" )
       return true;
    return false;
 }
 
+/**
+ * \ingroup gdcmHeader
+ * \brief   Determines if the Transfer Syntax was already encountered
+ *          and if it corresponds to a RLE Lossless one.
+ *
+ * @return  True when RLE Lossless found. False in all
+ *          other cases.
+ */
+bool gdcmHeader::IsRLELossLessTransferSyntax(void) {
+   gdcmElValue* Element = PubElValSet.GetElementByNumber(0x0002, 0x0010);
+   if ( !Element )
+      return false;
+   LoadElementValueSafe(Element);
+   std::string Transfer = Element->GetValue();
+   if ( Transfer == "1.2.840.10008.1.2.5" )
+      return true;
+   return false;
+}
+
+/**
+ * \ingroup gdcmHeader
+ * \brief   Determines if the Transfer Syntax was already encountered
+ *          and if it corresponds to a JPEG200 one.0
+ *
+ * @return  True when JPEG2000 (Lossly or LossLess) found. False in all
+ *          other cases.
+ */
+bool gdcmHeader::IsJPEG2000(void) {
+   gdcmElValue* Element = PubElValSet.GetElementByNumber(0x0002, 0x0010);
+   if ( !Element )
+      return false;
+   LoadElementValueSafe(Element);
+   std::string Transfer = Element->GetValue();
+   if (    (Transfer == "1.2.840.10008.1.2.4.90") 
+        || (Transfer == "1.2.840.10008.1.2.4.91") )
+      return true;
+   return false;
+}
+
 /**
  * \ingroup gdcmHeader
  * \brief   Predicate for dicom version 3 file.
@@ -587,31 +635,26 @@ void gdcmHeader::FixFoundLength(gdcmElValue * ElVal, guint32 FoundLength) {
       g = ReadInt16();
       n = ReadInt16();
       
- if (DEBUG) printf ("dans FindLengthOB (%04x,%04x)\n",g,n);
- long l = ftell(fp);
- if (DEBUG) printf("en  %d o(%o) x(%x)\n",l,l,l); 
+      long l = ftell(fp);
 
       if (errno == 1)
          return 0;
       TotalLength += 4;  // We even have to decount the group and element 
      
-      if ( g != 0xfffe           && g!=0xb00c ) /*for bogus headerJPR */ {
+      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);
-         if (DEBUG) printf("en  %d o(%o) x(%x)\n",l,l,l); 
          errno = 1;
          return 0;
       }
-      if ( n == 0xe0dd       || ( g==0xb00c && n==0x0eb6 ) ) /* for bogus header JPR */ 
+      if ( n == 0xe0dd       || ( g==0xb00c && n==0x0eb6 ) ) /* for bogus header  */ 
          FoundSequenceDelimiter = true;
       else if ( n != 0xe000 ){
          char msg[100];  // for sprintf. Sorry
          sprintf(msg,"wrong element (%04x) for an item sequence (%04x,%04x)\n",n, g,n);
         dbg.Verbose(1, "gdcmHeader::FindLengthOB: ",msg);
-         if (DEBUG) printf("wrong element (%04x) for an item sequence (%04x,%04x)\n",n, g,n);   
          errno = 1;
          return 0;
       }
@@ -619,7 +662,6 @@ void gdcmHeader::FixFoundLength(gdcmElValue * ElVal, guint32 FoundLength) {
       TotalLength += ItemLength + 4;  // We add 4 bytes since we just read
                                       // the ItemLength with ReadInt32
                                       
-      if (DEBUG) printf("TotalLength %d\n",TotalLength);
       SkipBytes(ItemLength);
    }
    fseek(fp, PositionOnEntry, SEEK_SET);
@@ -634,13 +676,12 @@ void gdcmHeader::FixFoundLength(gdcmElValue * ElVal, guint32 FoundLength) {
  */
  void gdcmHeader::FindLength (gdcmElValue * ElVal) {
    guint16 element = ElVal->GetElement();
-   guint16 group = ElVal->GetGroup(); // JPR a virer
-   string  vr      = ElVal->GetVR();
+   guint16 group   = ElVal->GetGroup();
+   std::string  vr      = ElVal->GetVR();
    guint16 length16;
-   if( (element == 0x0010) && (group == 0x7fe0) ) {// JPR
-      dbg.SetDebug(10);
-      dbg.Verbose(2, "gdcmHeader::FindLength: ", // JPR
+   if( (element == 0x0010) && (group == 0x7fe0) ) {
+      dbg.SetDebug(0);
+      dbg.Verbose(2, "gdcmHeader::FindLength: ",
                      "on est sur 7fe0 0010");
    }   
    
@@ -686,7 +727,7 @@ void gdcmHeader::FixFoundLength(gdcmElValue * ElVal, guint32 FoundLength) {
       // We shall use this second strategy. In order to make sure that we
       // can interpret the presence of an apparently big endian encoded
       // length of a "Group Length" without committing a big mistake, we
-      // add an additional check: we look in the allready parsed elements
+      // add an additional check: we look in the already parsed elements
       // for the presence of a "Transfer Syntax" whose value has to be "big
       // endian encoding". When this is the case, chances are we have got our
       // hands on a big endian encoded file: we switch the swap code to
@@ -788,7 +829,7 @@ guint16 gdcmHeader::SwapShort(guint16 a) {
 /**
  * \ingroup gdcmHeader
  * \brief   
- *
+ * @param ElVal 
  * @return 
  */
  void gdcmHeader::SkipElementValue(gdcmElValue * ElVal) {
@@ -798,7 +839,7 @@ guint16 gdcmHeader::SwapShort(guint16 a) {
 /**
  * \ingroup gdcmHeader
  * \brief   
- *
+ * @param NewSize
  * @return 
  */
  void gdcmHeader::SetMaxSizeLoadElementValue(long NewSize) {
@@ -820,7 +861,7 @@ guint16 gdcmHeader::SwapShort(guint16 a) {
 void gdcmHeader::LoadElementValue(gdcmElValue * ElVal) {
    size_t item_read;
    guint16 group  = ElVal->GetGroup();
-   string  vr     = ElVal->GetVR();
+   std::string  vr     = ElVal->GetVR();
    guint32 length = ElVal->GetLength();
    bool SkipLoad  = false;
 
@@ -840,6 +881,11 @@ void gdcmHeader::LoadElementValue(gdcmElValue * ElVal) {
    // 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
+   //          (fffe e00d) tells us an Element just ended
+   //          (fffe e0dd) tells us the current SEQuence just ended
+  
    if( group == 0xfffe )
       SkipLoad = true;
 
@@ -871,8 +917,7 @@ void gdcmHeader::LoadElementValue(gdcmElValue * ElVal) {
    // 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 aura en fait une serie d'entiers  
        // on devrait pouvoir faire + compact (?)
                
    if ( IsAnInteger(ElVal) ) {
@@ -942,20 +987,21 @@ void gdcmHeader::LoadElementValueSafe(gdcmElValue * ElVal) {
 
 /**
  * \ingroup gdcmHeader
- * \brief   
+ * \brief Reads a supposed to be 16 Bits integer
+ * \     (swaps it depending on processor endianity) 
  *
- * @return 
+ * @return integer acts as a boolean
  */
 guint16 gdcmHeader::ReadInt16(void) {
    guint16 g;
    size_t item_read;
    item_read = fread (&g, (size_t)2,(size_t)1, fp);
    if ( item_read != 1 ) {
-      dbg.Verbose(1, "gdcmHeader::ReadInt16", " Failed to read :");
-      if(feof(fp)) 
-         dbg.Verbose(1, "gdcmHeader::ReadInt16", " End of File encountered");
-     if(ferror(fp)) 
-         dbg.Verbose(1, "gdcmHeader::ReadInt16", " File Error");
+      // dbg.Verbose(0, "gdcmHeader::ReadInt16", " Failed to read :");
+      // if(feof(fp)) 
+      //    dbg.Verbose(0, "gdcmHeader::ReadInt16", " End of File encountered");
+      if(ferror(fp)) 
+         dbg.Verbose(0, "gdcmHeader::ReadInt16", " File Error");
       errno = 1;
       return 0;
    }
@@ -966,7 +1012,8 @@ guint16 gdcmHeader::ReadInt16(void) {
 
 /**
  * \ingroup gdcmHeader
- * \brief   
+ * \brief  Reads a supposed to be 32 Bits integer
+ * \       (swaps it depending on processor endianity)  
  *
  * @return 
  */
@@ -974,13 +1021,12 @@ guint32 gdcmHeader::ReadInt32(void) {
    guint32 g;
    size_t item_read;
    item_read = fread (&g, (size_t)4,(size_t)1, fp);
-   if ( item_read != 1 ) {
-   
-      dbg.Verbose(1, "gdcmHeader::ReadInt32", " Failed to read :");
-      if(feof(fp)) 
-         dbg.Verbose(1, "gdcmHeader::ReadInt32", " End of File encountered");
+   if ( item_read != 1 ) { 
+      //dbg.Verbose(0, "gdcmHeader::ReadInt32", " Failed to read :");
+      //if(feof(fp)) 
+      //   dbg.Verbose(0, "gdcmHeader::ReadInt32", " End of File encountered");
      if(ferror(fp)) 
-         dbg.Verbose(1, "gdcmHeader::ReadInt32", " File Error");   
+         dbg.Verbose(0, "gdcmHeader::ReadInt32", " File Error");   
       errno = 1;
       return 0;
    }
@@ -1032,9 +1078,12 @@ gdcmElValue* gdcmHeader::NewElValueByNumber(guint16 Group, guint16 Elem) {
 /**
  * \ingroup gdcmHeader
  * \brief   TODO
- * @param   
+ * @param   Value
+ * @param   Group
+ * @param   Elem
+ * \return integer acts as a boolean
  */
-int gdcmHeader::ReplaceOrCreateByNumber(string Value, guint16 Group, guint16 Elem ) {
+int gdcmHeader::ReplaceOrCreateByNumber(std::string Value, guint16 Group, guint16 Elem ) {
 
        // TODO : FIXME JPRx
        // curieux, non ?
@@ -1043,38 +1092,62 @@ int gdcmHeader::ReplaceOrCreateByNumber(string Value, guint16 Group, guint16 Ele
        // on affecte une valeur a cette ElValue a l'interieur du ElValSet
        // --> devrait pouvoir etre fait + simplement ???
        
-       gdcmElValue* nvElValue=NewElValueByNumber(Group, Elem);
-       PubElValSet.Add(nvElValue);     
-       PubElValSet.SetElValueByNumber(Value, Group, Elem);
-       return(1);
+   gdcmElValue* nvElValue=NewElValueByNumber(Group, Elem);
+   PubElValSet.Add(nvElValue); 
+   PubElValSet.SetElValueByNumber(Value, Group, Elem);
+   return(1);
 }   
 
 
 /**
  * \ingroup gdcmHeader
- * \brief   TODO
- * @param   
+ * \brief   Modify or (Creates if not found) an element
+ * @param   Value new value
+ * @param   Group
+ * @param   Elem
+ * \return integer acts as a boolean 
+ * 
  */
 int gdcmHeader::ReplaceOrCreateByNumber(char* Value, guint16 Group, guint16 Elem ) {
 
-       gdcmElValue* nvElValue=NewElValueByNumber(Group, Elem);
-       PubElValSet.Add(nvElValue);
-       string v = Value;       
-       PubElValSet.SetElValueByNumber(v, Group, Elem);
-       return(1);
+   gdcmElValue* nvElValue=NewElValueByNumber(Group, Elem);
+   PubElValSet.Add(nvElValue);
+   std::string v = Value;      
+   PubElValSet.SetElValueByNumber(v, Group, Elem);
+   return(1);
 }  
 
+
 /**
  * \ingroup gdcmHeader
- * \brief   TODO
- * @param   
+ * \brief   Set a new value if the invoked element exists
+ * @param   Value
+ * @param   Group
+ * @param   Elem
+ * \return integer acts as a boolean 
+ */
+int gdcmHeader::ReplaceIfExistByNumber(char* Value, guint16 Group, guint16 Elem ) {
+
+   gdcmElValue* elValue = PubElValSet.GetElementByNumber(Group, Elem);
+   std::string v = Value;      
+   PubElValSet.SetElValueByNumber(v, Group, Elem);
+   return 1;
+} 
+
+
+/**
+ * \ingroup gdcmHeader
+ * \brief   Checks if a given ElValue (group,number) 
+ * \ exists in the Public ElValSet
+ * @param   Group
+ * @param   Elem
+ * @return  integer acts as a boolean  
  */
  
- int gdcmHeader::CheckIfExistByNumber(guint16 Group, guint16 Elem ) {
-       return (PubElValSet.CheckIfExistByNumber(Group, Elem));
+int gdcmHeader::CheckIfExistByNumber(guint16 Group, guint16 Elem ) {
+   return (PubElValSet.CheckIfExistByNumber(Group, Elem));
  }
+  
 /**
  * \ingroup gdcmHeader
  * \brief   Build a new Element Value from all the low level arguments. 
@@ -1082,7 +1155,7 @@ int gdcmHeader::ReplaceOrCreateByNumber(char* Value, guint16 Group, guint16 Elem
  *          a default one when absent.
  * @param   Name    Name of the underlying DictEntry
  */
-gdcmElValue* gdcmHeader::NewElValueByName(string Name) {
+gdcmElValue* gdcmHeader::NewElValueByName(std::string Name) {
 
    gdcmDictEntry * NewTag = GetDictEntryByName(Name);
    if (!NewTag)
@@ -1111,8 +1184,6 @@ gdcmElValue * gdcmHeader::ReadNextElement(void) {
    n = ReadInt16();
    
    if ( (g==0x7fe0) && (n==0x0010) ) 
-       if (DEBUG) 
-               printf("in gdcmHeader::ReadNextElement try to read 7fe0 0010 \n");
    
    if (errno == 1)
       // We reached the EOF (or an error occured) and header parsing
@@ -1124,13 +1195,10 @@ gdcmElValue * gdcmHeader::ReadNextElement(void) {
    FindLength(NewElVal);
    if (errno == 1) {
       // Call it quits
-      if (DEBUG) printf("in gdcmHeader::ReadNextElement : g %04x n %04x errno %d\n",g, n, errno);
       return (gdcmElValue *)0;
    }
    NewElVal->SetOffset(ftell(fp));  
    if ( (g==0x7fe0) && (n==0x0010) ) 
-       if (DEBUG) 
-               printf("sortie de gdcmHeader::ReadNextElement 7fe0 0010 \n");
    return NewElVal;
 }
 
@@ -1144,7 +1212,7 @@ gdcmElValue * gdcmHeader::ReadNextElement(void) {
 bool gdcmHeader::IsAnInteger(gdcmElValue * ElVal) {
    guint16 group   = ElVal->GetGroup();
    guint16 element = ElVal->GetElement();
-   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
@@ -1154,9 +1222,6 @@ bool gdcmHeader::IsAnInteger(gdcmElValue * ElVal) {
       if (length == 4)
          return true;
       else {
-         if (DEBUG) printf("Erroneous Group Length element length (%04x , %04x) : %d\n",
-            group, element,length);
-                    
          dbg.Error("gdcmHeader::IsAnInteger",
             "Erroneous Group Length element length.");     
       }
@@ -1181,8 +1246,8 @@ size_t gdcmHeader::GetPixelOffset(void) {
    // When the "Image Location" is absent we default to group 0x7fe0.
    guint16 grPixel;
    guint16 numPixel;
-   string ImageLocation = GetPubElValByName("Image Location");
-   if ( ImageLocation == "gdcm::Unfound" ) {
+   std::string ImageLocation = GetPubElValByName("Image Location");
+   if ( ImageLocation == GDCM_UNFOUND ) {
       grPixel = 0x7fe0;
    } else {
       grPixel = (guint16) atoi( ImageLocation.c_str() );
@@ -1239,7 +1304,7 @@ gdcmDictEntry * gdcmHeader::GetDictEntryByNumber(guint16 group,
  * @param   Name name of the searched DictEntry
  * @return  Corresponding DictEntry when it exists, NULL otherwise.
  */
-gdcmDictEntry * gdcmHeader::GetDictEntryByName(string Name) {
+gdcmDictEntry * gdcmHeader::GetDictEntryByName(std::string Name) {
    gdcmDictEntry * found = (gdcmDictEntry*)0;
    if (!RefPubDict && !RefShaDict) {
       dbg.Verbose(0, "gdcmHeader::GetDictEntry",
@@ -1265,9 +1330,9 @@ gdcmDictEntry * gdcmHeader::GetDictEntryByName(string Name) {
  * @param   group Group of the researched tag.
  * @param   element Element of the researched tag.
  * @return  Corresponding element value when it exists, and the string
- *          "gdcm::Unfound" otherwise.
+ *          GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
-string gdcmHeader::GetPubElValByNumber(guint16 group, guint16 element) {
+std::string gdcmHeader::GetPubElValByNumber(guint16 group, guint16 element) {
    return PubElValSet.GetElValueByNumber(group, element);
 }
 
@@ -1283,12 +1348,12 @@ string gdcmHeader::GetPubElValByNumber(guint16 group, guint16 element) {
  * @param   group Group of the researched tag.
  * @param   element Element of the researched tag.
  * @return  Corresponding element value representation when it exists,
- *          and the string "gdcm::Unfound" otherwise.
+ *          and the string GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
-string gdcmHeader::GetPubElValRepByNumber(guint16 group, guint16 element) {
+std::string gdcmHeader::GetPubElValRepByNumber(guint16 group, guint16 element) {
    gdcmElValue* elem =  PubElValSet.GetElementByNumber(group, element);
    if ( !elem )
-      return "gdcm::Unfound";
+      return GDCM_UNFOUND;
    return elem->GetVR();
 }
 
@@ -1298,9 +1363,9 @@ string gdcmHeader::GetPubElValRepByNumber(guint16 group, guint16 element) {
  *          a given tag.
  * @param   TagName name of the researched element.
  * @return  Corresponding element value when it exists, and the string
- *          "gdcm::Unfound" otherwise.
+ *          GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
-string gdcmHeader::GetPubElValByName(string TagName) {
+std::string gdcmHeader::GetPubElValByName(std::string TagName) {
    return PubElValSet.GetElValueByName(TagName);
 }
 
@@ -1315,12 +1380,12 @@ string gdcmHeader::GetPubElValByName(string TagName) {
  *          of semantics than just the native C++ type.
  * @param   TagName name of the researched element.
  * @return  Corresponding element value representation when it exists,
- *          and the string "gdcm::Unfound" otherwise.
+ *          and the string GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
-string gdcmHeader::GetPubElValRepByName(string TagName) {
+std::string gdcmHeader::GetPubElValRepByName(std::string TagName) {
    gdcmElValue* elem =  PubElValSet.GetElementByName(TagName);
    if ( !elem )
-      return "gdcm::Unfound";
+      return GDCM_UNFOUND;
    return elem->GetVR();
 }
 
@@ -1331,9 +1396,9 @@ string gdcmHeader::GetPubElValRepByName(string TagName) {
  * @param   group Group of the researched tag.
  * @param   element Element of the researched tag.
  * @return  Corresponding element value representation when it exists,
- *          and the string "gdcm::Unfound" otherwise.
+ *          and the string GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
-string gdcmHeader::GetShaElValByNumber(guint16 group, guint16 element) {
+std::string gdcmHeader::GetShaElValByNumber(guint16 group, guint16 element) {
    return ShaElValSet.GetElValueByNumber(group, element);
 }
 
@@ -1349,12 +1414,12 @@ string gdcmHeader::GetShaElValByNumber(guint16 group, guint16 element) {
  * @param   group Group of the researched tag.
  * @param   element Element of the researched tag.
  * @return  Corresponding element value representation when it exists,
- *          and the string "gdcm::Unfound" otherwise.
+ *          and the string GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
-string gdcmHeader::GetShaElValRepByNumber(guint16 group, guint16 element) {
+std::string gdcmHeader::GetShaElValRepByNumber(guint16 group, guint16 element) {
    gdcmElValue* elem =  ShaElValSet.GetElementByNumber(group, element);
    if ( !elem )
-      return "gdcm::Unfound";
+      return GDCM_UNFOUND;
    return elem->GetVR();
 }
 
@@ -1364,9 +1429,9 @@ string gdcmHeader::GetShaElValRepByNumber(guint16 group, guint16 element) {
  *          for an element value of given tag.
  * @param   TagName name of the researched element.
  * @return  Corresponding element value when it exists, and the string
- *          "gdcm::Unfound" otherwise.
+ *          GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
-string gdcmHeader::GetShaElValByName(string TagName) {
+std::string gdcmHeader::GetShaElValByName(std::string TagName) {
    return ShaElValSet.GetElValueByName(TagName);
 }
 
@@ -1381,12 +1446,12 @@ string gdcmHeader::GetShaElValByName(string TagName) {
  *          of semantics than just the native C++ type.
  * @param   TagName name of the researched element.
  * @return  Corresponding element value representation when it exists,
- *          and the string "gdcm::Unfound" otherwise.
+ *          and the string GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
-string gdcmHeader::GetShaElValRepByName(string TagName) {
+std::string gdcmHeader::GetShaElValRepByName(std::string TagName) {
    gdcmElValue* elem =  ShaElValSet.GetElementByName(TagName);
    if ( !elem )
-      return "gdcm::Unfound";
+      return GDCM_UNFOUND;
    return elem->GetVR();
 }
 
@@ -1398,10 +1463,10 @@ string gdcmHeader::GetShaElValRepByName(string TagName) {
  * @param   group Group of the researched tag.
  * @param   element Element of the researched tag.
  * @return  Corresponding element value representation when it exists,
- *          and the string "gdcm::Unfound" otherwise.
+ *          and the string GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
-string gdcmHeader::GetElValByNumber(guint16 group, guint16 element) {
-   string pub = GetPubElValByNumber(group, element);
+std::string gdcmHeader::GetElValByNumber(guint16 group, guint16 element) {
+   std::string pub = GetPubElValByNumber(group, element);
    if (pub.length())
       return pub;
    return GetShaElValByNumber(group, element);
@@ -1420,10 +1485,10 @@ string gdcmHeader::GetElValByNumber(guint16 group, guint16 element) {
  * @param   group Group of the researched tag.
  * @param   element Element of the researched tag.
  * @return  Corresponding element value representation when it exists,
- *          and the string "gdcm::Unfound" otherwise.
+ *          and the string GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
-string gdcmHeader::GetElValRepByNumber(guint16 group, guint16 element) {
-   string pub = GetPubElValRepByNumber(group, element);
+std::string gdcmHeader::GetElValRepByNumber(guint16 group, guint16 element) {
+   std::string pub = GetPubElValRepByNumber(group, element);
    if (pub.length())
       return pub;
    return GetShaElValRepByNumber(group, element);
@@ -1436,10 +1501,10 @@ string gdcmHeader::GetElValRepByNumber(guint16 group, guint16 element) {
  *          for the element value of a given tag.
  * @param   TagName name of the researched element.
  * @return  Corresponding element value when it exists,
- *          and the string "gdcm::Unfound" otherwise.
+ *          and the string GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
-string gdcmHeader::GetElValByName(string TagName) {
-   string pub = GetPubElValByName(TagName);
+std::string gdcmHeader::GetElValByName(std::string TagName) {
+   std::string pub = GetPubElValByName(TagName);
    if (pub.length())
       return pub;
    return GetShaElValByName(TagName);
@@ -1457,10 +1522,10 @@ string gdcmHeader::GetElValByName(string TagName) {
  *          of semantics than just the native C++ type.
  * @param   TagName name of the researched element.
  * @return  Corresponding element value representation when it exists,
- *          and the string "gdcm::Unfound" otherwise.
+ *          and the string GDCM_UNFOUND ("gdcm::Unfound") otherwise.
  */
-string gdcmHeader::GetElValRepByName(string TagName) {
-   string pub = GetPubElValRepByName(TagName);
+std::string gdcmHeader::GetElValRepByName(std::string TagName) {
+   std::string pub = GetPubElValRepByName(TagName);
    if (pub.length())
       return pub;
    return GetShaElValRepByName(TagName);
@@ -1475,7 +1540,7 @@ string gdcmHeader::GetElValRepByName(string TagName) {
  * @param   group   group of the ElVal to modify
  * @param   element element of the ElVal to modify
  */
-int gdcmHeader::SetPubElValByNumber(string content, guint16 group,
+int gdcmHeader::SetPubElValByNumber(std::string content, guint16 group,
                                     guint16 element)
                                     
 //TODO  : homogeneiser les noms : SetPubElValByNumber   qui appelle PubElValSet.SetElValueByNumber 
@@ -1492,7 +1557,7 @@ int gdcmHeader::SetPubElValByNumber(string content, guint16 group,
  * @param   content new value to substitute with
  * @param   TagName name of the tag to be modified
  */
-int gdcmHeader::SetPubElValByName(string content, string TagName) {
+int gdcmHeader::SetPubElValByName(std::string content, std::string TagName) {
    return (  PubElValSet.SetElValueByName (content, TagName) );
 }
 
@@ -1523,7 +1588,7 @@ int gdcmHeader::SetPubElValLengthByNumber(guint32 length, guint16 group,
  * @param   element element of the ElVal to modify
  * @return  1 on success, 0 otherwise.
  */
-int gdcmHeader::SetShaElValByNumber(string content,
+int gdcmHeader::SetShaElValByNumber(std::string content,
                                     guint16 group, guint16 element) {
    return (  ShaElValSet.SetElValueByNumber (content, group, element) );
 }
@@ -1535,7 +1600,7 @@ int gdcmHeader::SetShaElValByNumber(string content,
  * @param   content new value to substitute with
  * @param   ShadowTagName name of the tag to be modified
  */
-int gdcmHeader::SetShaElValByName(string content, string ShadowTagName) {
+int gdcmHeader::SetShaElValByName(std::string content, std::string ShadowTagName) {
    return (  ShaElValSet.SetElValueByName (content, ShadowTagName) );
 }
 
@@ -1554,19 +1619,211 @@ void gdcmHeader::ParseHeader(bool exception_on_error) throw(gdcmFormatError) {
    }
 }
 
+/**
+ * \ingroup gdcmHeader
+ * \brief  This predicate, based on hopefully reasonnable 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.
+ * @return true when gdcmHeader is the one of a reasonable Dicom file,
+ *         false otherwise. 
+ */
+bool gdcmHeader::IsReadable(void) {
+   if (   GetElValByName("Image Dimensions") != GDCM_UNFOUND
+      && atoi(GetElValByName("Image Dimensions").c_str()) > 4 ) {
+      return false;
+   }
+   if ( GetElValByName("Bits Allocated")       == GDCM_UNFOUND )
+      return false;
+   if ( GetElValByName("Bits Stored")          == GDCM_UNFOUND )
+      return false;
+   if ( GetElValByName("High Bit")             == GDCM_UNFOUND )
+      return false;
+   if ( GetElValByName("Pixel Representation") == GDCM_UNFOUND )
+      return false;
+   return true;
+}
+
+/**
+ * \ingroup gdcmHeader
+ * \brief   Small utility function that creates a new manually crafted
+ *          (as opposed as read from the file) gdcmElValue with user
+ *          specified name and adds it to the public tag hash table.
+ * \note    A fake TagKey is generated so the PubDict can keep it's coherence.
+ * @param   NewTagName The name to be given to this new tag.
+ * @param   VR The Value Representation to be given to this new tag.
+ * @ return The newly hand crafted Element Value.
+ */
+gdcmElValue* gdcmHeader::NewManualElValToPubDict(std::string NewTagName, std::string VR) {
+   gdcmElValue* NewElVal = (gdcmElValue*)0;
+   guint32 StuffGroup = 0xffff;   // Group to be stuffed with additional info
+   guint32 FreeElem = 0;
+   gdcmDictEntry* NewEntry = (gdcmDictEntry*)0;
+
+   FreeElem = PubElValSet.GenerateFreeTagKeyInGroup(StuffGroup);
+   if (FreeElem == UINT32_MAX) {
+      dbg.Verbose(1, "gdcmHeader::NewManualElValToPubDict",
+                     "Group 0xffff in Public Dict is full");
+      return (gdcmElValue*)0;
+   }
+   NewEntry = new gdcmDictEntry(StuffGroup, FreeElem,
+                                VR, "GDCM", NewTagName);
+   NewElVal = new gdcmElValue(NewEntry);
+   PubElValSet.Add(NewElVal);
+   return NewElVal;
+}
+
+/**
+ * \ingroup gdcmHeader
+ * \brief   Loads the element values of all the elements present in the
+ *          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);
+
+   // Load 'non string' values   
+   std::string PhotometricInterpretation = GetPubElValByNumber(0x0028,0x0004);   
+   if( PhotometricInterpretation == "PALETTE COLOR " ){ 
+      LoadElementVoidArea(0x0028,0x1200);  // gray LUT   
+      LoadElementVoidArea(0x0028,0x1201);  // R    LUT
+      LoadElementVoidArea(0x0028,0x1202);  // G    LUT
+      LoadElementVoidArea(0x0028,0x1203);  // B    LUT
+      
+      LoadElementVoidArea(0x0028,0x1221);  // Segmented Red   Palette Color LUT Data
+      LoadElementVoidArea(0x0028,0x1222);  // Segmented Green Palette Color LUT Data
+      LoadElementVoidArea(0x0028,0x1223);  // Segmented Blue  Palette Color LUT Data
+   }
+
+   // --------------------------------------------------------------
+   // Special Patch to allow gdcm to read ACR-LibIDO formated images
+   //
+   // if recognition code tells us we deal with a LibIDO image
+   // we switch lineNumber and columnNumber
+   //
+   std::string RecCode;        
+   RecCode = GetPubElValByNumber(0x0008, 0x0010);
+   if (RecCode == "ACRNEMA_LIBIDO_1.1" ||
+       RecCode == "CANRME_AILIBOD1_1." ) {
+         filetype = ACR_LIBIDO; 
+         std::string rows    = GetPubElValByNumber(0x0028, 0x0010);
+         std::string columns = GetPubElValByNumber(0x0028, 0x0011);
+         SetPubElValByNumber(columns, 0x0028, 0x0010);
+         SetPubElValByNumber(rows   , 0x0028, 0x0011);
+   }
+   // ----------------- End of Special Patch ----------------
+}
+
+/**
+  * \ingroup gdcmHeader
+  * \brief
+  * @return
+  */ 
+void gdcmHeader::PrintPubElVal(std::ostream & os) {
+   PubElValSet.Print(os);
+}
+
+/**
+  * \ingroup gdcmHeader
+  * \brief
+  * @return
+  */  
+void gdcmHeader::PrintPubDict(std::ostream & os) {
+   RefPubDict->Print(os);
+}
+
+/**
+  * \ingroup gdcmHeader
+  * \brief
+  * @return
+  */ 
+int gdcmHeader::Write(FILE * fp, FileType type) {
+   return PubElValSet.Write(fp, type);
+}
+
+//
+// ------------------------ 'non string' elements related functions
+//
+
+/**
+ * \ingroup       gdcmHeader
+ * \brief         Loads (from disk) the element content 
+ *                when a string is not suitable
+ */
+void * gdcmHeader::LoadElementVoidArea(guint16 Group, guint16 Elem) {
+   gdcmElValue * Element= PubElValSet.GetElementByNumber(Group, Elem);
+   if ( !Element )
+      return NULL;
+   size_t o =(size_t)Element->GetOffset();
+   fseek(fp, o, SEEK_SET);
+   int l=Element->GetLength();
+   void * a = malloc(l);
+   if(!a) {
+       cout << "Big Broblem (LoadElementVoidArea, malloc) " 
+            << hex << Group << " " << Elem << std::endl;
+       return NULL;
+   }  
+   int res = PubElValSet.SetVoidAreaByNumber(a, Group, Elem);
+   // TODO check the result 
+   size_t l2 = fread(a, 1, l ,fp);
+   if(l != l2) {
+       cout << "Big Broblem (LoadElementVoidArea, fread) " 
+            << hex << Group << " " << Elem << std::endl;
+       free(a);
+       return NULL;
+   }  
+}
+
+/**
+ * \ingroup gdcmHeader
+ * \brief   Gets (from Header) the offset  of a 'non string' element value 
+ * \        (LoadElementValue has already be executed)
+ * @param   Group
+ * @param   Elem
+ * @return File Offset of the Element Value 
+ */
+ size_t gdcmHeader::GetPubElValOffsetByNumber(guint16 Group, guint16 Elem) {
+   gdcmElValue* elValue = PubElValSet.GetElementByNumber(Group, Elem);  
+   if (!elValue) {
+      dbg.Verbose(1, "gdcmHeader::GetElValueByNumber",
+                  "failed to Locate gdcmElValue");
+      return (size_t)0;
+   }
+   return elValue->GetOffset();
+}
+
+/**
+ * \ingroup gdcmHeader
+* \brief   Gets (from Header) a 'non string' element value 
+ * \        (LoadElementValue has already be executed)  
+ * @param   Group
+ * @param   Elem
+ * @return Pointer to the 'non string' area
+ */
+ void * gdcmHeader::GetPubElValVoidAreaByNumber(guint16 Group, guint16 Elem) {
+   gdcmElValue* elValue = PubElValSet.GetElementByNumber(Group, Elem);  
+   if (!elValue) {
+      dbg.Verbose(1, "gdcmHeader::GetElValueByNumber",
+                  "failed to Locate gdcmElValue");
+      return (NULL);
+   }
+   return elValue->GetVoidArea();
+}
+
 
 //
-// TODO : JPR
-// des que les element values sont chargees, stocker, 
-// en une seule fois, dans des entiers 
-// NX, NY, NZ, Bits allocated, Bits Stored, High Bit, Samples Per Pixel
-// (TODO : preciser les autres)
-// et refaire ceux des accesseurs qui renvoient les entiers correspondants
+// =============================================================================
+//   Heuristics based accessors
+//==============================================================================
 //
-// --> peut etre dangereux ?
-// si l'utilisateur modifie 'manuellement' l'un des paramètres
-// l'entier de sera pas modifié ...
-// (pb de la mise à jour en cas de redondance :-(
+
+// TODO : move to an other file.
+
 
 /**
  * \ingroup gdcmHeader
@@ -1576,8 +1833,8 @@ void gdcmHeader::ParseHeader(bool exception_on_error) throw(gdcmFormatError) {
 int gdcmHeader::GetXSize(void) {
    // We cannot check for "Columns" because the "Columns" tag is present
    // both in IMG (0028,0011) and OLY (6000,0011) sections of the dictionary.
-   string StrSize = GetPubElValByNumber(0x0028,0x0011);
-   if (StrSize == "gdcm::Unfound")
+   std::string StrSize = GetPubElValByNumber(0x0028,0x0011);
+   if (StrSize == GDCM_UNFOUND)
       return 0;
    return atoi(StrSize.c_str());
 }
@@ -1591,8 +1848,8 @@ int gdcmHeader::GetXSize(void) {
 int gdcmHeader::GetYSize(void) {
    // We cannot check for "Rows" because the "Rows" tag is present
    // both in IMG (0028,0010) and OLY (6000,0010) sections of the dictionary.
-   string StrSize = GetPubElValByNumber(0x0028,0x0010);
-   if (StrSize != "gdcm::Unfound")
+   std::string StrSize = GetPubElValByNumber(0x0028,0x0010);
+   if (StrSize != GDCM_UNFOUND)
       return atoi(StrSize.c_str());
    if ( IsDicomV3() )
       return 0;
@@ -1614,20 +1871,19 @@ int gdcmHeader::GetYSize(void) {
 int gdcmHeader::GetZSize(void) {
    // Both in DicomV3 and ACR/Nema the consider the "Number of Frames"
    // as the third dimension.
-   string StrSize = GetPubElValByNumber(0x0028,0x0008);
-   if (StrSize != "gdcm::Unfound")
+   std::string StrSize = GetPubElValByNumber(0x0028,0x0008);
+   if (StrSize != GDCM_UNFOUND)
       return atoi(StrSize.c_str());
 
    // We then consider the "Planes" entry as the third dimension [we
    // cannot retrieve by name since "Planes tag is present both in
    // IMG (0028,0012) and OLY (6000,0012) sections of the dictionary]. 
    StrSize = GetPubElValByNumber(0x0028,0x0012);
-   if (StrSize != "gdcm::Unfound")
+   if (StrSize != GDCM_UNFOUND)
       return atoi(StrSize.c_str());
    return 1;
 }
 
-
 /**
  * \ingroup gdcmHeader
  * \brief   Retrieve the number of Bits Stored
@@ -1636,8 +1892,8 @@ int gdcmHeader::GetZSize(void) {
  * @return  The encountered number of Bits Stored, 0 by default.
  */
 int gdcmHeader::GetBitsStored(void) { 
-   string StrSize = GetPubElValByNumber(0x0028,0x0101);
-   if (StrSize == "gdcm::Unfound")
+   std::string StrSize = GetPubElValByNumber(0x0028,0x0101);
+   if (StrSize == GDCM_UNFOUND)
       return 1;
    return atoi(StrSize.c_str());
 }
@@ -1651,11 +1907,26 @@ int gdcmHeader::GetBitsStored(void) {
  * @return  The encountered number of Samples Per Pixel, 1 by default.
  */
 int gdcmHeader::GetSamplesPerPixel(void) { 
-   string StrSize = GetPubElValByNumber(0x0028,0x0002);
-   if (StrSize == "gdcm::Unfound")
+   std::string StrSize = GetPubElValByNumber(0x0028,0x0002);
+   if (StrSize == GDCM_UNFOUND)
       return 1; // Well, it's supposed to be mandatory ...
    return atoi(StrSize.c_str());
 }
+
+/**
+ * \ingroup gdcmHeader
+ * \brief   Retrieve the Planar Configuration for RGB images
+ *          (0 : RGB Pixels , 1 : R Plane + G Plane + B Plane)
+ * 
+ * @return  The encountered Planar Configuration, 0 by default.
+ */
+int gdcmHeader::GetPlanarConfiguration(void) { 
+   std::string StrSize = GetPubElValByNumber(0x0028,0x0006);
+   if (StrSize == GDCM_UNFOUND)
+      return 0;
+   return atoi(StrSize.c_str());
+}
+
 /**
  * \ingroup gdcmHeader
  * \brief   Return the size (in bytes) of a single pixel of data.
@@ -1663,7 +1934,7 @@ int gdcmHeader::GetSamplesPerPixel(void) {
  *
  */
 int gdcmHeader::GetPixelSize(void) {
-   string PixelType = GetPixelType();
+   std::string PixelType = GetPixelType();
    if (PixelType == "8U"  || PixelType == "8S")
       return 1;
    if (PixelType == "16U" || PixelType == "16S")
@@ -1687,330 +1958,223 @@ int gdcmHeader::GetPixelSize(void) {
  * \warning 12 bit images appear as 16 bit.
  * @return  
  */
-string gdcmHeader::GetPixelType(void) {
-   string BitsAlloc;
+std::string gdcmHeader::GetPixelType(void) {
+   std::string BitsAlloc;
    BitsAlloc = GetElValByName("Bits Allocated");
-   if (BitsAlloc == "gdcm::Unfound") {
+   if (BitsAlloc == GDCM_UNFOUND) {
       dbg.Verbose(0, "gdcmHeader::GetPixelType: unfound Bits Allocated");
-      BitsAlloc = string("16");
+      BitsAlloc = std::string("16");
    }
    if (BitsAlloc == "12")
-      BitsAlloc = string("16");
+      BitsAlloc = std::string("16");
 
-   string Signed;
+   std::string Signed;
    Signed = GetElValByName("Pixel Representation");
-   if (Signed == "gdcm::Unfound") {
+   if (Signed == GDCM_UNFOUND) {
       dbg.Verbose(0, "gdcmHeader::GetPixelType: unfound Pixel Representation");
-      BitsAlloc = string("0");
+      BitsAlloc = std::string("0");
    }
    if (Signed == "0")
-      Signed = string("U");
+      Signed = std::string("U");
    else
-      Signed = string("S");
+      Signed = std::string("S");
 
    return( BitsAlloc + Signed);
 }
 
-
-/**
- * \ingroup gdcmHeader
- * \brief  This predicate, based on hopefully reasonnable 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.
- * @return true when gdcmHeader is the one of a reasonable Dicom file,
- *         false otherwise. 
- */
-bool gdcmHeader::IsReadable(void) {
-   if (   GetElValByName("Image Dimensions") != "gdcm::Unfound"
-      && atoi(GetElValByName("Image Dimensions").c_str()) > 4 ) {
-      return false;
-   }
-   if ( GetElValByName("Bits Allocated") == "gdcm::Unfound" )
-      return false;
-   if ( GetElValByName("Bits Stored") == "gdcm::Unfound" )
-      return false;
-   if ( GetElValByName("High Bit") == "gdcm::Unfound" )
-      return false;
-   if ( GetElValByName("Pixel Representation") == "gdcm::Unfound" )
-      return false;
-   return true;
-}
-
-/**
- * \ingroup gdcmHeader
- * \brief   Small utility function that creates a new manually crafted
- *          (as opposed as read from the file) gdcmElValue with user
- *          specified name and adds it to the public tag hash table.
- * \note    A fake TagKey is generated so the PubDict can keep it's coherence.
- * @param   NewTagName The name to be given to this new tag.
- * @param   VR The Value Representation to be given to this new tag.
- * @ return The newly hand crafted Element Value.
- */
-gdcmElValue* gdcmHeader::NewManualElValToPubDict(string NewTagName, string VR) {
-   gdcmElValue* NewElVal = (gdcmElValue*)0;
-   guint32 StuffGroup = 0xffff;   // Group to be stuffed with additional info
-   guint32 FreeElem = 0;
-   gdcmDictEntry* NewEntry = (gdcmDictEntry*)0;
-
-   FreeElem = PubElValSet.GenerateFreeTagKeyInGroup(StuffGroup);
-   if (FreeElem == UINT32_MAX) {
-      dbg.Verbose(1, "gdcmHeader::NewManualElValToPubDict",
-                     "Group 0xffff in Public Dict is full");
-      return (gdcmElValue*)0;
-   }
-   NewEntry = new gdcmDictEntry(StuffGroup, FreeElem,
-                                VR, "GDCM", NewTagName);
-   NewElVal = new gdcmElValue(NewEntry);
-   PubElValSet.Add(NewElVal);
-   return NewElVal;
-}
-
-/**
- * \ingroup gdcmHeader
- * \brief   Loads the element values of all the elements present in the
- *          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);
-   }
-}
-
 /**
   * \ingroup gdcmHeader
-  * \brief
-  * @return
-  */ 
-void gdcmHeader::PrintPubElVal(std::ostream & os) {
-   PubElValSet.Print(os);
+  * \brief gets the info from 0002,0010 : Transfert Syntax
+  * \           else 1.
+  * @return Transfert Syntax Name (as oposite to Transfert Syntax UID)
+  */
+std::string gdcmHeader::GetTransferSyntaxName(void) { 
+   std::string TransfertSyntax = GetPubElValByNumber(0x0002,0x0010);
+   if (TransfertSyntax == GDCM_UNFOUND) {
+      dbg.Verbose(0, "gdcmHeader::GetTransferSyntaxName: unfound Transfert Syntax (0002,0010)");
+      return "Uncompressed ACR-NEMA";
+   }
+   // we do it only when we need it
+   gdcmTS * ts = gdcmGlobal::GetTS();
+   std::string tsName=ts->GetValue(TransfertSyntax);
+   //delete ts; // Seg Fault when deleted ?!
+   return tsName;
 }
 
-/**
-  * \ingroup gdcmHeader
-  * \brief
-  * @return
-  */  
-void gdcmHeader::PrintPubDict(std::ostream & os) {
-   RefPubDict->Print(os);
-}
+// -------------------------------- Lookup Table related functions ------------
 
 /**
   * \ingroup gdcmHeader
-  * \brief
-  * @return
+  * \brief gets the info from 0028,1101 : Lookup Table Desc-Red
+  * \           else 0
+  * @return Lookup Table Length 
+  * \       when (0028,0004),Photometric Interpretation = [PALETTE COLOR ] 
   */
   
-int gdcmHeader::Write(FILE * fp, FileType type) {
-   return PubElValSet.Write(fp, type);
+int gdcmHeader::GetLUTLength(void) {
+   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)
+      return 0;
+   std::string LutDescriptionG = GetPubElValByNumber(0x0028,0x1102);
+   if (LutDescriptionG == GDCM_UNFOUND)
+      return 0;
+   std::string LutDescriptionB = GetPubElValByNumber(0x0028,0x1103);
+   if (LutDescriptionB == GDCM_UNFOUND)
+      return 0;
+   if( (LutDescriptionR != LutDescriptionG) || (LutDescriptionR != LutDescriptionB) ) {
+      dbg.Verbose(0, "gdcmHeader::GetLUTLength: The CLUT R,G,B are not equal");
+      return 0;   
+   } 
+   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;
 }
 
 /**
   * \ingroup gdcmHeader
-  * \brief
-  * @return
+  * \brief gets the info from 0028,1101 : Lookup Table Desc-Red
+  * \           else 0
+  * @return Lookup Table nBit 
+  * \       when (0028,0004),Photometric Interpretation = [PALETTE COLOR ] 
   */
-float gdcmHeader::GetXSpacing(void) {
-    float xspacing, yspacing;
-    string StrSpacing = GetPubElValByNumber(0x0028,0x0030);
-
-    if (StrSpacing == "gdcm::Unfound") {
-       dbg.Verbose(0, "gdcmHeader::GetXSpacing: unfound Pixel Spacing");
-       return 1.;
-     }
-   if( sscanf( StrSpacing.c_str(), "%f\\%f", &xspacing, &yspacing) != 2)
-     return 0.;
-   //else
-   return xspacing;
+  
+int gdcmHeader::GetLUTNbits(void) {
+   vector<std::string> tokens;
+   //int LutLength;
+   //int LutDepth;
+   int LutNbits;
+   // Just hope Lookup Table Desc-Red = Lookup Table Desc-Red = Lookup Table Desc-Blue
+   // Consistency already checked in GetLUTLength
+   std::string LutDescription = GetPubElValByNumber(0x0028,0x1101);
+   if (LutDescription == GDCM_UNFOUND)
+      return 0;
+   tokens.erase(tokens.begin(),tokens.end()); // clean any previous value
+   Tokenize (LutDescription, tokens, "\\");
+   //LutLength=atoi(tokens[0].c_str());
+   //LutDepth=atoi(tokens[1].c_str());
+   LutNbits=atoi(tokens[2].c_str());
+   tokens.clear();
+   return LutNbits;
 }
-
-/**
-  * \ingroup gdcmHeader
-  * \brief
-  * @return
-  */
-float gdcmHeader::GetYSpacing(void) {
-   float xspacing, yspacing;
-   string StrSpacing = GetPubElValByNumber(0x0028,0x0030);
   
-   if (StrSpacing == "gdcm::Unfound") {
-      dbg.Verbose(0, "gdcmHeader::GetYSpacing: unfound Pixel Spacing");
-      return 1.;
-    }
-
-  if( sscanf( StrSpacing.c_str(), "%f\\%f", &xspacing, &yspacing) != 2)
-    return 0.;
-
-  if (yspacing == 0.)
-  {
-    dbg.Verbose(0, "gdcmHeader::GetYSpacing: gdcmData/CT-MONO2-8-abdo.dcm problem");
-    // seems to be a bug in the header ...
-    sscanf( StrSpacing.c_str(), "%f\\0\\%f", &xspacing, &yspacing);
-  }
-  return yspacing;
-} 
-
 
 /**
   * \ingroup gdcmHeader
-  * \brief
-  * @return
-  */
-float gdcmHeader::GetZSpacing(void) {
-   // TODO : translate into English
-   // Spacing Between Slices : distance entre le milieu de chaque coupe
-   // Les coupes peuvent etre :
-   //   jointives     (Spacing between Slices = Slice Thickness)
-   //   chevauchantes (Spacing between Slices < Slice Thickness)
-   //   disjointes    (Spacing between Slices > Slice Thickness)
-   // Slice Thickness : epaisseur de tissus sur laquelle est acquis le signal
-   //   ca interesse le physicien de l'IRM, pas le visualisateur de volumes ...
-   //   Si le Spacing Between Slices est absent, 
-   //   on suppose que les coupes sont jointives
-   
-   string StrSpacingBSlices = GetPubElValByNumber(0x0018,0x0088);
-
-   if (StrSpacingBSlices == "gdcm::Unfound") {
-      dbg.Verbose(0, "gdcmHeader::GetZSpacing: unfound StrSpacingBSlices");
-      string StrSliceThickness = GetPubElValByNumber(0x0018,0x0050);       
-      if (StrSliceThickness == "gdcm::Unfound")
-         return 1.;
-      else
-         return atof(StrSliceThickness.c_str());  
-   } else {
-      return atof(StrSpacingBSlices.c_str());
-   }
+  * \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);  
 }
 
-//
-//  Image Position Patient :
-// If not found (AVR-NEMA), we consider Slice Location (20,1041)
-// or Location (20,50) as the Z coordinate, 
-// 0. for all the coordinates if Slice Location not found
-// TODO : find a way to inform the caller nothing was found
-// TODO : How to tell the caller a wrong number of values was found?
 /**
   * \ingroup gdcmHeader
-  * \brief
-  * @return
-  */
-float gdcmHeader::GetXImagePosition(void) {
-    float xImPos, yImPos, zImPos;
-    // 0020,0032 : Image Position Patient
-    // 0020,1041 : Slice Location
-    string StrImPos = GetPubElValByNumber(0x0020,0x0032);
-
-    if (StrImPos == "gdcm::Unfound") {
-       dbg.Verbose(0, "gdcmHeader::GetXImagePosition: unfound Image Position Patient (0020,0032)");
-       StrImPos = GetPubElValByNumber(0x0020,0x0030); // For ACR-NEMA images
-       if (StrImPos == "gdcm::Unfound") {
-          dbg.Verbose(0, "gdcmHeader::GetXImagePosition: unfound Image Position (RET) (0020,0030)");
-          string StrSliceLoc = GetPubElValByNumber(0x0020,0x1041); // for *very* old ACR-NEMA images
-          if (StrSliceLoc == "gdcm::Unfound") {
-            dbg.Verbose(0, "gdcmHeader::GetXImagePosition: unfound Slice Location (0020,1041)");
-             // How to tell the caller nothing was found?
-          } 
-       }  
-       return 0.;
-     }
-   if( sscanf( StrImPos.c_str(), "%f\\%f\\%f", &xImPos, &yImPos, &zImPos) != 3)
-     // How to tell the caller a wrong number of values was found?
-     return 0.;
-   //else
-   return xImPos;
+  * \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
-  * @return
-  */
-float gdcmHeader::GetYImagePosition(void) {
-    float xImPos, yImPos, zImPos;
-    // 0020,0032 : Image Position Patient
-    // 0020,1041 : Slice Location
-    // 0020,0050 : Location
-    string StrImPos = GetPubElValByNumber(0x0020,0x0032);
-
-    if (StrImPos == "gdcm::Unfound") {
-       dbg.Verbose(0, "gdcmHeader::GetYImagePosition: unfound Image Position Patient (0020,0032)");
-       StrImPos = GetPubElValByNumber(0x0020,0x0030); // For ACR-NEMA images
-       if (StrImPos == "gdcm::Unfound") {
-          dbg.Verbose(0, "gdcmHeader::GetYImagePosition: unfound Image Position (RET) (0020,0030)");
-          string StrSliceLoc = GetPubElValByNumber(0x0020,0x1041); // for *very* old ACR-NEMA images
-          if (StrSliceLoc == "gdcm::Unfound") {
-            dbg.Verbose(0, "gdcmHeader::GetYImagePosition: unfound Slice Location (0020,1041)");
-             // How to tell the caller nothing was found?
-          } 
-       }  
-       return 0.;
-     }
-   if( sscanf( StrImPos.c_str(), "%f\\%f\\%f", &xImPos, &yImPos, &zImPos) != 3)
-     // How to tell the caller a wrong number of values was found?
-     return 0.;
-   return yImPos;
+  * \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
-  */
-float gdcmHeader::GetZImagePosition(void) {
-   float xImPos, yImPos, zImPos;
-    // 0020,0032 : Image Position Patient
-    // 0020,1041 : Slice Location
-    // 0020,0050 : Location
-   
-   // TODO : How to tell the caller nothing was found?
-   // TODO : How to tell the caller a wrong number of values was found?
+  * \brief 
+  * @return Lookup Table RGB
+  * \       when (0028,0004),Photometric Interpretation = [PALETTE COLOR ]
+  * \        and (0028,1201),(0028,1202),(0028,1202) are found
+  * \warning : hazardous ! Use better GetPubElValVoidAreaByNumber
+  */ 
+void * gdcmHeader::GetLUTRGB(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");
+       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.
   
-   string StrImPos = GetPubElValByNumber(0x0020,0x0032);
-   if (StrImPos != "gdcm::Unfound") {
-      if( sscanf( StrImPos.c_str(), "%f\\%f\\%f", &xImPos, &yImPos, &zImPos) != 3) {
-         dbg.Verbose(0, "gdcmHeader::GetZImagePosition: wrong Image Position Patient (0020,0032)");
-         return 0.;  // bug in the element 0x0020,0x0032
-      } else {
-         return zImPos;
-      }    
-   }
+   l= GetLUTLength();  
+   if(l==0) 
+     return (NULL);     
+   int nBits=GetLUTNbits();
+  // a virer quand on aura trouve UNE image 
+  // qui correspond VRAIMENT à la definition !
+    cout << "l " << l << " nBits " << nBits;
    
-   StrImPos = GetPubElValByNumber(0x0020,0x0030); // For ACR-NEMA images
-   if (StrImPos != "gdcm::Unfound") {
-      if( sscanf( StrImPos.c_str(), "%f\\%f\\%f", &xImPos, &yImPos, &zImPos) != 3) {
-         dbg.Verbose(0, "gdcmHeader::GetZImagePosition: wrong Image Position (RET) (0020,0030)");
-         return 0.;  // bug in the element 0x0020,0x0032
-      } else {
-         return zImPos;
-      }    
-   }
-                 
-   string StrSliceLocation = GetPubElValByNumber(0x0020,0x1041);// for *very* old ACR-NEMA images
-   if (StrSliceLocation != "gdcm::Unfound") {
-      if( sscanf( StrSliceLocation.c_str(), "%f", &zImPos) !=1) {
-         dbg.Verbose(0, "gdcmHeader::GetZImagePosition: wrong Slice Location (0020,1041)");
-         return 0.;  // bug in the element 0x0020,0x1041
-      } else {
-         return zImPos;
-      }
-   }   
-   dbg.Verbose(0, "gdcmHeader::GetZImagePosition: unfound Slice Location (0020,1041)");
-
-   string StrLocation = GetPubElValByNumber(0x0020,0x0050);
-   if (StrLocation != "gdcm::Unfound") {
-      if( sscanf( StrLocation.c_str(), "%f", &zImPos) !=1) {
-         dbg.Verbose(0, "gdcmHeader::GetZImagePosition: wrong Location (0020,0050)");
-         return 0.;  // bug in the element 0x0020,0x0050
-      } else {
-         return zImPos;
+   l= l/(nBits/8);
+    
+   LutR =GetPubElValVoidAreaByNumber(0x0028,0x1201);
+   LutG =GetPubElValVoidAreaByNumber(0x0028,0x1202);
+   LutB =GetPubElValVoidAreaByNumber(0x0028,0x1203);
+   
+   // Warning : Any value for nBits  as to be considered as 8
+   //           Any value for Length as to be considered as 256
+   // That's DICOM ...
+   
+   // 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++;
       }
-   }
-   dbg.Verbose(0, "gdcmHeader::GetYImagePosition: unfound Slice Location");
+      return(LUTRGB); 
+   } else
    
-   return 0.; // Hopeless
+   */ {      // 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++) {
+      //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);   
+   } 
 }
-
-
-