From f73e41ef3703be34db1c7e08271e6ac8bd758144 Mon Sep 17 00:00:00 2001 From: Trevor Saunders Date: Sat, 2 Apr 2016 07:57:10 -0400 Subject: [PATCH] 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 * config/tc-avr.c (md_parse_option): Use strcasecmp () to compare strings. --- gas/ChangeLog | 5 +++++ gas/config/tc-avr.c | 13 +------------ 2 files changed, 6 insertions(+), 12 deletions(-) diff --git a/gas/ChangeLog b/gas/ChangeLog index 6c763918d53..8d6ba9b6b95 100644 --- a/gas/ChangeLog +++ b/gas/ChangeLog @@ -1,3 +1,8 @@ +2016-04-03 Trevor Saunders + + * config/tc-avr.c (md_parse_option): Use strcasecmp () to compare + strings. + 2016-04-02 Alan Modra PR 19896 diff --git a/gas/config/tc-avr.c b/gas/config/tc-avr.c index 852002f2f63..999ffb48715 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.30.2