Remove dead serial_interface_lookup calls
authorSimon Marchi <simon.marchi@ericsson.com>
Thu, 12 Jan 2017 15:39:35 +0000 (10:39 -0500)
committerSimon Marchi <simon.marchi@ericsson.com>
Thu, 12 Jan 2017 15:41:50 +0000 (10:41 -0500)
By inspecting the serial_add_interface calls, I found that the serial
interface names that we have today are:

 - hardwire
 - terminal
 - pipe
 - tcp
 - event

 The calls to serial_interface_lookup with any other names are most
 likely leftovers which can be removed since these serial interfaces
 don't exist anymore.  The commits that removed the "pc" and "parallel"
 interfaces are respectively:

  cb2a4ac5dae478fcd9d6e772530c3aba0576fc7a

and

  e386d4d2fb55042f77d0557a0849ed2464aee7b3

gdb/ChangeLog:

* serial.c (serial_open): Forget about "pc" and "lpt" serial interface.

gdb/ChangeLog
gdb/serial.c

index 89bbb8fbaf71f94766be28f75463ba8c1bc4a053..153f6cd7df6fb422a2210c1a989c8754b1e8d68e 100644 (file)
@@ -1,3 +1,7 @@
+2017-01-12  Simon Marchi  <simon.marchi@ericsson.com>
+
+       * serial.c (serial_open): Forget about "pc" and "lpt" serial interface.
+
 2017-01-11  Simon Marchi  <simon.marchi@ericsson.com>
 
        * cli/cli-decode.c (lookup_cmd_1): Fix typo in comment.
index ac7c1b99c1751023469263c95b6b5e75bf827958..afb70758c6022c3d2fcc424b129ac22fec5f90d7 100644 (file)
@@ -209,11 +209,7 @@ serial_open (const char *name)
   const struct serial_ops *ops;
   const char *open_name = name;
 
-  if (strcmp (name, "pc") == 0)
-    ops = serial_interface_lookup ("pc");
-  else if (startswith (name, "lpt"))
-    ops = serial_interface_lookup ("parallel");
-  else if (startswith (name, "|"))
+  if (startswith (name, "|"))
     {
       ops = serial_interface_lookup ("pipe");
       /* Discard ``|'' and any space before the command itself.  */