This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] Add test for following fork on position-independent executables
- From: Pedro Alves <palves at redhat dot com>
- To: Simon Marchi <simon dot marchi at ericsson dot com>, gdb-patches at sourceware dot org
- Date: Wed, 11 Apr 2018 19:24:03 +0100
- Subject: Re: [PATCH] Add test for following fork on position-independent executables
- References: <1523460523-19370-1-git-send-email-simon.marchi@ericsson.com>
On 04/11/2018 04:28 PM, Simon Marchi wrote:
> +static void
> +break_here ()
This is C, so "(void)" is better here.
> +{
> + int i;
> + for (i = 0; i < 30; i++)
> + sleep (1);
> +}
> +
> +int
> +main ()
> +{
> + fork ();
> + break_here();
Missing space before parens.
> +if [use_gdb_stub] {
> + return
> +}
> +
I guess this is why you were asking about following forks and
"target remote". AFAICT, this should just work in that case
too. Did you find out it doesn't?
Otherwise looks fine. Thanks for adding this.
Thanks,
Pedro Alves