This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[PATCH] avr: simplify some option parsing with strcasecmp ()


From: Trevor Saunders <tbsaunde+binutils@tbsaunde.org>

Hi,

     Instead of canonicalizing the argument we can just use a case insensitive
     compare.

built and regtested a cross to avr-elf, ok?

Trev

gas/ChangeLog:

2016-04-02  Trevor Saunders  <tbsaunde+binutils@tbsaunde.org>

	* config/tc-avr.c (md_parse_option): Use strcasecmp () to compare
	strings.
---
 gas/config/tc-avr.c | 13 +------------
 1 file changed, 1 insertion(+), 12 deletions(-)

diff --git a/gas/config/tc-avr.c b/gas/config/tc-avr.c
index 852002f..999ffb4 100644
--- a/gas/config/tc-avr.c
+++ b/gas/config/tc-avr.c
@@ -562,22 +562,11 @@ md_parse_option (int c, const char *arg)
     case OPTION_MMCU:
       {
 	int i;
-	char *s = xmalloc (strlen (arg) + 1);
-
-	{
-	  char *t = s;
-	  const char *arg1 = arg;
-
-	  do
-	    *t = TOLOWER (*arg1++);
-	  while (*t++);
-	}
 
 	for (i = 0; mcu_types[i].name; ++i)
-	  if (strcmp (mcu_types[i].name, s) == 0)
+	  if (strcasecmp (mcu_types[i].name, arg) == 0)
 	    break;
 
-	free (s);
 	if (!mcu_types[i].name)
 	  {
 	    show_mcu_list (stderr);
-- 
2.1.4


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]