X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=blobdiff_plain;f=src2%2FcreaImageIODicomImageReader.cpp;h=8a5863bf7413ad5fda0ad63521301573b53e29b5;hb=da2cb2feea7bdf425151ea3fd71abf9b8b72c366;hp=35075fb475510623e856e2c4e351dff8a3dd960d;hpb=96ca5a0d7b7ccd6b4291160c19afd48e3207db2a;p=creaImageIO.git

diff --git a/src2/creaImageIODicomImageReader.cpp b/src2/creaImageIODicomImageReader.cpp
index 35075fb..8a5863b 100644
--- a/src2/creaImageIODicomImageReader.cpp
+++ b/src2/creaImageIODicomImageReader.cpp
@@ -1,5 +1,5 @@
 #include <creaImageIODicomImageReader.h>
-#include <gdcmFile.h> 
+
 #include <vtkGdcmReader.h>
 #include <creaImageIOSystem.h>
 #include "boost/filesystem/path.hpp"
@@ -28,8 +28,7 @@ namespace creaImageIO
   //=====================================================================  
   bool DicomImageReader::CanRead(const std::string& filename)
   { 
-    
-	GDCM_NAME_SPACE::Document*doc;
+    GDCM_NAME_SPACE::Document*doc;
     GDCM_NAME_SPACE::File* file = GDCM_NAME_SPACE::File::New();
     file->SetLoadMode( GDCM_NAME_SPACE::LD_ALL);
     file->SetFileName(filename.c_str());
@@ -48,7 +47,6 @@ namespace creaImageIO
   //=====================================================================
   vtkImageData* DicomImageReader::ReadImage(const std::string& filename)
   {
-    
     vtkImageData* im = 0;
     try
       {
@@ -64,7 +62,7 @@ namespace creaImageIO
       }
     return im;
   }
-  
+
   //=====================================================================
   void DicomImageReader::PushBackExtensions(std::vector<std::string>& v)
   {
@@ -76,43 +74,45 @@ namespace creaImageIO
   //========================================================================
   std::string irclean(const std::string& str)
   {
-    if (str == "GDCM::Unfound") 
-      {
-	return "";
-      }
-    if (str[str.size()-1]==' ')
-      {
-	return str.substr(0,str.size()-1);
-      }
-    if (str[str.size()-1]==0)
-      {
-	return str.substr(0,str.size()-1);
-      }
+	  if(str.size() > 0)
+	  {
+		if (str == "GDCM::Unfound") 
+		  {
+		return "";
+		  }
+		if (str[str.size()-1]==' ')
+		  {
+		return str.substr(0,str.size()-1);
+		  }
+		if (str[str.size()-1]==0)
+		  {
+		return str.substr(0,str.size()-1);
+		  }
+	  }
     
     return str;
   }
   //========================================================================
-
+ 
   //=====================================================================
   void DicomImageReader::ReadAttributes(const std::string& filename, 
 		      std::map<std::string,std::string>& attr)
   {
- 
     GimmickMessage(2,"Reading attributes from DICOM file '"
 		   <<filename<<"'"<<std::endl);
-    
-	GDCM_NAME_SPACE::File* file = GDCM_NAME_SPACE::File::New();
 
-	GDCM_NAME_SPACE::Document *doc= GDCM_NAME_SPACE::File::New();
-	doc->SetLoadMode( GDCM_NAME_SPACE::LD_ALL);
+    //GDCM_NAME_SPACE::File* file = GDCM_NAME_SPACE::File::New();
+	boost::shared_ptr<GDCM_NAME_SPACE::File> file(GDCM_NAME_SPACE::File::New(), DicomImageReader::deleter());
+
+    GDCM_NAME_SPACE::Document *doc= GDCM_NAME_SPACE::File::New();
+    doc->SetLoadMode( GDCM_NAME_SPACE::LD_ALL);
     doc->SetFileName(filename.c_str());
     doc->Load();
     file->SetLoadMode( GDCM_NAME_SPACE::LD_ALL);
     file->SetFileName(filename.c_str());
     file->Load();
-    if (file->IsReadable() ||((GDCM_NAME_SPACE::Document*) file)->IsReadable())
+    if (file->IsReadable())// ||((GDCM_NAME_SPACE::Document*) file)->IsReadable())
       {
-	
 	std::map<std::string,std::string>::iterator i;
 	for (i=attr.begin();i!=attr.end();++i)
 	  {
@@ -134,20 +134,20 @@ namespace creaImageIO
 	      }
 	    else
 	      {
-		uint16_t gr;
 		uint16_t el;
+		uint16_t gr;
+		
 		tree::AttributeDescriptor::GetDicomGroupElementFromKey(i->first,gr,el);
 		if ( ( gr!=0 ) && ( el!=0 ) )
 		  {
-		    std::string val = file->GetEntryString(gr,el);
+		    std::string val =  file->GetEntryString(gr,el);
 		    i->second = irclean(val);
 		  }
 	      }
 	  }
       }
-    file->Delete();
   }
-  
+
   //=====================================================================
   
 } // namespace creaImageIO