This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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. 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.

-- 
Joel


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]