This is the mail archive of the gdb-patches@sourceware.org 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: [patch] Change cplus_specific to an alocated struct


>>>>> "Sami" == sami wagiaalla <swagiaal@redhat.com> writes:

Tom> It seems to me that soon we're going to want to add a bunch of
Tom> C++-specific fields, and we don't want to unnecessarily penalize the
Tom> other languages with our baggage.

Sami> I wasn't really planing one :D, but what do you think of this:

Sami> We leave the current struct as is and rename cplus_specific to
Sami> mangled_lang_specific (or just mangled_lang). And the the union we add
Sami> a cplus_specific that managed as things are in this patch, and is
Sami> actually cplus_specific ?

That sounds ok to me.

Sami> I think that is just a wrong use of lazy here. I meant to say
Sami> initialize it /if/ it is going to be used rather than when...

Sami> symbol_init_cplus_specific is called from symbol_set_names where the
Sami> bcache is updated.

Aha, thanks.

Tom


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