diff options
author | Pedro Alves <palves@redhat.com> | 2013-04-08 14:09:30 +0000 |
---|---|---|
committer | Pedro Alves <palves@redhat.com> | 2013-04-08 14:09:30 +0000 |
commit | f6de8ec2620da62f46cb4e3887b981677cdd9cdb (patch) | |
tree | b9a0d682130d2443d1a850a87384ee8f005b17f8 /gdb/breakpoint.h | |
parent | f65ce5fb99ce664302abe4af1263fdbb0518d069 (diff) | |
download | gdb-f6de8ec2620da62f46cb4e3887b981677cdd9cdb.zip gdb-f6de8ec2620da62f46cb4e3887b981677cdd9cdb.tar.gz gdb-f6de8ec2620da62f46cb4e3887b981677cdd9cdb.tar.bz2 |
create_breapoint / explicit mode: Error out if there's garbage after the breakpoint location.
If !PARSE_CONDITION_AND_THREAD, then ARG is just the location, nothing
else. The fact that the describing comment of create_breakpoint
doesn't mention this just looks like an oversight of when extra_string
was added. "parse_condition_and_thread" has been a misnomer ever
since extra_string was added -- better rename it avoid more confusion.
This makes it "parse_arg", as that'll remain stable even if/when more
explicit parameters are added.
gdb/
2013-04-08 Pedro Alves <palves@redhat.com>
Keith Seitz <keiths@redhat.com>
* breakpoint.c (create_breakpoint): Rename
"parse_condition_and_thread" parameter to "parse_arg". Update
describing comment. If !PARSE_ARG, then error out if ARG is not
the empty string after extracting the location.
* breakpoint.h (create_breakpoint): Rename
"parse_condition_and_thread" parameter to "parse_arg".
gdb/testsuite/
2013-04-08 Pedro Alves <palves@redhat.com>
* gdb.mi/mi-break.exp (test_error): Add tests with garbage after
the location.
Diffstat (limited to 'gdb/breakpoint.h')
-rw-r--r-- | gdb/breakpoint.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gdb/breakpoint.h b/gdb/breakpoint.h index 68f3ed9..d740625 100644 --- a/gdb/breakpoint.h +++ b/gdb/breakpoint.h @@ -1269,7 +1269,7 @@ enum breakpoint_create_flags extern int create_breakpoint (struct gdbarch *gdbarch, char *arg, char *cond_string, int thread, char *extra_string, - int parse_condition_and_thread, + int parse_arg, int tempflag, enum bptype wanted_type, int ignore_count, enum auto_boolean pending_break_support, |