[PATCH 1/3] libdw: unify die->abbrev lookups

Mark Wielaard mjw at redhat.com
Thu Dec 11 11:25:10 UTC 2014


On Wed, 2014-12-10 at 18:48 -0800, Josh Stone wrote:
> Add a new internal function, __libdw_dieabbrev, which deals with checking
> a die for an abbrev, and setting it as needed.
>
> +2014-12-10  Josh Stone  <jistone at redhat.com>
> +
> +	* libdwP.h (__libdw_dieabbrev): New die->abbrev lookup function.
> +	* dwarf_child.c (__libdw_find_attr, dwarf_child): Use it.
> +	* dwarf_getattrs.c (dwarf_getattrs): Likewise.
> +	* dwarf_haschildren.c (dwarf_haschildren): Likewise.
> +	* dwarf_tag.c (dwarf_tag): Likewise.
>
> +/* Get abbreviation of given DIE.  */
> +static inline Dwarf_Abbrev *
> +__nonnull_attribute__ (1)
> +__libdw_dieabbrev (Dwarf_Die *die, const unsigned char **readp)

Looks good. But please add a doc comment for the READP argument before
pushing this commit.

Thanks,

Mark


More information about the elfutils-devel mailing list