This is the mail archive of the
mailing list for the GDB project.
Re: [patch] Change cplus_specific to an alocated struct
- From: Tom Tromey <tromey at redhat dot com>
- To: sami wagiaalla <swagiaal at redhat dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Tue, 15 Jun 2010 16:56:43 -0600
- Subject: Re: [patch] Change cplus_specific to an alocated struct
- References: <4BFD4230.email@example.com> <firstname.lastname@example.org> <4C167FCA.email@example.com>
- Reply-to: Tom Tromey <tromey at redhat dot com>
>>>>> "Sami" == sami wagiaalla <firstname.lastname@example.org> 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.