aboutsummaryrefslogtreecommitdiff
path: root/gdb/testsuite/gdb.mi
diff options
context:
space:
mode:
authorSimon Marchi <simon.marchi@ericsson.com>2015-07-27 15:49:24 -0400
committerSimon Marchi <simon.marchi@ericsson.com>2015-07-27 15:49:24 -0400
commit13e1dec8960122ec0329f74e5915503e14db5ea2 (patch)
treec32be51b725795b0fc6cbc188ab32072e3abee23 /gdb/testsuite/gdb.mi
parentda33c9a7bff093e772a381bf9ab42d876089cdcf (diff)
downloadgdb-13e1dec8960122ec0329f74e5915503e14db5ea2.zip
gdb-13e1dec8960122ec0329f74e5915503e14db5ea2.tar.gz
gdb-13e1dec8960122ec0329f74e5915503e14db5ea2.tar.bz2
gdb.mi/mi-pending.c: Return NULL instead of nothing in thread function
Using gcc 5.2 (maybe other versions as well), building mi-pending.c gives these warnings: ./gdb.mi/mi-pending.c: In function ‘thread_func’: ./gdb.mi/mi-pending.c:34:5: warning: ‘return’ with no value, in function returning non-void return; ^ ./gdb.mi/mi-pending.c:38:5: warning: ‘return’ with no value, in function returning non-void return; ^ gdb_compile_pthreads assumes that the build was successful only if there is no output. These warnings therefore make gdb_compile_pthreads think that the build failed, and the test doesn't run. The easy fix is to replace the "return" with "return NULL". I am pushing this as obvious. gdb/testsuite/ChangeLog: * gdb.mi/mi-pending.c (thread_func): Replace return with return NULL.
Diffstat (limited to 'gdb/testsuite/gdb.mi')
-rw-r--r--gdb/testsuite/gdb.mi/mi-pending.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/gdb/testsuite/gdb.mi/mi-pending.c b/gdb/testsuite/gdb.mi/mi-pending.c
index 248c7a8..2f3464e 100644
--- a/gdb/testsuite/gdb.mi/mi-pending.c
+++ b/gdb/testsuite/gdb.mi/mi-pending.c
@@ -31,11 +31,11 @@ thread_func (void* arg)
h = dlopen (libname, RTLD_LAZY); /* set breakpoint here */
if (h == NULL)
- return;
+ return NULL;
p_func = dlsym (h, "pendfunc3");
if (p_func == NULL)
- return;
+ return NULL;
(*p_func) ();
}