[PATCH] bfd: Remove unused static find function from doc/chew.c

Mark Wielaard mark@klomp.org
Fri Nov 8 10:12:00 GMT 2024


Hi Jan,

On Fri, Nov 08, 2024 at 08:18:01AM +0100, Jan Beulich wrote:
> On 07.11.2024 18:40, Mark Wielaard wrote:
> > After commit 2e60790cf7c27d79f90f2dcb81e1930dc980bc1c "Remove the
> > paramstuff word" there is no caller left of the static find function
> > in doc/chew.c, so it can be removed.
> > 
> >    * doc/chew.c (find): Remove.
> 
> In case it's not deemed obvious anyway: Okay. Please consider replacing
> "can be" by "should be",

Pushed with that commit wording change.

> though - I'm surprised this didn't cause build
> issues, as compilers typically warn about unused statics.

bfd/doc/chew.c is compiled without warning flags enabled because it
has a special rule in bfd/doc/local.mk that doesn't include
WARN_CFLAGS_FOR_BUILD.

Maybe it should because enabling that flag shows various
warning/errors that should probably be cleaned up.

Cheers,

Mark


More information about the Binutils mailing list