From 51073136ce409884c772b712a69f1264a50f8252 Mon Sep 17 00:00:00 2001 From: Quincey Koziol Date: Thu, 23 Aug 2007 18:19:18 -0500 Subject: [svn-r14109] Description: Make certain to define the API function and typedef version macros in all situations. Tested on: FreeBSD/32 6.2 (duty) Mac OS X/32 10.4.10 (amazon) --- bin/make_vers | 6 ++++++ src/H5Eint.c | 10 +++++----- src/H5version.h | 24 ++++++++++++++++++++++++ 3 files changed, 35 insertions(+), 5 deletions(-) diff --git a/bin/make_vers b/bin/make_vers index b0beb7b..07b6de6 100755 --- a/bin/make_vers +++ b/bin/make_vers @@ -171,6 +171,9 @@ sub print_defaultapivers ($) { # Set up default/latest version name mapping print $fh "\n#if !defined($curr_vers_name) || $curr_vers_name == $curr_vers\n"; + print $fh "#ifndef $curr_vers_name\n"; + print $fh "#define $curr_vers_name $curr_vers\n"; + print $fh "#endif /* $curr_vers_name */\n"; print $fh "#define $curr_name $curr_name$curr_vers\n"; # Print function's dependent parameter types @@ -212,6 +215,9 @@ sub print_defaultapivers ($) { # Set up default/latest version name mapping print $fh "\n#if !defined($curr_vers_name) || $curr_vers_name == $curr_vers\n"; + print $fh "#ifndef $curr_vers_name\n"; + print $fh "#define $curr_vers_name $curr_vers\n"; + print $fh "#endif /* $curr_vers_name */\n"; print $fh "#define ${curr_name}_t $curr_name${curr_vers}_t\n"; # Loop to print earlier version name mappings diff --git a/src/H5Eint.c b/src/H5Eint.c index 95075d7..66fba91 100644 --- a/src/H5Eint.c +++ b/src/H5Eint.c @@ -68,7 +68,7 @@ typedef struct H5E_print_t { /* Local Prototypes */ /********************/ #ifndef H5_NO_DEPRECATED_SYMBOLS -static herr_t H5E_walk1_cb(unsigned n, const H5E_error1_t *err_desc, +static herr_t H5E_walk1_cb(int n, H5E_error1_t *err_desc, void *client_data); #endif /* H5_NO_DEPRECATED_SYMBOLS */ static herr_t H5E_walk2_cb(unsigned n, const H5E_error2_t *err_desc, @@ -215,7 +215,7 @@ H5E_get_msg(const H5E_msg_t *msg, H5E_type_t *type, char *msg_str, size_t size) *------------------------------------------------------------------------- */ static herr_t -H5E_walk1_cb(unsigned n, const H5E_error1_t *err_desc, void *client_data) +H5E_walk1_cb(int n, H5E_error1_t *err_desc, void *client_data) { H5E_print_t *eprint = (H5E_print_t *)client_data; FILE *stream; /* I/O stream to print output to */ @@ -291,7 +291,7 @@ H5E_walk1_cb(unsigned n, const H5E_error1_t *err_desc, void *client_data) have_desc=0; /* Print error message */ - fprintf(stream, "%*s#%03u: %s line %u in %s()%s%s\n", + fprintf(stream, "%*s#%03d: %s line %u in %s()%s%s\n", H5E_INDENT, "", n, err_desc->file_name, err_desc->line, err_desc->func_name, (have_desc ? ": " : ""), (have_desc ? err_desc->desc : "")); @@ -548,7 +548,7 @@ H5E_walk(const H5E_t *estack, H5E_direction_t direction, const H5E_walk_op_t *op old_err.desc = estack->slot[i].desc; old_err.line = estack->slot[i].line; - status = (op->u.func1)((unsigned)i, &old_err, client_data); + status = (op->u.func1)(i, &old_err, client_data); } /* end for */ } /* end if */ else { @@ -562,7 +562,7 @@ H5E_walk(const H5E_t *estack, H5E_direction_t direction, const H5E_walk_op_t *op old_err.desc = estack->slot[i].desc; old_err.line = estack->slot[i].line; - status = (op->u.func1)((unsigned)(estack->nused - (size_t)(i + 1)), &old_err, client_data); + status = (op->u.func1)((int)(estack->nused - (size_t)(i + 1)), &old_err, client_data); } /* end for */ } /* end else */ diff --git a/src/H5version.h b/src/H5version.h index fd534b1..e22ea8c 100644 --- a/src/H5version.h +++ b/src/H5version.h @@ -88,6 +88,9 @@ /*************/ #if !defined(H5Eclear_vers) || H5Eclear_vers == 2 +#ifndef H5Eclear_vers +#define H5Eclear_vers 2 +#endif /* H5Eclear_vers */ #define H5Eclear H5Eclear2 #elif H5Eclear_vers == 1 #define H5Eclear H5Eclear1 @@ -96,6 +99,9 @@ #endif /* H5Eclear_vers */ #if !defined(H5Eget_auto_vers) || H5Eget_auto_vers == 2 +#ifndef H5Eget_auto_vers +#define H5Eget_auto_vers 2 +#endif /* H5Eget_auto_vers */ #define H5Eget_auto H5Eget_auto2 #elif H5Eget_auto_vers == 1 #define H5Eget_auto H5Eget_auto1 @@ -104,6 +110,9 @@ #endif /* H5Eget_auto_vers */ #if !defined(H5Eprint_vers) || H5Eprint_vers == 2 +#ifndef H5Eprint_vers +#define H5Eprint_vers 2 +#endif /* H5Eprint_vers */ #define H5Eprint H5Eprint2 #elif H5Eprint_vers == 1 #define H5Eprint H5Eprint1 @@ -112,6 +121,9 @@ #endif /* H5Eprint_vers */ #if !defined(H5Epush_vers) || H5Epush_vers == 2 +#ifndef H5Epush_vers +#define H5Epush_vers 2 +#endif /* H5Epush_vers */ #define H5Epush H5Epush2 #elif H5Epush_vers == 1 #define H5Epush H5Epush1 @@ -120,6 +132,9 @@ #endif /* H5Epush_vers */ #if !defined(H5Eset_auto_vers) || H5Eset_auto_vers == 2 +#ifndef H5Eset_auto_vers +#define H5Eset_auto_vers 2 +#endif /* H5Eset_auto_vers */ #define H5Eset_auto H5Eset_auto2 #elif H5Eset_auto_vers == 1 #define H5Eset_auto H5Eset_auto1 @@ -128,6 +143,9 @@ #endif /* H5Eset_auto_vers */ #if !defined(H5Ewalk_vers) || H5Ewalk_vers == 2 +#ifndef H5Ewalk_vers +#define H5Ewalk_vers 2 +#endif /* H5Ewalk_vers */ #define H5Ewalk H5Ewalk2 #define H5E_error_t H5E_error2_t #define H5E_walk_t H5E_walk2_t @@ -140,6 +158,9 @@ #endif /* H5Ewalk_vers */ #if !defined(H5Gcreate_vers) || H5Gcreate_vers == 2 +#ifndef H5Gcreate_vers +#define H5Gcreate_vers 2 +#endif /* H5Gcreate_vers */ #define H5Gcreate H5Gcreate2 #elif H5Gcreate_vers == 1 #define H5Gcreate H5Gcreate1 @@ -152,6 +173,9 @@ /************/ #if !defined(H5E_auto_t_vers) || H5E_auto_t_vers == 2 +#ifndef H5E_auto_t_vers +#define H5E_auto_t_vers 2 +#endif /* H5E_auto_t_vers */ #define H5E_auto_t H5E_auto2_t #elif H5E_auto_t_vers == 1 #define H5E_auto_t H5E_auto1_t -- cgit v0.12