Command names: make them case sensitive
authorJerome Guitton <guitton@adacore.com>
Fri, 27 Jan 2017 16:06:32 +0000 (17:06 +0100)
committerJerome Guitton <guitton@adacore.com>
Wed, 8 Feb 2017 17:57:31 +0000 (18:57 +0100)
Case-insensitive search for command names is an obscure undocumented
feature, which seems to be unused, is not tested and not quite
consistent. Remove it.

gdb/ChangeLog:

* cli-decode.c (lookup_cmd_1, lookup_cmd_composition):
Remove case-insensitive search.

gdb/ChangeLog
gdb/cli/cli-decode.c

index 71ea08d938323b2be2617b4284f90ed76daf19ee..fd90cdcc4c0615a6dc12013c6dd17bb9fb318c11 100644 (file)
@@ -1,3 +1,8 @@
+2017-02-08  Jerome Guitton  <guitton@adacore.com>
+
+       * cli-decode.c (lookup_cmd_1, lookup_cmd_composition):
+       Remove case-insensitive search.
+
 2017-02-07  Jose E. Marchesi  <jose.marchesi@oracle.com>
 
        * sparc-tdep.c (sparc32_gdbarch_init): Do not place a + operator
index 436a7ed4ca92ea177b0d0219e332afca91c19ab0..155d6d18070b91a52bc20b5d56228bc6178952c8 100644 (file)
@@ -1378,19 +1378,6 @@ lookup_cmd_1 (const char **text, struct cmd_list_element *clist,
   nfound = 0;
   found = find_cmd (command, len, clist, ignore_help_classes, &nfound);
 
-  /* We didn't find the command in the entered case, so lower case it
-     and search again.  */
-  if (!found || nfound == 0)
-    {
-      for (tmp = 0; tmp < len; tmp++)
-       {
-         char x = command[tmp];
-
-         command[tmp] = isupper (x) ? tolower (x) : x;
-       }
-      found = find_cmd (command, len, clist, ignore_help_classes, &nfound);
-    }
-
   /* If nothing matches, we have a simple failure.  */
   if (nfound == 0)
     return 0;
@@ -1731,20 +1718,6 @@ lookup_cmd_composition (const char *text,
       nfound = 0;
       *cmd = find_cmd (command, len, cur_list, 1, &nfound);
       
-      /* We didn't find the command in the entered case, so lower case
-        it and search again.
-      */
-      if (!*cmd || nfound == 0)
-       {
-         for (tmp = 0; tmp < len; tmp++)
-           {
-             char x = command[tmp];
-
-             command[tmp] = isupper (x) ? tolower (x) : x;
-           }
-         *cmd = find_cmd (command, len, cur_list, 1, &nfound);
-       }
-      
       if (*cmd == CMD_LIST_AMBIGUOUS)
        {
          return 0;              /* ambiguous */