This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH 1/3] Add fbsd_nat_add_target.
- From: Pedro Alves <palves at redhat dot com>
- To: John Baldwin <jhb at freebsd dot org>, gdb-patches at sourceware dot org
- Date: Mon, 27 Apr 2015 20:10:18 +0100
- Subject: Re: [PATCH 1/3] Add fbsd_nat_add_target.
- Authentication-results: sourceware.org; auth=none
- References: <4032488 dot W8nPzteMFC at ralph dot baldwin dot cx>
On 04/26/2015 02:24 AM, John Baldwin wrote:
> Add a wrapper for add_target in fbsd-nat.c to override target operations
> common to all native FreeBSD targets.
>
> gdb/ChangeLog:
>
> * fbsd-nat.c (fbsd_pid_to_exec_file): Mark static.
> (fbsd_find_memory_regions): Mark static.
> (fbsd_nat_add_target): New function.
> * fbsd-nat.h: Export fbsd_nat_add_target and remove prototypes for
> fbsd_pid_to_exec_file and fbsd_find_memory_regions.
> * amd64fbsd-nat.c (_initialize_amd64fbsd_nat): Use fbsd_nat_add_target.
> * i386fbsd-nat.c (_initialize_i386fbsd_nat): Likewise.
> * ppcfbsd-nat.c (_initialize_ppcfbsd_nat): Likewise.
> * sparc64fbsd-nat.c (_initialize_sparc64fbsd_nat): Likewise.
OOC, any reason you didn't instead do it like:
struct target_ops *
fbsd_nat_target (void)
{
struct target_ops *t = inf_ptrace_target ();
t->to_pid_to_exec_file = fbsd_pid_to_exec_file;
t->to_find_memory_regions = fbsd_find_memory_regions;
return t;
}
and then use fbsd_nat_target instead of inf_ptrace_target
directly?
This maps a little better to a C++ world.
linux-nat.c does it the way you did as it keeps a separate
linux_ops target instance around.
Thanks,
Pedro Alves