This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] Add fullname field for MI -break-info command
- From: Daniel Jacobowitz <drow at false dot org>
- To: Vladimir Prus <ghost at cs dot msu dot su>
- Cc: gdb-patches at sources dot redhat dot com
- Date: Wed, 1 Feb 2006 17:32:32 -0500
- Subject: Re: [PATCH] Add fullname field for MI -break-info command
- References: <200601241932.16022.ghost@cs.msu.su> <20060124165136.GF28357@brasko.net> <drfqem$kni$1@sea.gmane.org>
On Sat, Jan 28, 2006 at 04:09:42PM +0300, Vladimir Prus wrote:
> Revised patch attached. I did verify that it causes no regressions for MI
> tests.
>
>
> Note, however, that I could not verify that it does not cause any
> regressions globally, because something seems wrong with my configuration
> or the test system. Sporadically, I get errors like this:
What system are you testing on? Presumably a bug in the host or in the
port of expect to the host.
I've verified no regressions on x86_64-pc-linux-gnu.
The patch looks OK. There's a couple of formatting errors, and
it needs ChangeLog entries, and then it can be applied.
Oh, and Nick was right: you don't need the loc_type check any more.
Everything in that case statement will be a breakpoint and have
a valid address.
> + struct symtab_and_line sal = find_pc_line (b->loc->address, 0);
> + char* fullname = symtab_to_fullname (sal.symtab);
char *fullname, please, for consistency.
> + if (fullname)
> + {
> + ui_out_field_string (uiout, "fullname", fullname);
> + }
Don't need the extra braces here.
> @@ -75,25 +78,27 @@
> # -break-insert -t srcfile:$line_callee4_head
> # -break-list
>
> +
> +
> mi_gdb_test "222-break-insert -t main" \
> - "222\\^done,bkpt=\{number=\"1\",type=\"breakpoint\",disp=\"del\",enabled=\"y\",addr=\"$hex\",func=\"main\",file=\".*basics.c\",line=\"$line_main_body\",times=\"0\"\}" \
> + "222\\^done,bkpt=\{number=\"1\",type=\"breakpoint\",disp=\"del\",enabled=\"y\",addr=\"$hex\",func=\"main\",file=\".*basics.c\",${fullname},line=\"$line_main_body\",times=\"0\"\}" \
> "break-insert -t operation"
And don't need the extra blank lines.
--
Daniel Jacobowitz
CodeSourcery