This is the mail archive of the elfutils-devel@sourceware.org mailing list for the elfutils 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: [PATCH 5/5] Add dwarf_endianity_string and dwarf_decimal_sign_string.


On Thu, 2012-08-16 at 14:41 -0700, Roland McGrath wrote:
> I'd rather see the readelf bits of this as part of 1/5 and the
> librarification as part of 2/5.

Lets just accept that this is the order in which I wrote the patch
series and tested it. It has been rebased, rewritten and reordered
several times now in the last month. And I got bitten already once by
committing something out of order and introducing a bug.

All I wanted to start with was some dwarf string functions to help write
better error messages for systemtap when we encounter unhandled DWARF
constants in the translator. The reviews have been very helpful and we
even cleaned up some code, fixed some bugs and added some functionality.
But lets now just get this patch series done instead of trying to create
the perfect patch commit reordering.

Thanks,

Mark

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