This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [RFA/patch] handle_inferior_event() extract some code into a separate function
- From: Mark Kettenis <kettenis at chello dot nl>
- To: brobecker at gnat dot com
- Cc: ac131313 at redhat dot com, gdb-patches at sources dot redhat dot com
- Date: Sat, 3 Jan 2004 13:51:10 +0100 (CET)
- Subject: Re: [RFA/patch] handle_inferior_event() extract some code into a separate function
- References: <20031219144323.GL826@gnat.com> <3FF59C76.3000902@gnu.org> <20040103115203.GU820@gnat.com>
Date: Sat, 3 Jan 2004 12:52:03 +0100
From: Joel Brobecker <brobecker@gnat.com>
> >This patch implements the first part, which is to extract the code
> >inside the we-stepped-into-a-function if block, and move it to its
> >own function. It's fairly mechanical.
> >
> >The next step will be to change the if condition itself to separate
> >out the targets where the new frame code is used from the ones where
> >the old one is still in use. I will send a patch as soon as this one
> >is reviewed.
>
> M'kay.
Thanks. It's in. Now to the next step...
I get:
gcc -c -g -O2 ... -Werror /home/kettenis/sandbox/virgin-gdb/src/gdb/infrun.c
cc1: warnings being treated as errors
/home/kettenis/sandbox/virgin-gdb/src/gdb/infrun.c: In function `handle_inferior_event':
/home/kettenis/sandbox/virgin-gdb/src/gdb/infrun.c:1337: warning: `real_stop_pc' might be used uninitialized in this function
And as far as I can see, this is a genuine problem :-(. Can you
please back that patch out?
Mark