Commit: Changing readelf's lint mode
Nick Clifton
nickc@redhat.com
Tue May 5 15:19:19 GMT 2020
Hi Guys,
I have had a change of heart about readelf's lint mode. I now think
that it would be better to leave the existing warning messages alone
and only make new warnings test for lint-mode being enabled, and then
only if they are of file-quality type rather than a file-problem type.
So I am checking the attached patch to revert my changes to the warn()
function, restoring it to its old behaviour. In the course of doing
this I also came across a bug in line number table decoder which was
not coping with an empty DWARF-5 format line number table. So I have
fixed this as well.
Cheers
Nick
binutils/ChangeLog
2020-05-05 Nick Clifton <nickc@redhat.com>
* dwarf.c (do_checks): New global variable.
(display_formatted_table): Warn about an unexpected number of
columns in the table, if checks are enabled. Do not complain
about the lack of data following the number of entries in the
table if the table is empty.
(display_debug_lines_decoded): Only warn about an unexpected
number of columns in a table if checks are enabled.
* dwarf.h (do_checks): Add a prototype.
* elfcomm.c (error): Remove weak attribute.
(warn): Likewise.
* readelf.c (do_checks): Delete.
(warn): Delete.
(process_section_headers): Only warn about empty sections if
checks are enabled.
gas/ChangeLog
2020-05-05 Nick Clifton <nickc@redhat.com>
* dwarf2dbg.c (out_dir_and_file_list): Add comments describing the
construction of a DWARF-5 directory name table.
* testsuite/gas/elf/pr25917.d: Update expected output.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: readelf-lint.patch
Type: text/x-patch
Size: 6949 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/binutils/attachments/20200505/6bd62555/attachment.bin>
More information about the Binutils
mailing list