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/09be6253a4c1a18d3e800f779da1213cac... commit 09be6253a4c1a18d3e800f779da1213caca49f43 Author: William Gropp <[email protected]> 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