This is the mail archive of the
mailing list for the elfutils project.
Re: [PATCH 5/5] Add dwarf_endianity_string and dwarf_decimal_sign_string.
- From: Mark Wielaard <mjw at redhat dot com>
- To: elfutils-devel at lists dot fedorahosted dot org
- Date: Fri, 17 Aug 2012 01:16:00 +0200
- Subject: 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.