This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH v5] Add Guile frame filter interface
- From: Andy Wingo <wingo at igalia dot com>
- To: Eli Zaretskii <eliz at gnu dot org>
- Cc: gdb-patches at sourceware dot org, dje at google dot com
- Date: Tue, 28 Apr 2015 12:04:17 +0200
- Subject: Re: [PATCH v5] Add Guile frame filter interface
- Authentication-results: sourceware.org; auth=none
- References: <87zj6hfg6t dot fsf at igalia dot com> <87pp7dferl dot fsf at igalia dot com> <83sic9dz6q dot fsf at gnu dot org>
Thanks for the feedback, Eli. Updated patch to come will fix nits. One
comment:
On Thu 09 Apr 2015 18:49, Eli Zaretskii <eliz@gnu.org> writes:
>> From: Andy Wingo <wingo@igalia.com>
>> Cc: dje@google.com
>> Date: Thu, 09 Apr 2015 18:27:42 +0200
>>
>> +frame @var{dec}. Each item of the list should either be a
>> +@value{GDBN} symbol (@pxref{Symbols In Guile}), a pair of a
>> +@value{GDBN} symbol and a @value{GDBN} value (@pxref{Values From
>> +Inferior In Guile}, or a pair of a string and a @value{GDBN} value.
>> +In the first case, the value will be loaded from the frame if needed.
>
> I'm confused: the first case is when the item is a symbol, so what
> value is being alluded to here?
I replaced the last sentence with:
In the first case, the @value{GDBN} symbol will be used to load the
corresponding value from the frame if needed.
Andy