This is the mail archive of the gdb-cvs@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[binutils-gdb] gdb.base/fileio.c: Fix several -Wreturn-type warnings


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=c8f6abd10d62f82874e31959bf986606bd919d39

commit c8f6abd10d62f82874e31959bf986606bd919d39
Author: Pedro Alves <palves@redhat.com>
Date:   Thu May 18 11:37:55 2017 +0100

    gdb.base/fileio.c: Fix several -Wreturn-type warnings
    
    All the "test_" functions warn like:
    
      src/gdb/testsuite/gdb.base/fileio.c: In function â??test_closeâ??:
      src/gdb/testsuite/gdb.base/fileio.c:280:1: warning: control reaches end of non-void function [-Wreturn-type]
       }
       ^
    
    Nothing looks at the return of these functions, so just make them
    return void.  While at it, "()" is not the same as "(void)" in C - fix
    that too.
    
    gdb/ChangeLog:
    2017-05-18  Pedro Alves  <palves@redhat.com>
    
    	* gdb.base/fileio.c (stop, test_open, test_write, test_read)
    	(test_lseek, test_close, test_stat, test_fstat, test_isatty)
    	(test_system, test_rename, test_unlink, test_time): Change
    	prototypes.
    	* gdb.base/fileio.exp (stop_msg): Adjust.

Diff:
---
 gdb/testsuite/ChangeLog           |  8 +++++++
 gdb/testsuite/gdb.base/fileio.c   | 50 +++++++++++++++++++--------------------
 gdb/testsuite/gdb.base/fileio.exp |  2 +-
 3 files changed, 34 insertions(+), 26 deletions(-)

diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index 2ebddca..428056e 100644
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,5 +1,13 @@
 2017-05-18  Pedro Alves  <palves@redhat.com>
 
+	* gdb.base/fileio.c (stop, test_open, test_write, test_read)
+	(test_lseek, test_close, test_stat, test_fstat, test_isatty)
+	(test_system, test_rename, test_unlink, test_time): Change
+	prototypes.
+	* gdb.base/fileio.exp (stop_msg): Adjust.
+
+2017-05-18  Pedro Alves  <palves@redhat.com>
+
 	* gdb.base/fileio.c (null_str): New global.
 	(test_stat): Use it.
 	* gdb.base/fileio.exp: Remove nowarnings.
diff --git a/gdb/testsuite/gdb.base/fileio.c b/gdb/testsuite/gdb.base/fileio.c
index 38537db..415f2d0 100644
--- a/gdb/testsuite/gdb.base/fileio.c
+++ b/gdb/testsuite/gdb.base/fileio.c
@@ -74,14 +74,14 @@ static const char *strerrno (int err);
 
 #define STRING      "Hello World"
 
-static void stop () {}
+static void stop (void) {}
 
 /* A NULL string.  We pass this to stat below instead of a NULL
    literal to avoid -Wnonnull warnings.  */
 const char *null_str;
 
-int
-test_open ()
+void
+test_open (void)
 {
   int ret;
 
@@ -140,8 +140,8 @@ test_open ()
   stop ();
 }
 
-int
-test_write ()
+void
+test_write (void)
 {
   int fd, ret;
 
@@ -181,8 +181,8 @@ test_write ()
   stop ();
 }
 
-int
-test_read ()
+void
+test_read (void)
 {
   int fd, ret;
   char buf[16];
@@ -213,8 +213,8 @@ test_read ()
   stop ();
 }
 
-int
-test_lseek ()
+void
+test_lseek (void)
 {
   int fd;
   off_t ret = 0;
@@ -255,8 +255,8 @@ test_lseek ()
   stop ();
 }
 
-int
-test_close ()
+void
+test_close (void)
 {
   int fd, ret;
 
@@ -281,8 +281,8 @@ test_close ()
   stop ();
 }
 
-int
-test_stat ()
+void
+test_stat (void)
 {
   int ret;
   struct stat st;
@@ -316,8 +316,8 @@ test_stat ()
   stop ();
 }
 
-int
-test_fstat ()
+void
+test_fstat (void)
 {
   int fd, ret;
   struct stat st;
@@ -347,8 +347,8 @@ test_fstat ()
   stop ();
 }
 
-int
-test_isatty ()
+void
+test_isatty (void)
 {
   int fd;
 
@@ -377,8 +377,8 @@ test_isatty ()
 
 char sys[1512];
 
-int
-test_system ()
+void
+test_system (void)
 {
   /*
    * Requires test framework to switch on "set remote system-call-allowed 1"
@@ -409,8 +409,8 @@ test_system ()
   stop ();
 }
 
-int
-test_rename ()
+void
+test_rename (void)
 {
   int ret;
   struct stat st;
@@ -464,8 +464,8 @@ test_rename ()
 
 char name[1256];
 
-int
-test_unlink ()
+void
+test_unlink (void)
 {
   int ret;
 
@@ -504,8 +504,8 @@ test_unlink ()
   stop ();
 }
 
-int
-test_time ()
+void
+test_time (void)
 {
   time_t ret, t;
 
diff --git a/gdb/testsuite/gdb.base/fileio.exp b/gdb/testsuite/gdb.base/fileio.exp
index 6bb7141..99afaff 100644
--- a/gdb/testsuite/gdb.base/fileio.exp
+++ b/gdb/testsuite/gdb.base/fileio.exp
@@ -69,7 +69,7 @@ if ![runto_main] then {
 
 gdb_test "break stop" "Breakpoint .*$srcfile.*" 
 
-set stop_msg ".*Breakpoint .* stop \\(\\) at.*$srcfile:.*static void stop \\(\\) {}.*"
+set stop_msg ".*Breakpoint .* stop \\(\\) at.*$srcfile:.*static void stop \\(void\\) {}.*"
 
 gdb_test continue \
 "Continuing\\..*open 1:.*OK$stop_msg" \


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]