Re: [PATCH] readelf: Don't shadow index function from string.h

2018-07-05 Thread Mark Wielaard
On Wed, Jul 04, 2018 at 04:13:26PM +0200, Mark Wielaard wrote: > On some ancient GCC versions (4.4.7 at least) -Wshadow warns about local > variables "shadowing" global function definitions. > > readelf.c: In function ‘print_debug_addr_section’: > readelf.c:5265: error: declaration of ‘index’

Re: [PATCH 2/2] Consolidate error.h inclusion in system.h

2018-07-05 Thread Mark Wielaard
On Wed, Jul 04, 2018 at 12:11:08PM +0100, Ross Burton wrote: > error.h isn't standard and so isn't part of the musl C library. To easy > future > porting, consolidate the inclusion of error.h into system.h. > > https://sourceware.org/bugzilla/show_bug.cgi?id=21008 Thanks, added ChangeLog entrie

Re: [PATCH 1/2] Require gawk in maintainer mode

2018-07-05 Thread Mark Wielaard
On Wed, Jul 04, 2018 at 12:11:07PM +0100, Ross Burton wrote: > gawk is required to build known_dwarf.h, so check for it in configure.ac. Thanks. Added a ChangeLog entry and pushed to master.

[PATCH] tests: Handle compressed sections in next_cfi testcase.

2018-07-05 Thread Mark Wielaard
Some toolchains use compressed ELF sections by default. This would make run-next-cfi-self.sh fail because it would try to decode the compressed data. Fix by decompressing the section first. https://sourceware.org/bugzilla/show_bug.cgi?id=23370 Signed-off-by: Mark Wielaard --- tests/ChangeLog |

[Bug general/23370] run-next-cfi-self.sh doesn't handle compressed ELF sections

2018-07-05 Thread mark at klomp dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=23370 Mark Wielaard changed: What|Removed |Added CC||mark at klomp dot org Summ