diff mbox series

lib: remove unused `STROF` definition (NFC)

Message ID 20210827152013.4061770-1-abdulras@google.com
State Committed
Headers
Series lib: remove unused `STROF` definition (NFC) |

Commit Message

Saleem Abdulrasool Aug. 27, 2021, 3:20 p.m. UTC 
  This definition was in the fallback path, where `sys/cdefs.h` is not
available.  Now that we have a single path through here, this macro gets
defined, though is unused.  Remove the unused macro definition.

Signed-off-by: Saleem Abdulrasool <abdulras@google.com>
---
 lib/ChangeLog       | 4 ++++
 lib/fixedsizehash.h | 1 -
 2 files changed, 4 insertions(+), 1 deletion(-)

Comments

Mark Wielaard Aug. 27, 2021, 4:31 p.m. UTC | #1 
Hi Saleem,

On Fri, 2021-08-27 at 15:20 +0000, Saleem Abdulrasool via Elfutils-devel wrote:
> This definition was in the fallback path, where `sys/cdefs.h` is not
> available.  Now that we have a single path through here, this macro gets
> defined, though is unused.  Remove the unused macro definition.

The STROF macro is not used, so this is indeed a NFC.
Pushed.

Thanks,

Mark
diff mbox series

Patch

diff --git a/lib/ChangeLog b/lib/ChangeLog
index a95f8041..5f1de1e2 100644
--- a/lib/ChangeLog
+++ b/lib/ChangeLog
@@ -1,3 +1,7 @@ 
+2021-08-21  Saleem Abdulrasool  <abdulras@google.com>
+
+	* fixedsizehash.h: Remove unused STROF macro.
+
 2021-08-20  Saleem Abdulrasool  <abdulras@google.com>
 
 	* fixedsizehash.h: Remove sys/cdefs.h include. Unconditionally
diff --git a/lib/fixedsizehash.h b/lib/fixedsizehash.h
index f333ad99..14f0fb88 100644
--- a/lib/fixedsizehash.h
+++ b/lib/fixedsizehash.h
@@ -33,7 +33,6 @@ 
 
 #include <system.h>
 
-#define STROF(t2) t2
 #define CONCAT_EXPANDED(t1,t2) t1 ## t2
 #define CONCAT(t1,t2) CONCAT_EXPANDED(t1,t2)