Hi,
Samuel Tardieu wrote:
> diff --git a/lib/readutmp.c b/lib/readutmp.c
> index 0938a094cb..fd84a1e4de 100644
> --- a/lib/readutmp.c
> +++ b/lib/readutmp.c
> @@ -301,7 +301,7 @@ finish_utmp (struct utmp_alloc a)
>
> /* Determine whether A already contains an entry of type BOOT_TIME. */
> _GL_ATTRIBUTE_MAYBE_UNUSED
> -static bool
> +static bool _GL_ATTRIBUTE_PURE
> have_boot_time (struct utmp_alloc a)
> {
> for (idx_t i = 0; i < a.filled; i++)
>
No, thanks. This is a 'static' function. It is a waste of time for us
developers to add attributes in order to declare information that GCC
has already found out by itself. See
<https://gcc.gnu.org/bugzilla/show_bug.cgi?id=109914> for details.
You can disable the --suggest-attribute=pure option through the
mechanism described in
<https://www.gnu.org/software/gnulib/manual/html_node/manywarnings.html>.
Bruno
