[PATCH 3/4] Add an debugscn_p() shdr param to debugscn_p() and update all callers.
Roland McGrath
roland at hack.frob.com
Thu Aug 23 23:49:23 UTC 2012
I'd put the SHDR parameter first. That's more consistent with how we've
chosen the signatures of other hooks.
I'm inclined toward the suggestion I just made in another message, to make
a NULL value for NAME valid by updating the existing implementations.
In that case, the strip.c change:
- if (! tname || ! ebl_debugscn_p (ebl, tname))
+ if (! tname || ! ebl_debugscn_p (ebl, tname, tshdr))
should be:
- if (! tname || ! ebl_debugscn_p (ebl, tname))
+ if (! ebl_debugscn_p (ebl, tshdr, tname))
so it's up to the backend hook how to respond when the name isn't available.
(Not that this scenario really matters, as it indicates a bogus ELF file.)
There are some style issues with your ChangeLog entries and some lines
you've made too long (all lines should be < 80 columns). Mark can help
you get all those nits right. (I'll be on vacation after today.)
Thanks,
Roland
More information about the elfutils-devel
mailing list