[PATCH,FORTRAN] Variable "value" possibly unused in io/read.c

Tobias Burnus burnus@net-b.de
Sat May 4 21:14:00 GMT 2013


David Edelsohn wrote:
> All uses of "value" are protected by
> #if defined HAVE_GFC_REAL_16 || defined HAVE_GFC_REAL_10
> so it may be unused, which generates a warning on some platforms.
>
> Bootstrapped on powerpc-ibm-aix7.1.0.0.
> Okay for trunk?
>
>         * io/read.c (si_max): Annotate value with attribute unused.

I think using

#if defined HAVE_GFC_REAL_16 || defined HAVE_GFC_REAL_10
   GFC_UINTEGER_LARGEST value;
#endif

is cleaner than using the unused attribute.

OK with that change. And thanks for the the cleanup!

Tobias

> Index: io/read.c
> ===================================================================
> --- io/read.c   (revision 198587)
> +++ io/read.c   (working copy)
> @@ -91,7 +91,7 @@
>   GFC_UINTEGER_LARGEST
>   si_max (int length)
>   {
> -  GFC_UINTEGER_LARGEST value;
> +  GFC_UINTEGER_LARGEST value __attribute__((unused));
>
>     switch (length)
>         {
>



More information about the Gcc-patches mailing list