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: [RFA] Rename dwarf2read.c mapped_index.index_table, et.al.


>>>>> "Doug" == Doug Evans <dje@google.com> writes:

Doug> There are two uses of member "index_table", in structs
Doug> dwarf2_per_objfile and mapped_index.  The use in mapped_index is
Doug> confusing (to me anyway).  I'm always asking: "index of what?"

Doug> I'm happy to use another name besides symbol_table.
Doug> symbol_hash_table? symbol_index_table?
Doug> But "index_table" doesn't adequately convey what the contents are.

Doug> Ok to check in?

IMO you don't have to ask.
But, FWIW, the renamings are fine by me.  Thank you for doing this.

Tom


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