Message ID | CAGWvnykEhW6pUMWNgjqk4X3nFPevKfRpQ44tX6Vp15=hOy2d9g@mail.gmail.com |
---|---|
State | New |
Headers | show |
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) > { >
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) {