[patch] Allow "disassemble 'Foo::bar(char *)'"
Paul Pluzhnikov
ppluzhnikov@google.com
Sat Nov 21 01:09:00 GMT 2009
On Fri, Nov 20, 2009 at 4:59 PM, Daniel Jacobowitz <drow@false.org> wrote:
>> Does this require a NEWS entry (incompatible change to disassemble
>> command) ?
>
> I think yes.
Thanks, NEWS mention added.
--
Paul Pluzhnikov
gdb/ChangeLog:
2009-11-20 Paul Pluzhnikov <ppluzhnikov@google.com>
* cli/cli-cmds.c (disassemble_command): Split on comma.
(init_cli_cmds): Update help.
* NEWS: Mention incompatible change to 'disassemble'.
gdb/doc/ChangeLog:
2009-11-20 Paul Pluzhnikov <ppluzhnikov@google.com>
* gdb.texinfo (Machine Code): Adjust.
gdb/testsuite/ChangeLog:
2009-11-20 Paul Pluzhnikov <ppluzhnikov@google.com>
* gdb.asm/asm-source.exp: Adjust.
* gdb.base/help.exp: Adjust.
-------------- next part --------------
Index: NEWS
===================================================================
RCS file: /cvs/src/src/gdb/NEWS,v
retrieving revision 1.337
diff -u -p -u -r1.337 NEWS
--- NEWS 1 Nov 2009 21:20:18 -0000 1.337
+++ NEWS 21 Nov 2009 01:06:06 -0000
@@ -22,6 +22,12 @@ Xilinx MicroBlaze microblaze
lists inferiors that are not running yet or that have exited
already. See also "New commands" and "New options" below.
+* Changed commands
+
+disassemble
+ The disassemble command, when invoked with two arguments, now requires
+ the arguments to be comma-separated.
+
* New commands (for set/show, see "New options" below)
record save [<FILENAME>]
Index: cli/cli-cmds.c
===================================================================
RCS file: /cvs/src/src/gdb/cli/cli-cmds.c,v
retrieving revision 1.93
diff -u -p -u -r1.93 cli-cmds.c
--- cli/cli-cmds.c 23 Oct 2009 00:49:33 -0000 1.93
+++ cli/cli-cmds.c 21 Nov 2009 01:06:06 -0000
@@ -983,7 +983,6 @@ disassemble_command (char *arg, int from
CORE_ADDR low, high;
char *name;
CORE_ADDR pc, pc_masked;
- char *space_index;
int flags;
name = NULL;
@@ -1022,13 +1021,12 @@ disassemble_command (char *arg, int from
return;
}
- /* FIXME: 'twould be nice to allow spaces in the expression for the first
- arg. Allow comma separater too? */
-
- if (!(space_index = (char *) strchr (arg, ' ')))
+ pc = value_as_address (parse_to_comma_and_eval (&arg));
+ if (arg[0] == ',')
+ ++arg;
+ if (arg[0] == '\0')
{
/* One argument. */
- pc = parse_and_eval_address (arg);
if (find_pc_partial_function (pc, &name, &low, &high) == 0)
error (_("No function contains specified address."));
#if defined(TUI)
@@ -1044,9 +1042,8 @@ disassemble_command (char *arg, int from
else
{
/* Two arguments. */
- *space_index = '\0';
- low = parse_and_eval_address (arg);
- high = parse_and_eval_address (space_index + 1);
+ low = pc;
+ high = parse_and_eval_address (arg);
}
print_disassembly (gdbarch, name, low, high, flags);
@@ -1461,7 +1458,7 @@ Default is the function surrounding the
With a /m modifier, source lines are included (if available).\n\
With a /r modifier, raw instructions in hex are included.\n\
With a single argument, the function surrounding that address is dumped.\n\
-Two arguments are taken as a range of memory to dump."));
+Two arguments (separated by comma) are taken as a range of memory to dump."));
set_cmd_completer (c, location_completer);
if (xdb_commands)
add_com_alias ("va", "disassemble", class_xdb, 0);
Index: doc/gdb.texinfo
===================================================================
RCS file: /cvs/src/src/gdb/doc/gdb.texinfo,v
retrieving revision 1.641
diff -u -p -u -r1.641 gdb.texinfo
--- doc/gdb.texinfo 2 Nov 2009 14:59:52 -0000 1.641
+++ doc/gdb.texinfo 21 Nov 2009 01:06:07 -0000
@@ -6539,10 +6539,10 @@ in symbolic form by specifying the @code
The default memory range is the function surrounding the
program counter of the selected frame. A single argument to this
command is a program counter value; @value{GDBN} dumps the function
-surrounding this value. Two arguments specify a range of addresses
-(first inclusive, second exclusive) to dump. In that case, the name of
-the function is also printed (since there could be several functions in
-the given range).
+surrounding this value. Two arguments (separated by comma) specify a
+range of addresses (first inclusive, second exclusive) to dump. In that
+case, the name of the function is also printed (since there could be
+several functions in the given range).
If the range of memory being disassembled contains current program counter,
the instruction at that location is shown with a @code{=>} marker.
Index: testsuite/gdb.asm/asm-source.exp
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/gdb.asm/asm-source.exp,v
retrieving revision 1.75
diff -u -p -u -r1.75 asm-source.exp
--- testsuite/gdb.asm/asm-source.exp 23 Oct 2009 00:49:33 -0000 1.75
+++ testsuite/gdb.asm/asm-source.exp 21 Nov 2009 01:06:07 -0000
@@ -453,12 +453,12 @@ proc test_dis { command var } {
# See if we can look at a global variable, three ways
gdb_test "print globalvar" ".* = 11" "look at global variable"
test_dis "x/i &globalvar" "globalvar"
-test_dis "disassem &globalvar &globalvar+1" "globalvar"
+test_dis "disassem &globalvar, &globalvar+1" "globalvar"
# See if we can look at a static variable, three ways
gdb_test "print staticvar" ".* = 5" "look at static variable"
test_dis "x/i &staticvar" "staticvar"
-test_dis "disassem &staticvar &staticvar+1" "staticvar"
+test_dis "disassem &staticvar, &staticvar+1" "staticvar"
# See if we can look at a static function
gdb_test "disassem foostatic" ".*<\\+0>:.*End of assembler dump." \
Index: testsuite/gdb.base/help.exp
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/gdb.base/help.exp,v
retrieving revision 1.35
diff -u -p -u -r1.35 help.exp
--- testsuite/gdb.base/help.exp 13 Aug 2009 14:58:27 -0000 1.35
+++ testsuite/gdb.base/help.exp 21 Nov 2009 01:06:07 -0000
@@ -124,7 +124,7 @@ gdb_test "help disable breakpoints" "Dis
# test help disable display
gdb_test "help disable display" "Disable some expressions to be displayed when program stops\.\[\r\n\]+Arguments are the code numbers of the expressions to stop displaying\.\[\r\n\]+No argument means disable all automatic-display expressions\.\[\r\n\]+Do \"info display\" to see current list of code numbers\." "help disable display"
# test help disassemble
-gdb_test "help disassemble" "Disassemble a specified section of memory\.\[\r\n\]+Default is the function surrounding the pc of the selected frame\.\[\r\n\]+With a /m modifier, source lines are included \\(if available\\)\.\[\r\n\]+With a /r modifier, raw instructions in hex are included\.\[\r\n\]+With a single argument, the function surrounding that address is dumped\.\[\r\n\]+Two arguments are taken as a range of memory to dump\." "help disassemble"
+gdb_test "help disassemble" "Disassemble a specified section of memory\.\[\r\n\]+Default is the function surrounding the pc of the selected frame\.\[\r\n\]+With a /m modifier, source lines are included \\(if available\\)\.\[\r\n\]+With a /r modifier, raw instructions in hex are included\.\[\r\n\]+With a single argument, the function surrounding that address is dumped\.\[\r\n\]+Two arguments \\(separated by comma\\) are taken as a range of memory to dump\." "help disassemble"
# test help display
gdb_test "help display" "Print value of expression EXP each time the program stops\.\[\r\n\]+/FMT may be used before EXP as in the \"print\" command\.\[\r\n\]+/FMT \"i\" or \"s\" or including a size-letter is allowed,\[\r\n\]+as in the \"x\" command, and then EXP is used to get the address to examine\[\r\n\]+and examining is done as in the \"x\" command\.\[\r\n\]+With no argument, display all currently requested auto-display expressions\.\[\r\n\]+Use \"undisplay\" to cancel display requests previously made\." "help display"
# test help do
More information about the Gdb-patches
mailing list