hdf5/Fix-warnings-for-missing-returns.patch

119 lines
3.4 KiB
Diff
Raw Normal View History

From: Egbert Eich <eich@suse.com>
Date: Tue Nov 7 12:56:49 2017 +0100
Subject: [PATCH]Fix warnings for missing returns
Git-commit: 3204e3b0f578906edd85bb95b0bbc9818490e2d0
References:
Signed-off-by: Egbert Eich <eich@suse.com>
Signed-off-by: Egbert Eich <eich@suse.com>
---
c++/src/H5Location.cpp | 1 +
test/dt_arith.c | 1 +
test/filter_fail.c | 1 +
tools/src/h5copy/h5copy.c | 2 +-
tools/src/h5dump/h5dump.c | 2 +-
tools/src/h5ls/h5ls.c | 2 +-
tools/src/h5stat/h5stat.c | 2 +-
tools/src/misc/h5mkgrp.c | 2 +-
8 files changed, 8 insertions(+), 5 deletions(-)
diff --git a/c++/src/H5Location.cpp b/c++/src/H5Location.cpp
index b2fa75d..a3fe206 100644
--- a/c++/src/H5Location.cpp
+++ b/c++/src/H5Location.cpp
@@ -95,6 +95,7 @@ bool H5Location::exists(const char* name, const LinkAccPropList& lapl) const
{
throwException("exists", "H5Lexists failed");
}
+ return false;
}
//--------------------------------------------------------------------------
diff --git a/test/dt_arith.c b/test/dt_arith.c
index c7f2986..8f3e6e4 100644
--- a/test/dt_arith.c
+++ b/test/dt_arith.c
@@ -3384,6 +3384,7 @@ error:
else if(run_test==TEST_DENORM || run_test==TEST_SPECIAL)
return 1;
#endif
+ return 1;
}
diff --git a/test/filter_fail.c b/test/filter_fail.c
index 76b3106..cda9d50 100644
--- a/test/filter_fail.c
+++ b/test/filter_fail.c
@@ -406,4 +406,5 @@ error:
nerrors, 1==nerrors?"":"S");
HDexit(EXIT_FAILURE);
}
+ return 1;
}
diff --git a/tools/src/h5copy/h5copy.c b/tools/src/h5copy/h5copy.c
index 390b93e..506c24e 100644
--- a/tools/src/h5copy/h5copy.c
+++ b/tools/src/h5copy/h5copy.c
@@ -55,7 +55,7 @@ char *str_flag = NULL;
*
*-------------------------------------------------------------------------
*/
-static void
+static void __attribute__((__noreturn__))
leave(int ret)
{
if (fname_src)
diff --git a/tools/src/h5dump/h5dump.c b/tools/src/h5dump/h5dump.c
index b53c212..747147b 100644
--- a/tools/src/h5dump/h5dump.c
+++ b/tools/src/h5dump/h5dump.c
@@ -208,7 +208,7 @@ static struct long_options l_opts[] = {
*
*-------------------------------------------------------------------------
*/
-static void
+static void __attribute__((__noreturn__))
leave(int ret)
{
h5tools_close();
diff --git a/tools/src/h5ls/h5ls.c b/tools/src/h5ls/h5ls.c
index 8e28d82..c1f1cba 100644
--- a/tools/src/h5ls/h5ls.c
+++ b/tools/src/h5ls/h5ls.c
@@ -2564,7 +2564,7 @@ out:
*
*-------------------------------------------------------------------------
*/
-static void
+static void __attribute__((__noreturn__))
leave(int ret)
{
h5tools_close();
diff --git a/tools/src/h5stat/h5stat.c b/tools/src/h5stat/h5stat.c
index 6aee7a8..b14a975 100644
--- a/tools/src/h5stat/h5stat.c
+++ b/tools/src/h5stat/h5stat.c
@@ -250,7 +250,7 @@ static struct long_options l_opts[] = {
{ NULL, 0, '\0' }
};
-static void
+static void __attribute__((__noreturn__))
leave(int ret)
{
h5tools_close();
diff --git a/tools/src/misc/h5mkgrp.c b/tools/src/misc/h5mkgrp.c
index 597b6b3..15caff6 100644
--- a/tools/src/misc/h5mkgrp.c
+++ b/tools/src/misc/h5mkgrp.c
@@ -58,7 +58,7 @@ param_t params; /* Command line parameter settings */
*
*-------------------------------------------------------------------------
*/
-static void
+static void __attribute__((__noreturn__))
leave(int ret)
{
size_t curr_group;