binutils/glibc .hashvals section ...
Nick Clifton
nickc@redhat.com
Fri Jan 27 17:44:00 GMT 2006
Hi Michael,
> Trivial patch follows; of course, since this requires glibc support one
> can't be overly optimistic wrt. inclusion ;-)
There is no reason why such a patch cannot be included in binutils.
People are free to modify glibc after all.
> but surely it's only polite to post here for discussion.
Well then here are some comments...
> + case DT_SUSE_HASHVALS: name = "SUSE_HASHVALS"; break;
I assume that this feature does not have to be specific to SUSE, so I
would suggest a more generic name, eg DT_GNU_HASHVALS.
> + if (info->hashvals)
> + {
> + s = bfd_make_section (abfd, ".suse.hashvals");
Similarly. In fact if the info->hashvals field was a "const char *"
then it could be the name of the section to create. (Not sure if this
is a good idea though). If the name of the section is going to be fixed
however then it ought to be specified as a #defined constant in a header
file somewhere. One that can be accessed by both binutils and glibc.
(Although you could quite reasonably point out that hard coded section
names are everywhere in the elflink.c file. To which I would say, well
they should *all* be replaced by #define'd constants...)
Other than that though the binutils part of the patch looks fine to me.
A few formatting tidy ups and replacements of fprintf with calls to
bfd_error_handler, but otherwise OK.
Cheers
Nick
More information about the Binutils
mailing list