From 85215aa078d5dde53cece0d5f38b53083efdef36 Mon Sep 17 00:00:00 2001
From: jpr <jpr>
Date: Wed, 22 Jun 2005 08:01:41 +0000
Subject: [PATCH] Comments

---
 src/gdcmDocument.cxx | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/src/gdcmDocument.cxx b/src/gdcmDocument.cxx
index e39d799b..af427024 100644
--- a/src/gdcmDocument.cxx
+++ b/src/gdcmDocument.cxx
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmDocument.cxx,v $
   Language:  C++
-  Date:      $Date: 2005/06/20 15:58:53 $
-  Version:   $Revision: 1.248 $
+  Date:      $Date: 2005/06/22 08:01:41 $
+  Version:   $Revision: 1.249 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -169,8 +169,12 @@ bool Document::Load( std::string const &fileName )
    std::string PhotometricInterpretation = GetEntryValue(0x0028,0x0004);   
    if( PhotometricInterpretation == "PALETTE COLOR " )
    {
-      LoadEntryBinArea(0x0028,0x1200);  // gray LUT   
-      /// FIXME FIXME FIXME
+   // FIXME
+   // Probabely this line should be outside the 'if'
+   // Try to find an image sample holding a 'gray LUT'
+      LoadEntryBinArea(0x0028,0x1200);  // gray LUT
+   
+      /// FIXME
       /// The tags refered by the three following lines used to be CORRECTLY
       /// defined as having an US Value Representation in the public
       /// dictionary. BUT the semantics implied by the three following
@@ -202,7 +206,7 @@ bool Document::Load( std::string const &fileName )
    if ( modLutSeq !=0 )
    {
       SQItem *sqi= modLutSeq->GetFirstSQItem();
-      if ( !sqi )
+      if ( sqi != 0 )
       {
          BinEntry *b = sqi->GetBinEntry(0x0028,0x3006);
          if ( b != 0 )
-- 
2.46.1