Skip to content

Commit 11ab621

Browse files
committed
mpi.h: file errhandeler typedef: use new form of name
The old/deprecated form of the file errhandler typedef used "fn" as a suffix. The new form uses the name "function". The MPI API typedef name has already been updated to use "function"; this commit updates the internal Open MPI typedef to use the name "function" to match the MPI API name and avoid confusion. Signed-off-by: Jeff Squyres <jsquyres@cisco.com>
1 parent aa5e1df commit 11ab621

File tree

5 files changed

+8
-8
lines changed

5 files changed

+8
-8
lines changed

ompi/errhandler/errhandler.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -202,7 +202,7 @@ ompi_errhandler_t *ompi_errhandler_create(ompi_errhandler_type_t object_type,
202202
new_errhandler->eh_comm_fn = (MPI_Comm_errhandler_function *)func;
203203
break;
204204
case (OMPI_ERRHANDLER_TYPE_FILE):
205-
new_errhandler->eh_file_fn = (ompi_file_errhandler_fn *)func;
205+
new_errhandler->eh_file_fn = (ompi_file_errhandler_function *)func;
206206
break;
207207
case (OMPI_ERRHANDLER_TYPE_WIN):
208208
new_errhandler->eh_win_fn = (MPI_Win_errhandler_function *)func;

ompi/errhandler/errhandler.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ struct ompi_errhandler_t {
117117
can be invoked on any MPI object type, so we need callbacks for
118118
all of three. */
119119
MPI_Comm_errhandler_function *eh_comm_fn;
120-
ompi_file_errhandler_fn *eh_file_fn;
120+
ompi_file_errhandler_function *eh_file_fn;
121121
MPI_Win_errhandler_function *eh_win_fn;
122122
ompi_errhandler_fortran_handler_fn_t *eh_fort_fn;
123123

ompi/include/mpi.h.in

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -385,11 +385,11 @@ typedef int (MPI_Datarep_conversion_function)(void *, MPI_Datatype,
385385
typedef void (MPI_Comm_errhandler_function)(MPI_Comm *, int *, ...);
386386

387387
/* This is a little hackish, but errhandler.h needs space for a
388-
MPI_File_errhandler_fn. While it could just be removed, this
388+
MPI_File_errhandler_function. While it could just be removed, this
389389
allows us to maintain a stable ABI within OMPI, at least for
390390
apps that don't use MPI I/O. */
391-
typedef void (ompi_file_errhandler_fn)(MPI_File *, int *, ...);
392-
typedef ompi_file_errhandler_fn MPI_File_errhandler_function;
391+
typedef void (ompi_file_errhandler_function)(MPI_File *, int *, ...);
392+
typedef ompi_file_errhandler_function MPI_File_errhandler_function;
393393
typedef void (MPI_Win_errhandler_function)(MPI_Win *, int *, ...);
394394
typedef void (MPI_User_function)(void *, void *, int *, MPI_Datatype *);
395395
typedef int (MPI_Comm_copy_attr_function)(MPI_Comm, int, void *,
@@ -412,7 +412,7 @@ typedef int (MPI_Grequest_cancel_function)(void *, int);
412412
*/
413413
typedef MPI_Comm_errhandler_function MPI_Comm_errhandler_fn
414414
__mpi_interface_removed__("MPI_Comm_errhandler_fn was removed in MPI-3.0; use MPI_Comm_errhandler_function instead");
415-
typedef ompi_file_errhandler_fn MPI_File_errhandler_fn
415+
typedef ompi_file_errhandler_function MPI_File_errhandler_fn
416416
__mpi_interface_removed__("MPI_File_errhandler_fn was removed in MPI-3.0; use MPI_File_errhandler_function instead");
417417
typedef MPI_Win_errhandler_function MPI_Win_errhandler_fn
418418
__mpi_interface_removed__("MPI_Win_errhandler_fn was removed in MPI-3.0; use MPI_Win_errhandler_function instead");

ompi/mpi/cxx/cxx_glue.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,7 @@ ompi_cxx_intercept_file_extra_state_t
144144
}
145145

146146
void ompi_cxx_errhandler_set_callbacks (struct ompi_errhandler_t *errhandler, MPI_Comm_errhandler_function *eh_comm_fn,
147-
ompi_file_errhandler_fn *eh_file_fn, MPI_Win_errhandler_function *eh_win_fn)
147+
ompi_file_errhandler_function *eh_file_fn, MPI_Win_errhandler_function *eh_win_fn)
148148
{
149149
errhandler->eh_comm_fn = eh_comm_fn;
150150
errhandler->eh_file_fn = eh_file_fn;

ompi/mpi/cxx/cxx_glue.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ ompi_cxx_intercept_file_extra_state_t
7979
void *extra_state_cxx);
8080

8181
void ompi_cxx_errhandler_set_callbacks (struct ompi_errhandler_t *errhandler, MPI_Comm_errhandler_function *eh_comm_fn,
82-
ompi_file_errhandler_fn *eh_file_fn, MPI_Win_errhandler_function *eh_win_fn);
82+
ompi_file_errhandler_function *eh_file_fn, MPI_Win_errhandler_function *eh_win_fn);
8383

8484
#if defined(c_plusplus) || defined(__cplusplus)
8585
}

0 commit comments

Comments
 (0)