This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 09/10] Move register_dump to regcache-dump.c


On 2018-02-21 08:58, Yao Qi wrote:
On Wed, Feb 21, 2018 at 12:57 PM, Simon Marchi <simon.marchi@polymtl.ca> wrote:

register_dump::dump accesses 'struct regcache_desc', which is declared in
regcache.c, so that register_dump::dump can't be moved to
regcache-dump.c.


Is there any reason we can't move struct regcache_descr to regcache.h?

The reason I didn't move is that I want to keep regcache_descr opaque.
To be clear, I don't mind moving regcache_descr to regcache.h and move
all regcache dump stuff into regcache-dump.{c,h}.

I think it would make a more logical split, but I'll let you decide :)


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]