]> git.ipfire.org Git - thirdparty/elfutils.git/commitdiff
elfclassify: Use (void) for no-argument static functions, not ()
authorMark Wielaard <mark@klomp.org>
Sat, 30 Aug 2025 13:47:24 +0000 (15:47 +0200)
committerMark Wielaard <mark@klomp.org>
Sat, 30 Aug 2025 14:02:58 +0000 (16:02 +0200)
In -std=gnu11 mode gcc will error out on nor argument () static
functions:

src/elfclassify.c:757:1: error: function declaration isn’t a prototype
 [-Werror=strict-prototypes]
  757 | check_checks ()
      | ^~~~~~~~~~~~
src/elfclassify.c: In function ‘check_checks’:
src/elfclassify.c:757:1: error: old-style function definition
 [-Werror=old-style-definition]

This isn't an issue in -std=gnu23 mode (the default with GCC 15):

  A definition using () is not considered an old-style definition in
  C23 mode, because it is equivalent to (void) in that case, but is
  considered an old-style definition for older standards

So just always use (void) instead of () for an empty argument list.

* src/elfclassify.c (check_checks): Define as check_checks (void).
(check_ar_members): Define as check_ar_members (void).

Signed-off-by: Mark Wielaard <mark@klomp.org>
src/elfclassify.c

index fd84f90dda9f723b37428b5a97df9f3f00a954ad..307771b23a71999ebd4006ed63269abf1befe419 100644 (file)
@@ -754,7 +754,7 @@ parse_opt (int key, char *arg __attribute__ ((unused)),
 }
 
 static bool
-check_checks ()
+check_checks (void)
 {
   bool checks_passed = true;
   bool checks[] =
@@ -824,7 +824,7 @@ check_checks ()
 }
 
 static bool
-check_ar_members ()
+check_ar_members (void)
 {
   char *ar_path = current_path;
   Elf *ar_elf = elf;