[mpich-discuss] Compilation warning with clang

[Eric Chamberland]

Hi,

I am getting these little warnings compiling with clang (3.8.1 with 
-Weverything):

/opt/mpich-3.x_debug/include/mpicxx.h:1528:6: warning: extra ';' after 
member function definition [-Wextra-semi]
     };
      ^
/opt/mpich-3.x_debug/include/mpicxx.h:1596:6: warning: extra ';' after 
member function definition [-Wextra-semi]
     };
      ^
/opt/mpich-3.x_debug/include/mpicxx.h:1687:6: warning: extra ';' after 
member function definition [-Wextra-semi]
     };
      ^
/opt/mpich-3.x_debug/include/mpicxx.h:2365:6: warning: extra ';' after 
member function definition [-Wextra-semi]
     };
      ^
/opt/mpich-3.x_debug/include/mpicxx.h:2480:6: warning: extra ';' after 
member function definition [-Wextra-semi]
     };
      ^

Please have a look at the attached patch.

Thanks,

Eric
-------------- next part --------------
commit 9cb5cbe8e2b8b72e769294e3168a7d6e3b6b1f8a
Author: Eric Chamberland <ericc at giref.ulaval.ca>
Date:   Fri Aug 26 16:27:06 2016 -0400

    maint: remove extras ; gereating warnins with clang
    
    warning: extra ';' after member function definition [-Wextra-semi]

diff --git a/src/binding/cxx/buildiface b/src/binding/cxx/buildiface
index ab0b1a4..2491f4d 100755
--- a/src/binding/cxx/buildiface
+++ b/src/binding/cxx/buildiface
@@ -3573,14 +3573,14 @@ sub Clone_method {
         MPI_Comm_dup( (MPI_Comm)the_real_comm, &ncomm); 
         Comm *clone = new $classname(ncomm); 
         return *clone; 
-    };
+    }
 #else
     virtual $classname & Clone(void) const { 
         MPI_Comm ncomm;
         MPI_Comm_dup( (MPI_Comm)the_real_comm, &ncomm); 
         $classname *clone = new $classname(ncomm); 
         return *clone; 
-    };
+    }
 #endif\n";
 }
 sub Comm_methods {
@@ -3673,12 +3673,12 @@ sub Nullcomm_methods {
     virtual Comm & Clone(void) const { 
         Comm *clone = new Nullcomm(MPI_COMM_NULL); 
         return *clone; 
-    };
+    }
 #else
     virtual Nullcomm & Clone(void) const { 
         Nullcomm *clone = new Nullcomm();
         return *clone; 
-    };
+    }
 #endif\n";
 }
 
-------------- next part --------------
_______________________________________________
discuss mailing list     discuss at mpich.org
To manage subscription options or unsubscribe:
https://lists.mpich.org/mailman/listinfo/discuss