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 12/32] Add target_ops argument to to_thread_name


On 01/15/2014 04:45 PM, Tom Tromey wrote:
> Pedro> The patch is fine with me as is, but I'll note that I don't
> Pedro> think there's any need for exec_set_find_memory_regions
> Pedro> nowadays.  Seems to me that exec_ops.to_find_memory_regions could
> Pedro> always be set to objfile_find_memory_regions unconditionally.
> 
> Good point.
> 
> Here is a patch I've added to my branch to fix this up.
> It's cleaner and also more obviously correct in the multi-target case.

Thanks.  Looks good to me.

-- 
Pedro Alves


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