From: malaterre <malaterre>
Date: Thu, 2 Sep 2004 04:35:01 +0000 (+0000)
Subject: ENH: minor tweaks
X-Git-Tag: Version0.6.bp~232
X-Git-Url: https://git.creatis.insa-lyon.fr/pubgit/?a=commitdiff_plain;ds=sidebyside;h=05a8ef5406eb1d78b0a43a714b41656a297a7f3c;p=gdcm.git

ENH: minor tweaks
---

diff --git a/Testing/TestChangeHeader.cxx b/Testing/TestChangeHeader.cxx
index 989b55fc..819d70b7 100644
--- a/Testing/TestChangeHeader.cxx
+++ b/Testing/TestChangeHeader.cxx
@@ -1,5 +1,3 @@
-#include <iostream>
-//#include "gdcm.h"
 #include "gdcmHeader.h"
 #include "gdcmFile.h"
 #include <string>
@@ -9,40 +7,35 @@
 // et des pixels d'une autre image
 
 
-int TestChangeHeader(int argc, char* argv[]) {  
-   std::string premier, deuxieme;
-   char resultat[200];
-
-   gdcmFile  *f1, *f2;
-//gdcmHeader *e1, *e2;
-   void* imageData;
-   int dataSize;
-
-   if (argc < 3) {
+int TestChangeHeader(int argc, char* argv[])
+{
+   if (argc < 3)
+   {
       std::cerr << "usage :" << std::endl << 
       argv[0] << " nomFichierPourEntete nomFichierPourDonnées" << std::endl;
-      return 0;
+      return 1;
    }
 
-   premier = argv[1];
-   f1 = new gdcmFile(premier);
+   std::string premier = argv[1];
+   gdcmFile  *f1 = new gdcmFile(premier);
 
-   deuxieme = argv[2];
-   f2 = new gdcmFile(deuxieme);
+   std::string deuxieme = argv[2];
+   gdcmFile  *f2 = new gdcmFile(deuxieme);
 
-//f1->PrintPubElVal();
+   //f1->PrintPubElVal();
 
-// On suppose que les champs DICOM du 2ieme fichier existent *effectivement*
+   // On suppose que les champs DICOM du 2ieme fichier existent *effectivement*
 
    std::string nbFrames = f2->GetHeader()->GetEntryByNumber(0x0028, 0x0008);
-   if(nbFrames != "gdcm::Unfound") {
+   if(nbFrames != "gdcm::Unfound")
+   {
       f1->GetHeader()->ReplaceOrCreateByNumber( nbFrames, 0x0028, 0x0008);
    }
          
-   f1->GetHeader()->ReplaceOrCreateByNumber(f2->GetHeader()->GetEntryByNumber(0x0028, 0x0010),
- 0x0028, 0x0010);// nbLig
-   f1->GetHeader()->ReplaceOrCreateByNumber( f2->GetHeader()->GetEntryByNumber(0x0028, 0x0011),
-          0x0028, 0x0011);// nbCol
+   f1->GetHeader()->ReplaceOrCreateByNumber(
+      f2->GetHeader()->GetEntryByNumber(0x0028, 0x0010), 0x0028, 0x0010);// nbLig
+   f1->GetHeader()->ReplaceOrCreateByNumber( 
+      f2->GetHeader()->GetEntryByNumber(0x0028, 0x0011), 0x0028, 0x0011);// nbCol
 
 
 // sans doute d'autres à mettre à jour...
@@ -52,9 +45,9 @@ int TestChangeHeader(int argc, char* argv[]) {
 //                        et qui fasse le boulot.
 
 
-   dataSize = f2->GetImageDataSize();
+   int dataSize = f2->GetImageDataSize();
    printf ("dataSize %d\n",dataSize);
-   imageData= f2->GetImageData();
+   void* imageData= f2->GetImageData();
 
 // TODO : ne devrait-on pas fusionner ces 2 fonctions ?
    f1->SetImageData(imageData,dataSize);
@@ -67,6 +60,8 @@ int TestChangeHeader(int argc, char* argv[]) {
    printf("lgr 7fe0, 0000 %s\n",s0.c_str());
    printf("lgr 7fe0, 0010 %s\n",s10.c_str());
 
+   char resultat[512];
+
    sprintf(resultat, "%s.vol", deuxieme.c_str());
    printf ("WriteDCM\n");
   //f1->WriteDcmExplVR("itk-gdcm-ex.dcm");
@@ -75,6 +70,3 @@ int TestChangeHeader(int argc, char* argv[]) {
 
    return 0;
 }
-
-
-