[PATCH v2] Use SaL symbol name when reporting breakpoint locations

Keith Seitz keiths@redhat.com
Fri Oct 27 17:59:00 GMT 2017


On 10/27/2017 07:32 AM, Pedro Alves wrote:
> On 10/20/2017 07:51 PM, Keith Seitz wrote:
> 
>> +
>> +    if {$source != ""} {
>> +	set source "/$source:$line"
> 
> I suspect this '/' breaks with remote host testing.  See e.g.,:
> https://sourceware.org/ml/gdb-patches/2013-04/msg00088.html

Thank you for pointing that out. I did not remember that. [I really need to set up a remote host testing environment here!]

It makes no difference to the test if the '/' is in there (when `source' is added to the final regexp return by this procedure, it is preceded with ".*", so it makes no difference). As in the referenced message/patch, I've removed that character.

> Otherwise LGTM.

Pushed with that change, thank you again!

Keith



More information about the Gdb-patches mailing list