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