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: [rfc] [04/12] Get rid of current_gdbarch in frame.c


Markus Deuling <deuling@de.ibm.com> writes:
> this patch gets rid of some of the current_gdbarch's in frame.c
>
> Is this ok to commit?

Yes, this looks fine --- thank you!

It took me a bit to realize that "FRAME" and "REGCACHE" in your
ChangeLog entry were references to function arguments.  But those
aren't really the names the arguments have in those functions.  Could
you say "Use the architecture of the frame or regcache values received
as arguments"?


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