]> Creatis software - gdcm.git/commitdiff
Fix typo
authorjpr <jpr>
Thu, 3 Mar 2005 11:39:24 +0000 (11:39 +0000)
committerjpr <jpr>
Thu, 3 Mar 2005 11:39:24 +0000 (11:39 +0000)
Add comments
Remove now useless stuff

vtk/vtkGdcmReader.cxx
vtk/vtkGdcmWriter.cxx
vtk/vtkGdcmWriter.h

index 6db14bad8d58cf810a6740d8d0a34b4e337357e9..983dd3e4e9dd7c3a2442ec7a898c6fa2747f699c 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: vtkGdcmReader.cxx,v $
   Language:  C++
-  Date:      $Date: 2005/01/21 11:40:56 $
-  Version:   $Revision: 1.66 $
+  Date:      $Date: 2005/03/03 11:39:24 $
+  Version:   $Revision: 1.67 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -75,7 +75,7 @@
 #include <vtkPointData.h>
 #include <vtkLookupTable.h>
 
-vtkCxxRevisionMacro(vtkGdcmReader, "$Revision: 1.66 $");
+vtkCxxRevisionMacro(vtkGdcmReader, "$Revision: 1.67 $");
 vtkStandardNewMacro(vtkGdcmReader);
 
 //-----------------------------------------------------------------------------
@@ -226,7 +226,6 @@ void vtkGdcmReader::ExecuteInformation()
       {
          vtkDebugMacro(<< "16 bits signed image");
          this->SetDataScalarTypeToShort();
-         //vtkErrorMacro(<< "Cannot handle 16 bit signed files");
       }
       else if ( ImageType == "32U" )
       {
@@ -257,7 +256,7 @@ void vtkGdcmReader::ExecuteInformation()
  * Update => ouput->Update => UpdateData => Execute => ExecuteData 
  * (see vtkSource.cxx for last step).
  * This function (redefinition of vtkImageReader::ExecuteData, see 
- * VTK/IO/vtkImageReader.cxx) reads a data from a file. The datas
+ * VTK/IO/vtkImageReader.cxx) reads a data from a file. The data
  * extent/axes are assumed to be the same as the file extent/order.
  */
 void vtkGdcmReader::ExecuteData(vtkDataObject *output)
@@ -533,8 +532,8 @@ int vtkGdcmReader::CheckFileCoherence()
       } 
       else 
       {
-         // We didn't have a workable reference file yet. Set this one
-         // as the reference.
+         // We didn't have a workable reference file yet. 
+         // Set this one as the reference.
          FoundReferenceFile = true;
          vtkDebugMacro(<< "This file taken as coherence reference:"
                         << filename->c_str());
index 7ffdc082789d5b4f30dee32d0355c21970262276..a9259ee74ec8f53846eecaed723f99b673a33f7f 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: vtkGdcmWriter.cxx,v $
   Language:  C++
-  Date:      $Date: 2005/03/01 16:12:20 $
-  Version:   $Revision: 1.18 $
+  Date:      $Date: 2005/03/03 11:39:24 $
+  Version:   $Revision: 1.19 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -27,7 +27,7 @@
 #include <vtkPointData.h>
 #include <vtkLookupTable.h>
 
-vtkCxxRevisionMacro(vtkGdcmWriter, "$Revision: 1.18 $");
+vtkCxxRevisionMacro(vtkGdcmWriter, "$Revision: 1.19 $");
 vtkStandardNewMacro(vtkGdcmWriter);
 
 //-----------------------------------------------------------------------------
@@ -37,11 +37,6 @@ vtkGdcmWriter::vtkGdcmWriter()
    this->LookupTable = NULL;
    this->FileDimensionality = 3;
    this->WriteType = VTK_GDCM_WRITE_TYPE_EXPLICIT_VR;
-
-   UIDPrefix = "";
-   StudyInstanceUID = "";
-   SeriesInstanceUID = "";
-   FrameOfReferenceInstanceUID = "";
 }
 
 vtkGdcmWriter::~vtkGdcmWriter()
@@ -50,9 +45,9 @@ vtkGdcmWriter::~vtkGdcmWriter()
 
 //-----------------------------------------------------------------------------
 // Print
-void vtkGdcmWriter::PrintSelf(ostreamos, vtkIndent indent)
+void vtkGdcmWriter::PrintSelf(ostream &os, vtkIndent indent)
 {
-   this->Superclass::PrintSelf(os,indent);
+   this->Superclass::PrintSelf(os, indent);
 
    os << indent << "Write type : " << this->GetWriteTypeAsString();
 }
@@ -76,37 +71,12 @@ const char *vtkGdcmWriter::GetWriteTypeAsString()
    }
 }
 
-void vtkGdcmWriter::SetUIDPrefix(const char *prefix)
-{
-   UIDPrefix = prefix;
-}
-
-const char *vtkGdcmWriter::GetUIDPrefix()
-{
-   return UIDPrefix.c_str();
-}
-
-void vtkGdcmWriter::NewStudyInstanceUID()
-{
-   StudyInstanceUID = "";
-}
-
-void vtkGdcmWriter::NewSeriesInstanceUID()
-{
-   SeriesInstanceUID = "";
-}
-
-void vtkGdcmWriter::NewFrameOfReferenceInstanceUID()
-{
-   FrameOfReferenceInstanceUID = "";
-}
-
 //-----------------------------------------------------------------------------
 // Protected
 /**
  * Copy the image and reverse the Y axis
  */
-// The output datas must be deleted by the user of the method !!!
+// The output data must be deleted by the user of the method !!!
 size_t ReverseData(vtkImageData *image,unsigned char **data)
 {
    int inc[3];
@@ -149,9 +119,9 @@ size_t ReverseData(vtkImageData *image,unsigned char **data)
 }
 
 /**
- * Set the datas informations in the file
+ * Set the data informations in the file
  */
-void SetImageInformation(gdcm::FileHelper *file,vtkImageData *image)
+void SetImageInformation(gdcm::FileHelper *file, vtkImageData *image)
 {
    std::ostringstream str;
 
@@ -209,6 +179,9 @@ void SetImageInformation(gdcm::FileHelper *file,vtkImageData *image)
    str << image->GetNumberOfScalarComponents();
    file->InsertValEntry(str.str(),0x0028,0x0002); // Samples per Pixel
 
+   /// \todo : Spacing Between Slices is meaningfull ONLY for CT an MR modality
+   ///       We should perform some checkings before forcing the Entry creation
+
    // Spacing
    double *sp = image->GetSpacing();
 
@@ -225,6 +198,9 @@ void SetImageInformation(gdcm::FileHelper *file,vtkImageData *image)
    // Origin
    double *org = image->GetOrigin();
 
+   /// \todo : Image Position Patient is meaningfull ONLY for CT an MR modality
+   ///       We should perform some checkings before forcing the Entry creation
+
    str.seekp(0);
    str << org[0] << "\\" << org[1] << "\\" << org[2];
    file->InsertValEntry(str.str(),0x0020,0x0032); // Image Position Patient
@@ -250,24 +226,25 @@ void SetImageInformation(gdcm::FileHelper *file,vtkImageData *image)
  * Write of the files
  * The call to this method is recursive if there is some files to write
  */ 
-void vtkGdcmWriter::RecursiveWrite(int axis, vtkImageData *image, ofstream *file)
+void vtkGdcmWriter::RecursiveWrite(int axis, vtkImageData *image, 
+                    ofstream *file)
 {
    if(file)
    {
-      vtkErrorMacro( <<  "File musn't be opened");
+      vtkErrorMacro( <<  "File must not be open");
       return;
    }
 
    if( image->GetScalarType() == VTK_FLOAT || 
        image->GetScalarType() == VTK_DOUBLE )
    {
-      vtkErrorMacro(<< "Bad input type. Scalar type musn't be of type "
+      vtkErrorMacro(<< "Bad input type. Scalar type must not be of type "
                     << "VTK_FLOAT or VTKDOUBLE (found:"
                     << image->GetScalarTypeAsString());
       return;
    }
 
-   RecursiveWrite(axis,image,image,file);
+   RecursiveWrite(axis,image, image, file);
    //WriteDcmFile(this->FileName,image);
 }
 
@@ -279,7 +256,7 @@ void vtkGdcmWriter::RecursiveWrite(int axis, vtkImageData *cache,
    // if the file is already open then just write to it
    if( file )
    {
-      vtkErrorMacro( <<  "File musn't be opened");
+      vtkErrorMacro( <<  "File musn't be open");
       return;
    }
 
@@ -289,7 +266,7 @@ void vtkGdcmWriter::RecursiveWrite(int axis, vtkImageData *cache,
       // determine the name
       if (this->FileName)
       {
-         sprintf(this->InternalFileName,"%s",this->FileName);
+         sprintf(this->InternalFileName, "%s", this->FileName);
       }
       else 
       {
@@ -318,7 +295,7 @@ void vtkGdcmWriter::RecursiveWrite(int axis, vtkImageData *cache,
       return;
    }
 
-   // if the current region is too high a dimension forthe file
+   // if the current region is too high a dimension for the file
    // the we will split the current axis
    cache->GetAxisUpdateExtent(axis, min, max);
 
@@ -344,28 +321,13 @@ void vtkGdcmWriter::RecursiveWrite(int axis, vtkImageData *cache,
    cache->SetAxisUpdateExtent(axis, min, max);
 }
 
-void vtkGdcmWriter::WriteDcmFile(char *fileName,vtkImageData *image)
+void vtkGdcmWriter::WriteDcmFile(char *fileName, vtkImageData *image)
 {
    // From here, the write of the file begins
    gdcm::FileHelper *dcmFile = new gdcm::FileHelper();
 
-   // Set the image UID
-   if( StudyInstanceUID.empty() )
-      StudyInstanceUID = gdcm::Util::CreateUniqueUID( UIDPrefix );
-   if( SeriesInstanceUID.empty() )
-      SeriesInstanceUID = gdcm::Util::CreateUniqueUID( UIDPrefix );
-   if( FrameOfReferenceInstanceUID.empty() )
-      FrameOfReferenceInstanceUID = gdcm::Util::CreateUniqueUID( UIDPrefix );
-   std::string uid = gdcm::Util::CreateUniqueUID( UIDPrefix );
-
-   dcmFile->InsertValEntry(uid,0x0008,0x0018); //[SOP Instance UID]
-   dcmFile->InsertValEntry(uid,0x0002,0x0003); //[Media Stored SOP Instance UID]
-   dcmFile->InsertValEntry(StudyInstanceUID,0x0020,0x000d); //[Study Instance UID]
-   dcmFile->InsertValEntry(SeriesInstanceUID,0x0020,0x000e); //[Series Instance UID]
-   dcmFile->InsertValEntry(FrameOfReferenceInstanceUID,0x0020, 0x0052); //[Frame of Reference UID] 
-
    // Set the image informations
-   SetImageInformation(dcmFile,image);
+   SetImageInformation(dcmFile, image);
 
    // Write the image
    switch(this->WriteType)
@@ -388,7 +350,7 @@ void vtkGdcmWriter::WriteDcmFile(char *fileName,vtkImageData *image)
 
    if(!dcmFile->Write(fileName))
    {
-      vtkErrorMacro( << "File "  <<  this->FileName  <<  "couldn't be written by "
+      vtkErrorMacro( << "File "  <<  this->FileName  <<  "cannot be written by "
                      << " the gdcm library");
    }
 
index 3de9c069d8ad371a6be3e1bb1004ab7f1cac8434..f5b18e9715b54c3e1d0487ae0d29b8c5165ed7cf 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: vtkGdcmWriter.h,v $
   Language:  C++
-  Date:      $Date: 2005/01/28 10:07:35 $
-  Version:   $Revision: 1.5 $
+  Date:      $Date: 2005/03/03 11:39:24 $
+  Version:   $Revision: 1.6 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -38,26 +38,19 @@ public:
    static vtkGdcmWriter *New();
    vtkTypeRevisionMacro(vtkGdcmWriter, vtkImageWriter);
 
-   void PrintSelf(ostreamos, vtkIndent indent);
+   void PrintSelf(ostream &os, vtkIndent indent);
 
-   vtkSetObjectMacro(LookupTable,vtkLookupTable);
-   vtkGetObjectMacro(LookupTable,vtkLookupTable);
+   vtkSetObjectMacro(LookupTable, vtkLookupTable);
+   vtkGetObjectMacro(LookupTable, vtkLookupTable);
 
-   void SetWriteTypeToDcmImplVR()     { SetWriteType(VTK_GDCM_WRITE_TYPE_EXPLICIT_VR); };
-   void SetWriteTypeToDcmExplVR()     { SetWriteType(VTK_GDCM_WRITE_TYPE_IMPLICIT_VR); };
-   void SetWriteTypeToAcr()           { SetWriteType(VTK_GDCM_WRITE_TYPE_ACR); };
-   void SetWriteTypeToAcrLibido()     { SetWriteType(VTK_GDCM_WRITE_TYPE_ACR_LIBIDO); };
-   vtkSetMacro(WriteType,int);
-   vtkGetMacro(WriteType,int);
+   void SetWriteTypeToDcmImplVR(){SetWriteType(VTK_GDCM_WRITE_TYPE_EXPLICIT_VR);};
+   void SetWriteTypeToDcmExplVR(){SetWriteType(VTK_GDCM_WRITE_TYPE_IMPLICIT_VR);};
+   void SetWriteTypeToAcr()      {SetWriteType(VTK_GDCM_WRITE_TYPE_ACR);        };
+   void SetWriteTypeToAcrLibido(){SetWriteType(VTK_GDCM_WRITE_TYPE_ACR_LIBIDO); };
+   vtkSetMacro(WriteType, int);
+   vtkGetMacro(WriteType, int);
    const char *GetWriteTypeAsString();
 
-   void SetUIDPrefix(const char *prefix);
-   const char *GetUIDPrefix();
-
-   void NewStudyInstanceUID();
-   void NewSeriesInstanceUID();
-   void NewFrameOfReferenceInstanceUID();
-
 protected:
    vtkGdcmWriter();
    ~vtkGdcmWriter();
@@ -65,19 +58,12 @@ protected:
   virtual void RecursiveWrite(int axis, vtkImageData *image, ofstream *file);
   virtual void RecursiveWrite(int axis, vtkImageData *image, 
                               vtkImageData *cache, ofstream *file);
-  void WriteDcmFile(char *fileName,vtkImageData *image);
+  void WriteDcmFile(char *fileName, vtkImageData *image);
 
 private:
 // Variables
    vtkLookupTable *LookupTable;
    int WriteType;
-
-   //BTX
-   std::string UIDPrefix;
-   std::string StudyInstanceUID;
-   std::string SeriesInstanceUID;
-   std::string FrameOfReferenceInstanceUID;
-   //ETX
 };
 
 //-----------------------------------------------------------------------------