From fc555c0a4bf23b3418e0e9cc3a0800172b716b90 Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Thu, 23 Jun 2022 08:12:44 -0700 Subject: 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. --- top/maint.mk | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'top') diff --git a/top/maint.mk b/top/maint.mk index 2740c11d36..c1fdf9ca2c 100644 --- a/top/maint.mk +++ b/top/maint.mk @@ -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: -- cgit v1.2.1