This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [RFA] Introduce notion of "search name"
I.e., through out GDB there is the assumption that symtab internally
uses simple strings.
We shouldn't. Recalling that one of the underlying problems here was
the need to construct search names on the fly using more complext debug
info, we should instead have interfaces such as:
This is a worthy goal, but I see it as a goal separate from what the
patch is trying to achieve.
This is in part true. The mistake is to seperate cleanups (or
refactorings) from enhancements. Enhancement and fixes needs to go
hand-in-hand with a corresponding cleanups. If we don't do this, we
quickly find that those long over-due changes never occure.
> From what Paul is saying, the change you
> are objecting to introduces a check that is correct within the current
> implementation, and makes the kind of assumption that's already used
> everywhere. Why can't we use the current framework as is until the
> cleanup is done (BTW: who's going to do it, and when?). Admitedly, we
> have to document a temporary assumption, but I don't think this will
> affect the symbol interface cleanup you're suggesting all that much.
If this, as an implementation detail, were to be strictly hidden behind
the symtab interface, it wouldn't be a problem. Unfortunatly, as I
illustrated, the SYMBOL_*_NAME macros permieate GDB, so hand in hand
with this change, should be changes to screw down that aspect of the
symtab interface, and then have Ada use that.
enjoy,
Andew