From 12572db97b786e78f79c1b119151fae53287b633 Mon Sep 17 00:00:00 2001 From: Christian Heinrich Date: Tue, 13 Mar 2018 14:18:34 +0100 Subject: [PATCH] [SMPI] Replay: Remove side effect from decode_datatype The MPI_CURRENT_TYPE variable shouldn't be modified here. --- src/smpi/internals/smpi_replay.cpp | 17 ++++++++--------- 1 file changed, 8 insertions(+), 9 deletions(-) diff --git a/src/smpi/internals/smpi_replay.cpp b/src/smpi/internals/smpi_replay.cpp index 48052053fd..9e6883dec1 100644 --- a/src/smpi/internals/smpi_replay.cpp +++ b/src/smpi/internals/smpi_replay.cpp @@ -93,31 +93,30 @@ static MPI_Datatype decode_datatype(const char *const action) { switch(atoi(action)) { case 0: - MPI_CURRENT_TYPE=MPI_DOUBLE; + return MPI_DOUBLE; break; case 1: - MPI_CURRENT_TYPE=MPI_INT; + return MPI_INT; break; case 2: - MPI_CURRENT_TYPE=MPI_CHAR; + return MPI_CHAR; break; case 3: - MPI_CURRENT_TYPE=MPI_SHORT; + return MPI_SHORT; break; case 4: - MPI_CURRENT_TYPE=MPI_LONG; + return MPI_LONG; break; case 5: - MPI_CURRENT_TYPE=MPI_FLOAT; + return MPI_FLOAT; break; case 6: - MPI_CURRENT_TYPE=MPI_BYTE; + return MPI_BYTE; break; default: - MPI_CURRENT_TYPE=MPI_DEFAULT_TYPE; + return MPI_DEFAULT_TYPE; break; } - return MPI_CURRENT_TYPE; } const char* encode_datatype(MPI_Datatype datatype) -- 2.20.1