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 3/5] s/struct process_info/struct inferior/


On 03/21/2012 12:28 AM, Doug Evans wrote:
> One thought that comes to mind is that if this renaming is good, then
> it's incomplete: rename current_process to current_inferior, and make
> similar changes throughout gdb.  E.g., don't just change "struct
> process_info *process" to "struct inferior *process", change it to
> "struct inferior *inferior".
> Another example: rename linux_add_process to linux_add_inferior.
> [I realize it's more work, but to me the patch leaves gdb in a
> half-changed state otherwise.]

Agreed.  If this change is a right way to go, I'll complete the left
renaming.  I was not sure about people's thoughts on this change, so
sent a half-baked patch first.

-- 
Yao (éå)


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