[PATCH, gdbserver] Further cleanup of FDPIC/DSBT divergences

Mike Frysinger vapier@gentoo.org
Mon Aug 26 05:31:00 GMT 2013


On Friday 09 August 2013 13:20:24 Luis Machado wrote:
> --- a/gdb/gdbserver/linux-low.c
> +++ b/gdb/gdbserver/linux-low.c
> 
>  }
> -#else
> -# define linux_read_loadmap NULL
> -#endif /* defined PT_GETDSBT || defined PTRACE_GETFDPIC */
> +#endif /* defined PTRACE_GETFDPIC */
> 
> @@ -6036,7 +6013,11 @@ static struct target_ops linux_target_ops = {
>    linux_common_core_of_thread,
> +#if defined PTRACE_GETFDPIC
>    linux_read_loadmap,
> +#else
> +  NULL,
> +#endif /* defined PTRACE_GETFDPIC */
>    linux_process_qsupported,
>    linux_supports_tracepoints,

personally, i liked the style before -- keep the logic where the func is 
defined rather than in the initialization of the target ops.

otherwise, the changes to the linux-low.c file LGTM
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <http://sourceware.org/pipermail/gdb-patches/attachments/20130826/7eedae83/attachment.sig>


More information about the Gdb-patches mailing list