This is the mail archive of the
gdb-cvs@sourceware.org
mailing list for the GDB project.
[binutils-gdb] record-full: Remove unused function netorder16
- From: Simon Marchi <simark at sourceware dot org>
- To: gdb-cvs at sourceware dot org
- Date: 26 Jun 2017 14:52:44 -0000
- Subject: [binutils-gdb] record-full: Remove unused function netorder16
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=873c08142c4c377d2d3ed0b1b1b9e262ddc06c61
commit 873c08142c4c377d2d3ed0b1b1b9e262ddc06c61
Author: Simon Marchi <simon.marchi@ericsson.com>
Date: Mon Jun 26 15:08:36 2017 +0200
record-full: Remove unused function netorder16
clang shows this warning:
/home/emaisin/src/binutils-gdb/gdb/record-full.c:2344:1: error: unused function 'netorder16' [-Werror,-Wunused-function]
netorder16 (uint16_t input)
^
Remove this function, which, AFAIK, has never been used. Note that GCC
doesn't warn about this, because the function is marked as inline.
According to gcc's man page, it should ideed not warn:
-Wunused-function
Warn whenever a static function is declared but not defined or a non-inline static function is unused. This warning is enabled by -Wall.
So it's probably not a GCC bug that it doesn't find this unused function, but a
different definition of "unused".
gdb/ChangeLog:
* record-full.c (netorder16): Remove.
Diff:
---
gdb/ChangeLog | 4 ++++
gdb/record-full.c | 10 ----------
2 files changed, 4 insertions(+), 10 deletions(-)
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index e817dc7..ac5965a 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,9 @@
2017-06-26 Simon Marchi <simon.marchi@ericsson.com>
+ * record-full.c (netorder16): Remove.
+
+2017-06-26 Simon Marchi <simon.marchi@ericsson.com>
+
* common/diagnostics.h: Define macros for GCC.
(DIAGNOSTIC_IGNORE_UNUSED_FUNCTION): New macro.
* common/vec.h: Include diagnostics.h.
diff --git a/gdb/record-full.c b/gdb/record-full.c
index 8e774a2..7f6ecc7 100644
--- a/gdb/record-full.c
+++ b/gdb/record-full.c
@@ -2340,16 +2340,6 @@ netorder32 (uint32_t input)
return ret;
}
-static inline uint16_t
-netorder16 (uint16_t input)
-{
- uint16_t ret;
-
- store_unsigned_integer ((gdb_byte *) &ret, sizeof (ret),
- BFD_ENDIAN_BIG, input);
- return ret;
-}
-
/* Restore the execution log from a core_bfd file. */
static void
record_full_restore (void)