This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [patch/rfc] FRAME_FP() -> get_frame_base()
- From: muller at ics dot u-strasbg dot fr
- To: Andrew Cagney <ac131313 at redhat dot com>,gdb-patches at sources dot redhat dot com
- Date: Mon, 25 Nov 2002 00:06:52 +0100
- Subject: Re: [patch/rfc] FRAME_FP() -> get_frame_base()
- References: <3DDA9741.4050001@redhat.com>
At 13:46 24/11/2002 -0500, you wrote:
>> 2002-11-19 Andrew Cagney <ac131313@redhat.com>
>>
>> * frame.h (FRAME_FP): Delete macro.
>> (get_frame_base): New function declaration.
>> * frame.c (get_frame_base): New function.
>> (get_frame_id): Use ->frame.
>> (frame_find_by_id): Rewrite to use get_frame_id.
>> * blockframe.c: Use get_frame_base instead of FRAME_FP.
>> * cris-tdep.c, d10v-tdep.c, findvar.c, h8500-tdep.c: Ditto.
>> * hppa-tdep.c, i386-tdep.c, infcmd.c, infrun.c: Ditto.
>> * m68hc11-tdep.c, m68k-tdep.c, mcore-tdep.c, mips-tdep.c: Ditto.
>> * mn10200-tdep.c, mn10300-tdep.c, rs6000-tdep.c: Ditto.
>> * sh-tdep.c, sparc-tdep.c, stack.c, tracepoint.c: Ditto.
>> * v850-tdep.c, valops.c, z8k-tdep.c: Ditto.
>>
>I've checked this in.
Andrew, you forgot varobj.c file
I committed this as obvious fix.
2002-11-24 Pierre Muller <muller@ics.u-strasbg.fr>
* varobj.c (find_frame_addr_in_frame_chain):
Use get_frame_base instead of FRAME_FP,
obvious fix.
$ cvs diff -u -p varobj.c
Index: varobj.c
===================================================================
RCS file: /cvs/src/src/gdb/varobj.c,v
retrieving revision 1.35
diff -u -p -r1.35 varobj.c
--- varobj.c 24 Nov 2002 14:16:38 -0000 1.35
+++ varobj.c 24 Nov 2002 23:02:22 -0000
@@ -412,7 +412,7 @@ find_frame_addr_in_frame_chain (CORE_ADD
frame = get_prev_frame (frame);
if (frame == NULL)
return NULL;
- if (FRAME_FP (frame) == frame_addr)
+ if (get_frame_base (frame) == frame_addr)
return frame;
}
}