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] Remove redundant symbol naming from new_symbol


On 05/27/2010 07:38 AM, sami wagiaalla wrote:
This patch removes the redundant calls.

FWIW, I think this patch should be approved; the additional calls to dwarf2_full_name do appear to be redundant, since SYMBOL_SET_NAMES is called unconditionally for every symbol with DW_AT_name.


[The only question is whether dwarf2_full_name and dwarf2_physname could return different "names" for these DIEs, and since the DIEs do not represent a class method, the "names" will be identical.]

Keith


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