c99 elfxx-riscv.c fix

Alan Modra amodra@gmail.com
Wed Feb 19 03:19:00 GMT 2020


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;
 }

-- 
Alan Modra
Australia Development Lab, IBM



More information about the Binutils mailing list