diff options
author | Sean McBride <sean@rogue-research.com> | 2023-06-19 05:13:38 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-06-19 05:13:38 (GMT) |
commit | 65d8c9347010771473b53c91adcec2f281772213 (patch) | |
tree | 487567dae0dc005de896f616b90e67744239a5e2 /testpar | |
parent | 1f20354ee6cdfa9fd157ac9cdfff9acdf320a32d (diff) | |
download | hdf5-65d8c9347010771473b53c91adcec2f281772213.zip hdf5-65d8c9347010771473b53c91adcec2f281772213.tar.gz hdf5-65d8c9347010771473b53c91adcec2f281772213.tar.bz2 |
Many fixes to various compiler warnings (#3124)
* Fixed various -Wmissing-variable-declarations by adding static keyword
* In a few cases, renamed the variable suffix from _g to _s.
* Fixed some -Wmissing-variable-declarations by using different declaration macros
* Fixed various -Wconditional-uninitialized warnings by just initializing variable to zero
* Fixed various -Wcomma warnings
* Fixed clang -Wstrict-prototypes warnings
* Fixed various -Wunused-variable warnings
* Updated some casts to fix the only 3 -Wcast-qual warnings
* Fixed the only -Wsometimes-uninitialized warning
Diffstat (limited to 'testpar')
-rw-r--r-- | testpar/t_filters_parallel.c | 4 | ||||
-rw-r--r-- | testpar/t_init_term.c | 2 | ||||
-rw-r--r-- | testpar/t_pflush1.c | 2 | ||||
-rw-r--r-- | testpar/t_pflush2.c | 2 | ||||
-rw-r--r-- | testpar/t_prestart.c | 2 | ||||
-rw-r--r-- | testpar/t_pshutdown.c | 2 |
6 files changed, 7 insertions, 7 deletions
diff --git a/testpar/t_filters_parallel.c b/testpar/t_filters_parallel.c index f6c300a..9fa5a9c 100644 --- a/testpar/t_filters_parallel.c +++ b/testpar/t_filters_parallel.c @@ -20,8 +20,8 @@ #include "t_filters_parallel.h" -const char *FILENAME[] = {"t_filters_parallel", NULL}; -char filenames[1][256]; +static const char *FILENAME[] = {"t_filters_parallel", NULL}; +char filenames[1][256]; static MPI_Comm comm = MPI_COMM_WORLD; static MPI_Info info = MPI_INFO_NULL; diff --git a/testpar/t_init_term.c b/testpar/t_init_term.c index 2f2ad61..e492d92 100644 --- a/testpar/t_init_term.c +++ b/testpar/t_init_term.c @@ -22,7 +22,7 @@ int nerrors = 0; /* errors count */ -const char *FILENAME[] = {"after_mpi_fin", NULL}; +static const char *FILENAME[] = {"after_mpi_fin", NULL}; int main(int argc, char **argv) diff --git a/testpar/t_pflush1.c b/testpar/t_pflush1.c index edfbcfe..f7131d5 100644 --- a/testpar/t_pflush1.c +++ b/testpar/t_pflush1.c @@ -22,7 +22,7 @@ */ #include "h5test.h" -const char *FILENAME[] = {"flush", "noflush", NULL}; +static const char *FILENAME[] = {"flush", "noflush", NULL}; static int *data_g = NULL; diff --git a/testpar/t_pflush2.c b/testpar/t_pflush2.c index 0250463..e346714 100644 --- a/testpar/t_pflush2.c +++ b/testpar/t_pflush2.c @@ -23,7 +23,7 @@ #include "h5test.h" -const char *FILENAME[] = {"flush", "noflush", NULL}; +static const char *FILENAME[] = {"flush", "noflush", NULL}; static int *data_g = NULL; diff --git a/testpar/t_prestart.c b/testpar/t_prestart.c index bfa72b6..dc291af 100644 --- a/testpar/t_prestart.c +++ b/testpar/t_prestart.c @@ -22,7 +22,7 @@ int nerrors = 0; /* errors count */ -const char *FILENAME[] = {"shutdown", NULL}; +static const char *FILENAME[] = {"shutdown", NULL}; int main(int argc, char **argv) diff --git a/testpar/t_pshutdown.c b/testpar/t_pshutdown.c index 278fd09..af98a76 100644 --- a/testpar/t_pshutdown.c +++ b/testpar/t_pshutdown.c @@ -26,7 +26,7 @@ int nerrors = 0; /* errors count */ -const char *FILENAME[] = {"shutdown", NULL}; +static const char *FILENAME[] = {"shutdown", NULL}; int main(int argc, char **argv) |