[PATCH v2] readelf: use hex prefix for program header alignment

Hans-Peter Nilsson hp@bitrange.com
Fri Dec 9 02:14:00 GMT 2016

Previous message (by thread): [PATCH v2] readelf: use hex prefix for program header alignment
Next message (by thread): [PATCH] Fix crash when disassembling invalid range on powerpc vle
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

On Fri, 9 Dec 2016, Alan Modra wrote:
> On Thu, Dec 08, 2016 at 12:09:21PM -0500, Hans-Peter Nilsson wrote:
> > On Thu, 8 Dec 2016, Ã‰tienne Buira wrote:
> > > ---
> > >  binutils/readelf.c | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > Missing updates to the test-suite.  I'm guessing that's going to
> > be a big patch.  Also missing a changelog and it'd be nice with
> > some background why you want to change this now.
>
> Like you, I'd normally be concerned about readelf changes, and I was
> until I saw no test failures and looked carefully at the patch.

Wonderful  ...except it means we're missing coverage for
the case below. 1/2 :)

> The output being changed is for readelf -l without --wide on 64-bit
> ELF files.  It seems we invariably run readelf -l with --wide on such
> files.  Thus no testsuite impact.
>
> I think the patch is good for consistency with other readelf -l
> output.

Certainly, I was just curious whether there was a specific
reason that the OP wanted this change now.

brgds, H-P

Previous message (by thread): [PATCH v2] readelf: use hex prefix for program header alignment
Next message (by thread): [PATCH] Fix crash when disassembling invalid range on powerpc vle
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the Binutils mailing list