This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: RFA rs6000-nat.c rc used out of scope
- To: gdb-patches at sources dot redhat dot com
- Subject: Re: RFA rs6000-nat.c rc used out of scope
- From: gkm at cygnus dot com (glen mccready)
- Date: 25 Apr 2000 18:06:50 GMT
- Newsgroups: cygnus.patches.gdb
- Organization: Cygnus Solutions
- References: <200004250941.LAA01745@mail.macqel.be> <1000425163752.ZM9160@ocotillo.lan>
In article <1000425163752.ZM9160@ocotillo.lan>,
Kevin Buettner <kevinb@cygnus.com> wrote:
>On Apr 25, 11:41am, Philippe De Muyter wrote:
>
>> Building gdb on aix4.1.5 fails with :
>> gdb/rs6000-nat.c: In function `xcoff_relocate_symtab':
>> gdb/rs6000-nat.c:675: `rc' undeclared (first use this function)
>> gdb/rs6000-nat.c:675: (Each undeclared identifier is reported only once
>> gdb/rs6000-nat.c:675: for each function it appears in.)
>>
>> Here is a fix (using diff -b, to hide indent fixes)
>>
>> OK to commit ? (and do we really need an approval for such sort of things,
>> or would a FYI be enough ?)
>
>Your patch is fine, and I would normally approve it. However, I see
>that the last person in this function (I won't name names) did not
>apply the patch that he said he would. (I will bug him about it
>privately.) Anyway, his proposed patch (which somehow didn't get
>applied) would fix the problem with rc being out of scope as well
>eliminate the use of alloca() from this function.
That'd be me. I've committed the patch that was meant to appear there.
Sorry about that folks; during some other confusion I managed to confuse
myself -- or something.
glen