c99 elfxx-riscv.c fix
Palmer Dabbelt
palmer@dabbelt.com
Thu Mar 5 20:27:00 GMT 2020
On Tue, 18 Feb 2020 19:19:46 PST (-0800), amodra@gmail.com wrote:
> I'm not against using c99, but we can't use c99 without enabling c99
> support for older compilers that don't enable c99 by default. So if
> you want to use c99 contructs in binutils you'll need to first arrange
> for -std=c99 to be passed to older compilers.
>
> * elfxx-riscv.c (riscv_multi_letter_ext_valid_p): Don't use C99.
>
> diff --git a/bfd/elfxx-riscv.c b/bfd/elfxx-riscv.c
> index 0a0711ef8d..dc6db0c307 100644
> --- a/bfd/elfxx-riscv.c
> +++ b/bfd/elfxx-riscv.c
> @@ -1401,11 +1401,11 @@ static bfd_boolean
> riscv_multi_letter_ext_valid_p (const char *ext,
> const char *const *known_exts)
> {
> - for (size_t i = 0; known_exts[i]; ++i)
> - {
> - if (!strcmp (ext, known_exts[i]))
> - return TRUE;
> - }
> + size_t i;
> +
> + for (i = 0; known_exts[i]; ++i)
> + if (!strcmp (ext, known_exts[i]))
> + return TRUE;
>
> return FALSE;
> }
Thanks!
More information about the Binutils
mailing list