This is the mail archive of the binutils@sourceware.org mailing list for the binutils 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: [gold patch] Incremental 13/18: Dynamic relocations


Cary Coutant <ccoutant@google.com> writes:

>> Sized_relobj_base is turning into a poor choice of names. ÂIt looks the
>> heirarchy is now
>>
>> Object
>> Relobj
>> Sized_relobj_base
>> Sized_relobj / Sized_incr_relobj
>>
>> I don't mind having Sized_relobj_base when just a couple of things refer
>> to it, but it looks wrong when lots of functions use it. ÂThe name of a
>> class should indicate what sort of thing it is, rather than its position
>> in the class heirarchy. ÂWe should consider renaming, e.g.,
>> Sized_relobj_base => Sized_relobj, Sized_relobj => Sized_relobj_file,
>> Sized_incr_relobj => Sized_relobj_incr. ÂI'm open to other suggestions.
>
> Yeah, I started out with something like this, but (a) I couldn't come
> up with a name I liked for Sized_relobj_file, and (b) I was trying to
> avoid wholesale changes to so many other files that were otherwise
> unaffected.
>
> In the ChangeLog, should I enumerate all the files touched, or is it
> better to just say "Rename x to y; change all references" in object.h?

Just saying "change all references" is fine.

Thanks.

Ian


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