]> Creatis software - gdcm.git/blobdiff - src/gdcmFile.cxx
BUG: JP suggest me to do this change
[gdcm.git] / src / gdcmFile.cxx
index 7b46a51a54377b27084643ee151a40769deee28c..0777c3d7c07c80cc721218171f4a2384b57ce781 100644 (file)
@@ -43,6 +43,16 @@ gdcmFile::gdcmFile(gdcmHeader *header) {
  *        one sets an a posteriori shadow dictionary (efficiency can be
  *        seen as a side effect).   
  * @param filename file to be opened for parsing
+ * @param   exception_on_error whether we throw an exception or not
+ * @param   enable_sequences = true to allow the header 
+ *          to be parsed *inside* the SeQuences, 
+ *          when they have an actual length 
+ * \warning enable_sequences *has to be* true for reading PAPYRUS 3.0 files
+ * @param   ignore_shadow to allow skipping the shadow elements, 
+ *          to save memory space.
+ * \warning The TRUE value for this param has to be used 
+ *          with a FALSE value for the 'enable_sequence' param.
+ *          ('public elements' may be embedded in 'shadow Sequences')
  */
 gdcmFile::gdcmFile(std::string & filename, 
                    bool exception_on_error,
@@ -71,8 +81,18 @@ gdcmFile::gdcmFile(std::string & filename,
  *        one sets an a posteriori shadow dictionary (efficiency can be
  *        seen as a side effect).   
  * @param filename file to be opened for parsing
+ * @param   exception_on_error whether we throw an exception or not
+ * @param   enable_sequences = true to allow the header 
+ *          to be parsed *inside* the SeQuences, 
+ *          when they have an actual length 
+ * \warning enable_sequences *has to be* true for reading PAPYRUS 3.0 files
+ * @param   ignore_shadow to allow skipping the shadow elements, 
+ *          to save memory space.
+ * \warning The TRUE value for this param has to be used 
+ *          with a FALSE value for the 'enable_sequence' param.
+ *          ('public elements' may be embedded in 'shadow Sequences')
  */
- gdcmFile::gdcmFile(const char * filename, 
+ gdcmFile::gdcmFile(const char *filename, 
                    bool exception_on_error,
                    bool enable_sequences, 
                    bool ignore_shadow) {
@@ -204,7 +224,7 @@ size_t gdcmFile::GetImageDataSizeRaw(void) {
  *          NULL if alloc fails 
  */
 void * gdcmFile::GetImageData (void) {
-   PixelData = (void *) malloc(lgrTotale);
+   PixelData = new char[lgrTotale];
    if (PixelData)
       GetImageDataIntoVector(PixelData, lgrTotale);
       
@@ -234,15 +254,16 @@ void * gdcmFile::GetImageData (void) {
  *          failure e.g. MaxSize is lower than necessary.
  */
 size_t gdcmFile::GetImageDataIntoVector (void* destination, size_t MaxSize) {
-   size_t l = GetImageDataIntoVectorRaw (destination, MaxSize);
+   //size_t l = GetImageDataIntoVectorRaw (destination, MaxSize);
+        GetImageDataIntoVectorRaw (destination, MaxSize);
    PixelRead=0 ; // no PixelRaw
    if (!Header->HasLUT())
       return lgrTotale; 
                             
    // from Lut R + Lut G + Lut B
-   unsigned char * newDest = (unsigned char *)malloc(lgrTotale);
-   unsigned char * a       = (unsigned char *)destination;      
-   unsigned char * lutRGBA =                  Header->GetLUTRGBA();
+   unsigned char *newDest = new unsigned char[lgrTotale];
+   unsigned char *a       = (unsigned char *)destination;       
+   unsigned char *lutRGBA =                  Header->GetLUTRGBA();
    if (lutRGBA) {          
       int l = lgrTotaleRaw;
       memmove(newDest, destination, l);// move Gray pixels to temp area            
@@ -253,7 +274,7 @@ size_t gdcmFile::GetImageDataIntoVector (void* destination, size_t MaxSize) {
          *a++ = lutRGBA[j+1];
          *a++ = lutRGBA[j+2];
       }
-      free(newDest);
+      delete[] newDest;
     
    // now, it's an RGB image
    // Lets's write it in the Header
@@ -281,7 +302,7 @@ size_t gdcmFile::GetImageDataIntoVector (void* destination, size_t MaxSize) {
       std::string rgb= "MONOCHROME1 ";      // Photometric Interpretation
       Header->SetEntryByNumber(rgb,0x0028,0x0004);                                
    }            
-   // TODO : Drop Palette Color out of the Header?          
+   /// \todo Drop Palette Color out of the Header?          
    return lgrTotale; 
 }
 
@@ -297,10 +318,10 @@ size_t gdcmFile::GetImageDataIntoVector (void* destination, size_t MaxSize) {
  */
 void * gdcmFile::GetImageDataRaw (void) {
    if (Header->HasLUT())
-      lgrTotale /= 3;  // TODO Let gdcmHeadar user a chance 
-                       // to get the right value
-                      // Create a member lgrTotaleRaw ???
-   PixelData = (void *) malloc(lgrTotale);
+      /// \todo Let gdcmHeadar user a chance to get the right value
+               /// Create a member lgrTotaleRaw ???
+      lgrTotale /= 3;
+   PixelData = new char[lgrTotale];
    if (PixelData)
       GetImageDataIntoVectorRaw(PixelData, lgrTotale);
    PixelRead=1; // PixelRaw
@@ -331,7 +352,7 @@ void * gdcmFile::GetImageDataRaw (void) {
  * @return  On success, the number of bytes actually copied. Zero on
  *          failure e.g. MaxSize is lower than necessary.
  */
-size_t gdcmFile::GetImageDataIntoVectorRaw (voiddestination, size_t MaxSize) {
+size_t gdcmFile::GetImageDataIntoVectorRaw (void *destination, size_t MaxSize) {
 
    int nb, nbu, highBit, signe;
    std::string str_nbFrames, str_nb, str_nbu, str_highBit, str_signe;
@@ -471,17 +492,17 @@ size_t gdcmFile::GetImageDataIntoVectorRaw (void* destination, size_t MaxSize) {
             int l = Header->GetXSize()*Header->GetYSize();
             int nbFrames = Header->GetZSize();
 
-            unsigned char * newDest = (unsigned char*) malloc(lgrTotale);
-            unsigned char *x  = newDest;
-            unsigned char * a = (unsigned char *)destination;
-            unsigned char * b = a + l;
-            unsigned char * c = b + l;
+            unsigned char *newDest = new unsigned char[lgrTotale];
+            unsigned char *x = newDest;
+            unsigned char *a = (unsigned char *)destination;
+            unsigned char *b = a + l;
+            unsigned char *c = b + l;
             double R,G,B;
 
-            // TODO : Replace by the 'well known' 
-            //        integer computation counterpart
-           // see http://lestourtereaux.free.fr/papers/data/yuvrgb.pdf
-            // for code optimisation
+            /// \todo : Replace by the 'well known' integer computation
+            /// counterpart
+                /// see http://lestourtereaux.free.fr/papers/data/yuvrgb.pdf
+            /// for code optimisation
            
             for (int i=0;i<nbFrames;i++) {
                for (int j=0;j<l; j++) {
@@ -503,7 +524,7 @@ size_t gdcmFile::GetImageDataIntoVectorRaw (void* destination, size_t MaxSize) {
                }
            }
             memmove(destination,newDest,lgrTotale);
-            free(newDest);
+            delete[] newDest;
 
         } else {
          
@@ -512,11 +533,11 @@ size_t gdcmFile::GetImageDataIntoVectorRaw (void* destination, size_t MaxSize) {
 
             int l = Header->GetXSize()*Header->GetYSize()*Header->GetZSize();
 
-            char * newDest = (char*) malloc(lgrTotale);
-            char * x = newDest;
-            char * a = (char *)destination;
-            char * b = a + l;
-            char * c = b + l;
+            char *newDest = new char[lgrTotale];
+            char *x = newDest;
+            char *a = (char *)destination;
+            char *b = a + l;
+            char *c = b + l;
 
             for (int j=0;j<l; j++) {
                *(x++) = *(a++);
@@ -524,7 +545,7 @@ size_t gdcmFile::GetImageDataIntoVectorRaw (void* destination, size_t MaxSize) {
                *(x++) = *(c++);  
             }           
             memmove(destination,newDest,lgrTotale);
-            free(newDest);
+            delete[] newDest;
         }        
          break;
        }     
@@ -546,7 +567,7 @@ size_t gdcmFile::GetImageDataIntoVectorRaw (void* destination, size_t MaxSize) {
    std::string planConfig = "0"; // Planar Configuration
    Header->SetEntryByNumber(planConfig,0x0028,0x0006);
         
-        // TODO : Drop Palette Color out of the Header? 
+        /// \todo Drop Palette Color out of the Header? 
    return lgrTotale; 
 }
 
@@ -562,10 +583,11 @@ size_t gdcmFile::GetImageDataIntoVectorRaw (void* destination, size_t MaxSize) {
  *
  * @return boolean     
  */
-bool gdcmFile::SetImageData(void * inData, size_t ExpectedSize) {
+bool gdcmFile::SetImageData(void *inData, size_t ExpectedSize) {
    Header->SetImageDataSize(ExpectedSize);
    PixelData = inData;
    lgrTotale = ExpectedSize;
+   PixelRead = 1;
    return(true);
 }
 
@@ -580,7 +602,7 @@ bool gdcmFile::SetImageData(void * inData, size_t ExpectedSize) {
  */
 
 bool gdcmFile::WriteRawData (std::string fileName) {
-   FILE * fp1;
+   FILE *fp1;
    fp1 = fopen(fileName.c_str(),"wb");
    if (fp1 == NULL) {
       printf("Fail to open (write) file [%s] \n",fileName.c_str());
@@ -602,7 +624,7 @@ bool gdcmFile::WriteRawData (std::string fileName) {
  */
 
 bool gdcmFile::WriteDcmImplVR (std::string fileName) {
-   return WriteBase(fileName, ImplicitVR);
+   return WriteBase(fileName, gdcmImplicitVR);
 }
 
 /**
@@ -614,7 +636,7 @@ bool gdcmFile::WriteDcmImplVR (std::string fileName) {
  * @return false if write fails        
  */
  
-bool gdcmFile::WriteDcmImplVR (const charfileName) {
+bool gdcmFile::WriteDcmImplVR (const char *fileName) {
    return WriteDcmImplVR (std::string (fileName));
 }
        
@@ -628,7 +650,7 @@ bool gdcmFile::WriteDcmImplVR (const char* fileName) {
  */
 
 bool gdcmFile::WriteDcmExplVR (std::string fileName) {
-   return WriteBase(fileName, ExplicitVR);
+   return WriteBase(fileName, gdcmExplicitVR);
 }
        
 /**
@@ -647,7 +669,7 @@ bool gdcmFile::WriteDcmExplVR (std::string fileName) {
  */
 
 bool gdcmFile::WriteAcr (std::string fileName) {
-   return WriteBase(fileName, ACR);
+   return WriteBase(fileName, gdcmACR);
 }
 
 //-----------------------------------------------------------------------------
@@ -663,9 +685,9 @@ bool gdcmFile::WriteAcr (std::string fileName) {
  */
 bool gdcmFile::WriteBase (std::string fileName, FileType type) {
 
-   FILE * fp1;
+   FILE *fp1;
    
-   if (PixelRead==-1 && type != ExplicitVR) {
+   if (PixelRead==-1 && type != gdcmExplicitVR) {
       return false;               
    }
 
@@ -675,13 +697,14 @@ bool gdcmFile::WriteBase (std::string fileName, FileType type) {
       return (false);
    }
 
-   if ( (type == ImplicitVR) || (type == ExplicitVR) ) {
-      char * filePreamble;
+   if ( (type == gdcmImplicitVR) || (type == gdcmExplicitVR) ) {
+      char *filePreamble;
       // writing Dicom File Preamble
-      filePreamble=(char*)calloc(128,1);
+      filePreamble=new char[128];
+      memset(filePreamble,0,128);
       fwrite(filePreamble,128,1,fp1);
       fwrite("DICM",4,1,fp1);
-      free (filePreamble);
+      delete[] filePreamble;
    }
 
    // --------------------------------------------------------------
@@ -691,11 +714,11 @@ bool gdcmFile::WriteBase (std::string fileName, FileType type) {
    // we reproduce on disk the switch between lineNumber and columnNumber
    // just before writting ...
    
-   // TODO : the best trick would be *change* the recognition code
-   //        but pb expected if user deals with, e.g. COMPLEX images
+   /// \todo the best trick would be *change* the recognition code
+   ///       but pb expected if user deals with, e.g. COMPLEX images
 
    std::string rows, columns; 
-   if ( Header->GetFileType() == ACR_LIBIDO){
+   if ( Header->GetFileType() == gdcmACR_LIBIDO){
          rows    = Header->GetEntryByNumber(0x0028, 0x0010);
          columns = Header->GetEntryByNumber(0x0028, 0x0011);
          Header->SetEntryByNumber(columns,  0x0028, 0x0010);
@@ -703,7 +726,7 @@ bool gdcmFile::WriteBase (std::string fileName, FileType type) {
    }   
    // ----------------- End of Special Patch ----------------
    
-   // TODO : get the grPixel, numPixel values (for some ACR-NEMA images only)
+   /// \todo get the grPixel, numPixel values (for some ACR-NEMA images only)
    
    guint16 grPixel =Header->GetGrPixel();
    guint16 numPixel=Header->GetNumPixel();;
@@ -713,7 +736,7 @@ bool gdcmFile::WriteBase (std::string fileName, FileType type) {
           
    TagKey key = gdcmDictEntry::TranslateToKey(grPixel, numPixel); 
    TagHeaderEntryHT::iterator p2;
-   gdcmHeaderEntry * PixelElement;
+   gdcmHeaderEntry *PixelElement;
    
    IterHT it= Header->GetEntry().equal_range(key); // get a pair of iterators first-last synonym   
 
@@ -741,7 +764,7 @@ bool gdcmFile::WriteBase (std::string fileName, FileType type) {
    // ...and we restore the Header to be Dicom Compliant again 
    // just after writting
 
-   if (Header->GetFileType() == ACR_LIBIDO){
+   if (Header->GetFileType() == gdcmACR_LIBIDO){
          Header->SetEntryByNumber(rows   , 0x0028, 0x0010);
          Header->SetEntryByNumber(columns, 0x0028, 0x0011);
    }   
@@ -763,7 +786,7 @@ bool gdcmFile::WriteBase (std::string fileName, FileType type) {
  * @param   lgr Area Length
  * @param   nb Pixels Bit number 
  */
-void gdcmFile::SwapZone(voidim, int swap, int lgr, int nb) {
+void gdcmFile::SwapZone(void *im, int swap, int lgr, int nb) {
 guint32 s32;
 guint16 fort,faible;
 int i;
@@ -781,8 +804,8 @@ if(nb == 16)
       case 4321:
 
          for(i=0;i<lgr/2;i++) {
-            ((unsigned short int*)im)[i]= ((((unsigned short int*)im)[i])>>8)
-                                        | ((((unsigned short int*)im)[i])<<8);
+            ((unsigned short int *)im)[i]= ((((unsigned short int *)im)[i])>>8)
+                                        | ((((unsigned short int *)im)[i])<<8);
        }
          break;
                        
@@ -798,32 +821,32 @@ if( nb == 32 )
 
       case 4321:
          for(i=0;i<lgr/4;i++) {
-            faible=  ((unsigned long int*)im)[i]&0x0000ffff;    /* 4321 */
-            fort  =((unsigned long int*)im)[i]>>16;
+            faible=  ((unsigned long int *)im)[i]&0x0000ffff;    /* 4321 */
+            fort  =((unsigned long int *)im)[i]>>16;
             fort=  (fort>>8)   | (fort<<8);
             faible=(faible>>8) | (faible<<8);
             s32=faible;
-            ((unsigned long int*)im)[i]=(s32<<16)|fort;
+            ((unsigned long int *)im)[i]=(s32<<16)|fort;
          }
          break;
 
       case 2143:
          for(i=0;i<lgr/4;i++) {
-            faible=  ((unsigned long int*)im)[i]&0x0000ffff;    /* 2143 */
-            fort=((unsigned long int*)im)[i]>>16;
+            faible=  ((unsigned long int *)im)[i]&0x0000ffff;    /* 2143 */
+            fort=((unsigned long int *)im)[i]>>16;
             fort=  (fort>>8)   | (fort<<8);
             faible=(faible>>8) | (faible<<8);
             s32=fort; 
-            ((unsigned long int*)im)[i]=(s32<<16)|faible;
+            ((unsigned long int *)im)[i]=(s32<<16)|faible;
          }
          break;
   
       case 3412:
          for(i=0;i<lgr/4;i++) {
-            faible=  ((unsigned long int*)im)[i]&0x0000ffff;    /* 3412 */
-            fort=((unsigned long int*)im)[i]>>16;                  
+            faible=  ((unsigned long int *)im)[i]&0x0000ffff;    /* 3412 */
+            fort=((unsigned long int *)im)[i]>>16;                  
             s32=faible; 
-            ((unsigned long int*)im)[i]=(s32<<16)|fort;
+            ((unsigned long int *)im)[i]=(s32<<16)|fort;
          }                 
          break; 
                                
@@ -840,7 +863,7 @@ return;
  * @param   destination where the pixel data should be stored.
  *
  */
-bool gdcmFile::ReadPixelData(voiddestination) {
+bool gdcmFile::ReadPixelData(void *destination) {
 
    FILE *fp;
 
@@ -859,7 +882,7 @@ bool gdcmFile::ReadPixelData(void* destination) {
       int nbPixels = Header->GetXSize() * Header->GetYSize();
       unsigned char b0, b1, b2;
       
-      unsigned short intpdestination = (unsigned short int*)destination;    
+      unsigned short int *pdestination = (unsigned short int*)destination;    
       for(int p=0;p<nbPixels;p+=2) {
          fread(&b0,1,1,fp);
          fread(&b1,1,1,fp);
@@ -938,7 +961,7 @@ bool gdcmFile::ReadPixelData(void* destination) {
       
    if (ln != 0) {
       // What is it used for ?!?
-      char *BasicOffsetTableItemValue = (char *)malloc(ln+1);        
+      char *BasicOffsetTableItemValue = new char[ln+1];
       fread(BasicOffsetTableItemValue,ln,1,fp); 
    }
    
@@ -970,7 +993,7 @@ bool gdcmFile::ReadPixelData(void* destination) {
                   
          JPEGLosslessDecodeImage (fp,  // Reading Fragment pixels
                                     (unsigned short *)destination,
-                                    Header->GetPixelSize()*8* Header->GetSamplesPerPixel(),
+                                    Header->GetPixelSize() * 8 * Header->GetSamplesPerPixel(),
                                      ln);                                                         
          res=1; // in order not to break the loop