This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH 12/15] Add target/symbol.h, update users
- From: Doug Evans <dje at google dot com>
- To: Gary Benson <gbenson at redhat dot com>
- Cc: Tom Tromey <tromey at redhat dot com>, gdb-patches <gdb-patches at sourceware dot org>
- Date: Fri, 11 Jul 2014 11:29:09 -0700
- Subject: Re: [PATCH 12/15] Add target/symbol.h, update users
- Authentication-results: sourceware.org; auth=none
- References: <1404902255-11101-1-git-send-email-gbenson at redhat dot com> <1404902255-11101-13-git-send-email-gbenson at redhat dot com> <87bnsxdrtj dot fsf at fleche dot redhat dot com> <CADPb22TyZHcu-ee=cT=d9ADiFWEjFhUJGYyv=155NYKOptryxA at mail dot gmail dot com> <87d2dd81of dot fsf at fleche dot redhat dot com> <20140711125653 dot GD2695 at blade dot nx>
On Fri, Jul 11, 2014 at 5:56 AM, Gary Benson <gbenson@redhat.com> wrote:
> Tom Tromey wrote:
>> Doug> As target/* scales up, is there a risk of the code becoming
>> Doug> harder to read if target_ is used as a general prefix for
>> Doug> things in target/*? Dunno. Just wondering.
>>
>> My long term goal is that gdb and gdbserver share the entire target
>> stack. I think these patches further this goal. I don't find the
>> result harder to read at all.
>
> Doug, are you ok for me to leave it as it is, or, do you have an
> alternative you would like me to implement instead?
Hi. I'm not sure TBH.
I think it might be ok as is, but IWBN to see the full patch with comments.