[mpich-commits] [mpich] MPICH primary repository branch, master, updated. v3.1b1-26-g09be625

mysql vizuser noreply at mpich.org
Tue Aug 27 10:08:54 CDT 2013


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "MPICH primary repository".

The branch, master has been updated
       via  09be6253a4c1a18d3e800f779da1213caca49f43 (commit)
      from  203de0e0906adfa8ec530fefd29e994062e3be17 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
http://git.mpich.org/mpich.git/commitdiff/09be6253a4c1a18d3e800f779da1213caca49f43

commit 09be6253a4c1a18d3e800f779da1213caca49f43
Author: William Gropp <wgropp at illinois.edu>
Date:   Sun Aug 25 10:02:24 2013 -0500

    Fix two errors in basic MPI_T test
    
    One was discovered on reading the code; the correct test for the enum
    case was not made.  The second was discovered by the Blue Waters team when
    testing the most recent Cray MPI release.  Its not clear how this passed our
    tests.

diff --git a/test/mpi/mpi_t/mpi_t_str.c b/test/mpi/mpi_t/mpi_t_str.c
index 5cc67c5..cf1d5b2 100644
--- a/test/mpi/mpi_t/mpi_t_str.c
+++ b/test/mpi/mpi_t/mpi_t_str.c
@@ -99,10 +99,12 @@ int main(int argc, char **argv)
 
         /* not much of a string test, just need a quick spot to stick a test for
          * the existence of the correct MPI_T prototype (tt#1727) */
-        if (enumtype != MPI_T_ENUM_NULL) {
+        /* Include test that enumtype is defined */
+        if (dtype == MPI_INT && enumtype != MPI_T_ENUM_NULL) {
             int num_enumtype = -1;
             name_len = STR_SZ;
-            MPI_T_enum_get_info(enumtype, &num_enumtype, name, name_len);
+            MPI_T_enum_get_info(enumtype, &num_enumtype, name, &name_len);
+            check(num_enumtype >= 0);
         }
     }
 

-----------------------------------------------------------------------

Summary of changes:
 test/mpi/mpi_t/mpi_t_str.c |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)


hooks/post-receive
-- 
MPICH primary repository


More information about the commits mailing list