]> Savannah Git Hosting - gnulib.git/commitdiff
maint.mk: fix syntax-check for unnecessary hash.h header use
authorJim Meyering <meyering@fb.com>
Thu, 23 Jun 2022 15:12:44 +0000 (08:12 -0700)
committerBruno Haible <bruno@clisp.org>
Wed, 31 Aug 2022 23:15:36 +0000 (01:15 +0200)
* top/maint.mk (_hash_re): Correct the regexp. It had two problems:
- lacked the "hash_" prefix, so would match non-uses
- failed to detect uses of the hash_x?initialize functions
The former led to an excess/unneeded use of hash.h in coreutils' cut.c.
The latter would have led to a FP "not needed" report for copy.c.

ChangeLog
top/maint.mk

index 20d5f58e52e568b64fdc2782ca56401c7984d942..a5b6552d0a974b2a62056d8b207b99432030899f 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2022-06-23  Jim Meyering  <meyering@fb.com>
+
+       maint.mk: fix syntax-check for unnecessary hash.h header use
+       * top/maint.mk (_hash_re): Correct the regexp. It had two problems:
+       - lacked the "hash_" prefix, so would match non-uses
+       - failed to detect uses of the hash_x?initialize functions
+       The former led to an excess/unneeded use of hash.h in coreutils' cut.c.
+       The latter would have led to a FP "not needed" report for copy.c.
+
 2022-06-22  Bruno Haible  <bruno@clisp.org>
 
        fchmodat: Fix test failure on native Windows.
index 92cef425cf350409214c6ed62c8fa1030a71e110..9696c87bad73f3eb2e85775a8657de503976937b 100644 (file)
@@ -622,9 +622,9 @@ sc_prohibit_xalloc_without_use:
          $(_sc_header_without_use)
 
 # Extract function names:
-# perl -lne '/^(?:extern )?(?:void|char) \*?(\w+) *\(/ and print $1' lib/hash.h
+# perl -lne '/^(?:extern )?(?:void|char|Hash_table) \*?(\w+) *\(/ and print $1' lib/hash.h
 _hash_re = \
-clear|delete|free|get_(first|next)|insert|lookup|print_statistics|reset_tuning
+hash_(re(set_tuning|move)|xin(itialize|sert)|in(itialize|sert)|get_(firs|nex)t|print_statistics|(delet|fre)e|lookup|clear)
 _hash_fn = \<($(_hash_re)) *\(
 _hash_struct = (struct )?\<[Hh]ash_(table|tuning)\>
 sc_prohibit_hash_without_use: