This is the mail archive of the
gdb-patches@sourceware.cygnus.com
mailing list for the GDB project.
Re: [PATCH] revert win32-nat.c stack handling to working version
- To: Chris Faylor <cgf at cygnus dot com>
- Subject: Re: [PATCH] revert win32-nat.c stack handling to working version
- From: Stan Shebs <shebs at apple dot com>
- Date: Mon, 27 Mar 2000 11:53:22 -0800
- Cc: gdb-patches at sourceware dot cygnus dot com
- Organization: Core Tools, Apple Computer
- References: <20000324213253.A23842@cygnus.com>
- Reply-To: shebs at apple dot com
Chris Faylor wrote:
>
> I added some "functionality" to win32-nat.c a while ago while under the
> mistaken impression that using the Microsoft stack-walking APIs would
> somehow provide more functionality than what was previously available in
> gdb. Instead, I just broke things so that stack walking stopped after a
> few frames.
>
> The enclosed patch reverts gdb to its previous behavior and also adds an
> additional unrelated trap for illegal instructions, which was missing
> before (I know this should be a separate patch but...).
Seems like it would be helpful to add a comment in the sources somewhere
so that some future hacker won't go down the same fruitless path. When
I first started hacking on GCC, I was impressed by the number of #if 0
chunks that were preceded by "you would think this would be the right
thing to do, but here's why it loses", along with the explanation. You
wouldn't want to do this all the time, but this case seems worth
documenting.
Stan