Enable/update legacy linespecs in MI.

MI is currently using string_to_event_location to enable the use of legacy
linespecs, but using this function (until this patchset) had the (as yet
unnoticed) side effect of allowing both MI and CLI representation for
explicit locations.

This patch simply changes MI to use the same legacy linespec functions
that the python and guile interpreters use.  This eliminates the CLI syntax
for explicit locations (in MI).

gdb/ChangeLog

	* mi/mi-cmd-break.c (mi_cmd_break_insert_1): Use
	string_to_event_location_basic instead of string_to_event_location.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index f7591ff192f93155941ef8c79ae6d37f8a0c7003..04daf07eef15a792c50c706be20e90ee6e06c8c4 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2016-02-09  Keith Seitz  <keiths@redhat.com>
+
+       * mi/mi-cmd-break.c (mi_cmd_break_insert_1): Use
+       string_to_event_location_basic instead of string_to_event_location.
+
 2016-02-09  Keith Seitz  <keiths@redhat.com>
 
        * guile/scm-breakpoint.c (gdbscm_register_breakpoint_x): Skip

diff --git a/gdb/mi/mi-cmd-break.c b/gdb/mi/mi-cmd-break.c
index ef3ce29b476f13894bde0cdaccfbe0c70e031d16..3d4062925b32d53f4e2e18bfc8b179407119186f 100644 (file)
--- a/gdb/mi/mi-cmd-break.c
+++ b/gdb/mi/mi-cmd-break.c
@@ -341,7 +341,7 @@ mi_cmd_break_insert_1 (int dprintf, char *command, char **argv, int argc)
     }
   else
     {
-      location = string_to_event_location (&address, current_language);
+      location = string_to_event_location_basic (&address, current_language);
       if (*address)
        {
          delete_event_location (location);