Re: [RFA/patch] handle_inferior_event() extract some code into a separate function

Mark Kettenis
2004-01-03T12:51:23Z

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