From: Trevor Saunders <tbsaunde+binutils@tbsaunde.org>
Date: Sat, 2 Apr 2016 11:57:10 +0000 (-0400)
Subject: avr: simplify some option parsing with strcasecmp ()
X-Git-Url: http://git.efficios.com/?a=commitdiff_plain;h=f73e41ef3703be34db1c7e08271e6ac8bd758144;p=deliverable%2Fbinutils-gdb.git

avr: simplify some option parsing with strcasecmp ()

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

gas/ChangeLog:

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

	* config/tc-avr.c (md_parse_option): Use strcasecmp () to compare
	strings.
---

diff --git a/gas/ChangeLog b/gas/ChangeLog
index 6c763918d5..8d6ba9b6b9 100644
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,8 @@
+2016-04-03  Trevor Saunders  <tbsaunde+binutils@tbsaunde.org>
+
+	* config/tc-avr.c (md_parse_option): Use strcasecmp () to compare
+	strings.
+
 2016-04-02  Alan Modra  <amodra@gmail.com>
 
 	PR 19896
diff --git a/gas/config/tc-avr.c b/gas/config/tc-avr.c
index 852002f2f6..999ffb4871 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);