From cfc4bc0156d2846054d5e79cc6a9b5770a2a729f Mon Sep 17 00:00:00 2001
From: regrain <regrain>
Date: Tue, 29 Nov 2005 13:02:44 +0000
Subject: [PATCH]    * Improvement of the TestInline    -- BeNours

---
 Testing/CMakeLists.txt |  10 +-
 Testing/TestInline.cxx | 249 ++++++++++++++++++++++++++++++++++++-----
 src/gdcmUtil.cxx       |  12 +-
 src/gdcmUtil.h         |  23 +++-
 4 files changed, 256 insertions(+), 38 deletions(-)

diff --git a/Testing/CMakeLists.txt b/Testing/CMakeLists.txt
index 207d2f2c..f1b1926f 100644
--- a/Testing/CMakeLists.txt
+++ b/Testing/CMakeLists.txt
@@ -24,11 +24,11 @@ SET(TEST_SOURCES
   TestEnumVR.cxx
 
 )
-#IF(UNIX)
-#  SET(TEST_SOURCES ${TEST_SOURCES}
-#    TestInline.cxx
-#  )
-#ENDIF(UNIX)
+IF(UNIX)
+  SET(TEST_SOURCES ${TEST_SOURCES}
+    TestInline.cxx
+  )
+ENDIF(UNIX)
 
 # add tests that require data
 IF (GDCM_DATA_ROOT)
diff --git a/Testing/TestInline.cxx b/Testing/TestInline.cxx
index 1709a2c8..b78e094b 100755
--- a/Testing/TestInline.cxx
+++ b/Testing/TestInline.cxx
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: TestInline.cxx,v $
   Language:  C++
-  Date:      $Date: 2005/11/18 11:46:37 $
-  Version:   $Revision: 1.9 $
+  Date:      $Date: 2005/11/29 13:02:44 $
+  Version:   $Revision: 1.10 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -31,6 +31,7 @@
 
 #include "gdcmUtil.h"
 
+#include <sys/times.h>
 #include <iostream>
 
 void        frswap (double &a, double &b);
@@ -38,9 +39,13 @@ void        fpswap (double *a, double *b);
 inline void ifrswap(double &a, double &b);
 inline void ifpswap(double *a, double *b);
 
-uint16_t     passDirect(uint16_t a,  uint16_t b);
-uint16_t     passRef(uint16_t &a, uint16_t &b);
-uint16_t     passPtr(uint16_t *a, uint16_t *b);
+uint8_t     passDirect8(uint8_t a,  uint8_t b);
+uint8_t     passRef8(uint8_t &a, uint8_t &b);
+uint8_t     passPtr8(uint8_t *a, uint8_t *b);
+
+uint16_t     passDirect16(uint16_t a,  uint16_t b);
+uint16_t     passRef16(uint16_t &a, uint16_t &b);
+uint16_t     passPtr16(uint16_t *a, uint16_t *b);
 
 uint32_t     passDirect32(uint32_t a,  uint32_t b);
 uint32_t     passRef32(uint32_t &a, uint32_t &b);
@@ -53,7 +58,7 @@ double     passPtr(double *a, double *b);
 #define       \
 mswap(a, b)   \
 {             \
-   tmp = a;   \
+   double tmp = a;   \
    a   = b;   \
    b   = tmp; \
 }
@@ -107,15 +112,41 @@ uint32_t passDirect32(uint32_t a, uint32_t b)
 } 
 
 
-uint16_t passRef(uint16_t &a, uint16_t &b)
+uint16_t passRef16(uint16_t &a, uint16_t &b)
+{
+   return a + b;
+} 
+uint16_t passPtr16(uint16_t *a, uint16_t *b)
+{
+   return *a + *b;
+} 
+uint16_t passDirect16(uint16_t a, uint16_t b)
+{
+   return a + b;
+} 
+
+uint8_t passRef8(uint8_t &a, uint8_t &b)
+{
+   return a + b;
+} 
+uint8_t passPtr8(uint8_t *a, uint8_t *b)
+{
+   return *a + *b;
+} 
+uint8_t passDirect8(uint8_t a, uint8_t b)
+{
+   return a + b;
+} 
+
+float passRefFloat(float &a, float &b)
 {
    return a + b;
 } 
-uint16_t passPtr(uint16_t *a, uint16_t *b)
+float passPtrFloat(float *a, float *b)
 {
    return *a + *b;
 } 
-uint16_t passDirect(uint16_t a, uint16_t b)
+float passDirectFloat(float a, float b)
 {
    return a + b;
 } 
@@ -142,15 +173,36 @@ int TestInline(int argc, char *argv[])
    if (argc > 1)
       nbLoop = atoi(argv[1]);
    else
-      nbLoop = 10000000;      
+      nbLoop = 500000000;
 
    //clock_t r1, r2;
    struct tms tms1, tms2;
    
    double a = 1, b = 2;
-   double tmp;
    
-   uint16_t x=1, y=2;    
+   uint8_t  x8 =1, y8 =2;    
+   uint16_t x16=1, y16=2;    
+   uint32_t x32=1, y32=2;    
+   float  fx =1.0f, fy=1.0f;
+   double dx =1.0 , dy=1.0;
+ // ----------------------------------------
+ 
+   std::cout << "Direct "<< std::endl;
+   //r1 = times(&tms1);
+   times(&tms1);   
+   for(i = 0 ; i< nbLoop ; i++)
+   {
+      double tmp;
+      tmp=a;
+      a=b;
+      b=tmp;
+   }
+   //r2 = times(&tms2);
+   times(&tms2);   
+    std::cout 
+        << (long) ((tms2.tms_utime)  - (tms1.tms_utime)) 
+        << std::endl;
+   
  // ----------------------------------------
  
    std::cout << "Use a macro "<< std::endl;
@@ -231,14 +283,63 @@ int TestInline(int argc, char *argv[])
 //To check the 2 following cases, we just put the two 'static' functions
 //hifpswap and  hNoifpswap in gdcmUtil.h
     
-   std::cout << "Use inline, .h defined, WITH inline keyword pointer function"
+   std::cout << "Use inline, .h defined, WITH inline keyword pointer method"
+             << std::endl;
+   //r1 = times(&tms1);
+   gdcm::Util util;
+   times(&tms1);   
+    for(i = 0 ; i< nbLoop ; i++)
+   {
+      util.hifpswap (&a, &b);
+   }
+   //r2 = times(&tms2);
+   times(&tms2);   
+   std::cout 
+        << (long) ((tms2.tms_utime)  - (tms1.tms_utime)) 
+        << std::endl;  
+
+   
+ // ----------------------------------------
+
+   std::cout << "Use inline, .h defined, NO inline keyword pointer method"
              << std::endl;
    //r1 = times(&tms1);
    times(&tms1);   
     for(i = 0 ; i< nbLoop ; i++)
    {
-      //gdcm::Util::hifpswap (&a, &b);
-      hifpswap (&a, &b);  
+      util.hNoifpswap (&a, &b);
+   }
+   //r2 = times(&tms2);
+   times(&tms2);   
+   std::cout 
+        << (long) ((tms2.tms_utime)  - (tms1.tms_utime)) 
+        << std::endl; 
+
+ // ----------------------------------------
+
+   std::cout << "Use , .h defined, NO inline keyword pointer method"
+             << std::endl;
+   //r1 = times(&tms1);
+   times(&tms1);   
+    for(i = 0 ; i< nbLoop ; i++)
+   {
+      util.hfpswap (&a, &b);
+   }
+   //r2 = times(&tms2);
+   times(&tms2);   
+   std::cout 
+        << (long) ((tms2.tms_utime)  - (tms1.tms_utime))
+        <<std::endl;
+ 
+ // ----------------------------------------
+
+   std::cout << "Use inline, .h defined, WITH inline keyword pointer static method"
+             << std::endl;
+   //r1 = times(&tms1);
+   times(&tms1);   
+    for(i = 0 ; i< nbLoop ; i++)
+   {
+      gdcm::Util::sthifpswap (&a, &b);
    }
    //r2 = times(&tms2);
    times(&tms2);   
@@ -249,14 +350,13 @@ int TestInline(int argc, char *argv[])
    
  // ----------------------------------------
 
-   std::cout << "Use inline, .h defined, NO inline keyword pointer function"
+   std::cout << "Use inline, .h defined, NO inline keyword pointer static method"
              << std::endl;
    //r1 = times(&tms1);
    times(&tms1);   
     for(i = 0 ; i< nbLoop ; i++)
    {
-      //gdcm::Util::hNoifpswap (&a, &b);
-      hNoifpswap (&a, &b);  
+      gdcm::Util::sthNoifpswap (&a, &b);
    }
    //r2 = times(&tms2);
    times(&tms2);   
@@ -265,6 +365,54 @@ int TestInline(int argc, char *argv[])
         << std::endl; 
 
 
+ // ----------------------------------------
+ 
+   std::cout << "Pass uint_8 param directly"
+             << std::endl;
+   //r1 = times(&tms1);
+   times(&tms1);   
+    for(i = 0 ; i< nbLoop ; i++)
+   {
+      passDirect8 (x8, y8);  
+   }
+   //r2 = times(&tms2);
+   times(&tms2);   
+   std::cout 
+        << (long) ((tms2.tms_utime)  - (tms1.tms_utime)) 
+        << std::endl; 
+
+ // ----------------------------------------
+ 
+   std::cout << "Pass uint_8 param as ref"
+             << std::endl;
+   //r1 = times(&tms1);
+   times(&tms1);   
+    for(i = 0 ; i< nbLoop ; i++)
+   {
+      passRef8 (x8, y8);  
+   }
+   //r2 = times(&tms2);
+   times(&tms2);   
+   std::cout 
+        << (long) ((tms2.tms_utime)  - (tms1.tms_utime)) 
+        << std::endl; 
+
+ // ----------------------------------------
+ 
+   std::cout << "Pass uint_8 param as ptr"
+             << std::endl;
+   //r1 = times(&tms1);
+   times(&tms1);   
+    for(i = 0 ; i< nbLoop ; i++)
+   {
+      passPtr8 (&x8, &y8);  
+   }
+   //r2 = times(&tms2);
+   times(&tms2);   
+   std::cout 
+        << (long) ((tms2.tms_utime)  - (tms1.tms_utime)) 
+        << std::endl; 
+
  // ----------------------------------------
  
    std::cout << "Pass uint_16 param directly"
@@ -273,7 +421,7 @@ int TestInline(int argc, char *argv[])
    times(&tms1);   
     for(i = 0 ; i< nbLoop ; i++)
    {
-      passDirect (x, y);  
+      passDirect16 (x16, y16);  
    }
    //r2 = times(&tms2);
    times(&tms2);   
@@ -289,7 +437,7 @@ int TestInline(int argc, char *argv[])
    times(&tms1);   
     for(i = 0 ; i< nbLoop ; i++)
    {
-      passRef (x, y);  
+      passRef16 (x16, y16);  
    }
    //r2 = times(&tms2);
    times(&tms2);   
@@ -305,7 +453,7 @@ int TestInline(int argc, char *argv[])
    times(&tms1);   
     for(i = 0 ; i< nbLoop ; i++)
    {
-      passPtr (&x, &y);  
+      passPtr16 (&x16, &y16);  
    }
    //r2 = times(&tms2);
    times(&tms2);   
@@ -317,14 +465,13 @@ int TestInline(int argc, char *argv[])
 
  // ----------------------------------------
 
-   uint32_t m =1, n=2; 
    std::cout << "Pass uint_32 param directly"
              << std::endl;
    //r1 = times(&tms1);
    times(&tms1);   
     for(i = 0 ; i< nbLoop ; i++)
    {
-      passDirect32 (m, n);  
+      passDirect32 (x32, y32);  
    }
    //r2 = times(&tms2);
    times(&tms2);   
@@ -340,7 +487,7 @@ int TestInline(int argc, char *argv[])
    times(&tms1);   
     for(i = 0 ; i< nbLoop ; i++)
    {
-      passRef32 (m, n);  
+      passRef32 (x32, y32 );  
    }
    //r2 = times(&tms2);
    times(&tms2);   
@@ -356,7 +503,7 @@ int TestInline(int argc, char *argv[])
    times(&tms1);   
     for(i = 0 ; i< nbLoop ; i++)
    {
-      passPtr32 (&m, &n);  
+      passPtr32 (&x32, &y32);  
    }
    //r2 = times(&tms2);
    times(&tms2);   
@@ -366,9 +513,57 @@ int TestInline(int argc, char *argv[])
 
  // ----------------------------------------
  
+   std::cout << "Pass float param directly"
+             << std::endl;
+   //r1 = times(&tms1);
+   times(&tms1);   
+    for(i = 0 ; i< nbLoop ; i++)
+   {
+      passDirectFloat (fx, fy);  
+   }
+   //r2 = times(&tms2);
+   times(&tms2);   
+
+   std::cout 
+        << (long) ((tms2.tms_utime)  - (tms1.tms_utime)) 
+        << std::endl; 
+
+ // ----------------------------------------
+ 
+   std::cout << "Pass float param as ref"
+             << std::endl;
+   //r1 = times(&tms1);
+   times(&tms1);   
+    for(i = 0 ; i< nbLoop ; i++)
+   {
+      passRefFloat (fx, fy);  
+   }
+   //r2 = times(&tms2);
+   times(&tms2);   
+
+   std::cout 
+        << (long) ((tms2.tms_utime)  - (tms1.tms_utime)) 
+        << std::endl; 
+
+ // ----------------------------------------
+ 
+   std::cout << "Pass float param as ptr"
+             << std::endl;
+   //r1 = times(&tms1);
+   times(&tms1);   
+    for(i = 0 ; i< nbLoop ; i++)
+   {
+      passPtrFloat (&fx, &fy);  
+   }
+   //r2 = times(&tms2);
+   times(&tms2);   
+
+   std::cout 
+        << (long) ((tms2.tms_utime)  - (tms1.tms_utime)) 
+        << std::endl; 
+
+ // ----------------------------------------
  
-   double dx=1.0, dy=2.0;
-   
    std::cout << "Pass double param directly"
              << std::endl;
    //r1 = times(&tms1);
diff --git a/src/gdcmUtil.cxx b/src/gdcmUtil.cxx
index a89ca687..936e0dde 100644
--- a/src/gdcmUtil.cxx
+++ b/src/gdcmUtil.cxx
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmUtil.cxx,v $
   Language:  C++
-  Date:      $Date: 2005/11/28 16:50:33 $
-  Version:   $Revision: 1.178 $
+  Date:      $Date: 2005/11/29 13:02:46 $
+  Version:   $Revision: 1.179 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -1147,6 +1147,14 @@ std::string Util::GetIPAddress()
    return str;
 }
 
+void Util::hfpswap(double *a, double *b)
+{
+   double tmp;
+   tmp=*a;
+   *a=*b;
+   *b=tmp;
+}
+
 //-------------------------------------------------------------------------
 } // end namespace gdcm
 
diff --git a/src/gdcmUtil.h b/src/gdcmUtil.h
index ecda0167..9ef1c976 100644
--- a/src/gdcmUtil.h
+++ b/src/gdcmUtil.h
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmUtil.h,v $
   Language:  C++
-  Date:      $Date: 2005/10/27 17:04:36 $
-  Version:   $Revision: 1.65 $
+  Date:      $Date: 2005/11/29 13:02:47 $
+  Version:   $Revision: 1.66 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -82,14 +82,29 @@ public:
 // hifpswap and  hNoifpswap in a .h
 // They will be remove ASAP
 
-   static inline void hifpswap(double *a, double *b)    
+   inline void hifpswap(double *a, double *b)     
    {
       double tmp;
       tmp = *a;
       *a = *b;
       *b = tmp;
    }
-   static void hNoifpswap(double *a, double *b)    
+   void hNoifpswap(double *a, double *b)    
+   {
+      double tmp;
+      tmp = *a;
+      *a = *b;
+      *b = tmp;
+   }    
+   void hfpswap(double *a, double *b);   
+   static inline void sthifpswap(double *a, double *b)     
+   {
+      double tmp;
+      tmp = *a;
+      *a = *b;
+      *b = tmp;
+   }
+   static void sthNoifpswap(double *a, double *b)    
    {
       double tmp;
       tmp = *a;
-- 
2.45.1