On Monday, November 18, 2002, at 12:40 PM, Jim Blandy wrote:
Okay, that looks like a good change to me --- please commit it.
But rather than making pop_context a hairy macro, could you make it a
function in buildsym.c, right after push_context?
Gladly. Committed the following:
2002-11-18 Klee Dienes <kdienes@apple.com>
* buildsym.h (pop_context): Convert to function, defined in
buildsym.c.
* buildsym.c: Include gdb_assert.h.
(pop_context): Implement as C function. Add check for stack
underflow.
* dbxread.c (process_one_symbol): Complain and stop processing
that symbol if we are already at the top of the context stack for
a function-end N_FUN (this would imply an umatched RBRAC). Ditto
when processing N_RBRAC.