From 5e9b84f7a2e13af9ed00e262ad844e192b88ea8c Mon Sep 17 00:00:00 2001 From: Nick Alcock Date: Wed, 29 Jul 2020 13:50:10 +0100 Subject: binutils, ld: dequote libctf error messages These are not identifiers and should not be quoted. (Also, quoting them just looks odd.) Adjust diagnostics tests accordingly. binutils/ChangeLog 2020-08-27 Nick Alcock * objdump.c (dump_ctf_errs): Unquote CTF error messages. * readelf.c (dump_ctf_errs): Likewise. ld/ChangeLog 2020-08-27 Nick Alcock * ldlang.c (dump_ctf_errs): Unquote CTF error messages. (ldlang_open_ctf): Likewise. (lang_merge_ctf): Likewise. (lang_write_ctf): Likewise. * testsuite/ld-ctf/diag-ctf-version-f.d: Adjust. * testsuite/ld-ctf/diag-cttname-invalid.d: Adjust. * testsuite/ld-ctf/diag-decompression-failure.d: Adjust. * testsuite/ld-ctf/diag-parname.d: Adjust. * testsuite/ld-ctf/diag-unsupported-flag.d: Adjust. * testsuite/ld-ctf/diag-wrong-magic-number-mixed.d: Adjust. * testsuite/ld-ctf/diag-wrong-magic-number.d: Adjust. --- ld/ChangeLog | 14 ++++++++++++++ ld/ldlang.c | 8 ++++---- ld/testsuite/ld-ctf/diag-ctf-version-f.d | 2 +- ld/testsuite/ld-ctf/diag-cttname-invalid.d | 2 +- ld/testsuite/ld-ctf/diag-decompression-failure.d | 2 +- ld/testsuite/ld-ctf/diag-parname.d | 2 +- ld/testsuite/ld-ctf/diag-unsupported-flag.d | 2 +- ld/testsuite/ld-ctf/diag-wrong-magic-number-mixed.d | 2 +- ld/testsuite/ld-ctf/diag-wrong-magic-number.d | 2 +- 9 files changed, 25 insertions(+), 11 deletions(-) (limited to 'ld') diff --git a/ld/ChangeLog b/ld/ChangeLog index 4589432..4fd5546 100644 --- a/ld/ChangeLog +++ b/ld/ChangeLog @@ -1,5 +1,19 @@ 2020-08-27 Nick Alcock + * ldlang.c (dump_ctf_errs): Unquote CTF error messages. + (ldlang_open_ctf): Likewise. + (lang_merge_ctf): Likewise. + (lang_write_ctf): Likewise. + * testsuite/ld-ctf/diag-ctf-version-f.d: Adjust. + * testsuite/ld-ctf/diag-cttname-invalid.d: Adjust. + * testsuite/ld-ctf/diag-decompression-failure.d: Adjust. + * testsuite/ld-ctf/diag-parname.d: Adjust. + * testsuite/ld-ctf/diag-unsupported-flag.d: Adjust. + * testsuite/ld-ctf/diag-wrong-magic-number-mixed.d: Adjust. + * testsuite/ld-ctf/diag-wrong-magic-number.d: Adjust. + +2020-08-27 Nick Alcock + * ldlang.c (lang_ctf_errs_warnings): Support calls with NULL fp. Adjust for new err parameter to ctf_errwarning_next. Only check for assertion failures when fp is non-NULL. diff --git a/ld/ldlang.c b/ld/ldlang.c index dc6f1b4..4249b3a 100644 --- a/ld/ldlang.c +++ b/ld/ldlang.c @@ -3683,7 +3683,7 @@ lang_ctf_errs_warnings (ctf_file_t *fp) while ((text = ctf_errwarning_next (fp, &i, &is_warning, &err)) != NULL) { - einfo (_("%s: `%s'\n"), is_warning ? _("CTF warning"): _("CTF error"), + einfo (_("%s: %s\n"), is_warning ? _("CTF warning"): _("CTF error"), text); free (text); } @@ -3724,7 +3724,7 @@ ldlang_open_ctf (void) { lang_ctf_errs_warnings (NULL); einfo (_("%P: warning: CTF section in %pB not loaded; " - "its types will be discarded: `%s'\n"), file->the_bfd, + "its types will be discarded: %s\n"), file->the_bfd, ctf_errmsg (err)); } continue; @@ -3814,7 +3814,7 @@ lang_merge_ctf (void) { lang_ctf_errs_warnings (ctf_output); einfo (_("%P: warning: CTF linking failed; " - "output will have no CTF section: `%s'\n"), + "output will have no CTF section: %s\n"), ctf_errmsg (ctf_errno (ctf_output))); if (output_sect) { @@ -3874,7 +3874,7 @@ lang_write_ctf (int late) if (!output_sect->contents) { einfo (_("%P: warning: CTF section emission failed; " - "output will have no CTF section: `%s'\n"), + "output will have no CTF section: %s\n"), ctf_errmsg (ctf_errno (ctf_output))); output_sect->size = 0; output_sect->flags |= SEC_EXCLUDE; diff --git a/ld/testsuite/ld-ctf/diag-ctf-version-f.d b/ld/testsuite/ld-ctf/diag-ctf-version-f.d index 860aae9..811ed5b 100644 --- a/ld/testsuite/ld-ctf/diag-ctf-version-f.d +++ b/ld/testsuite/ld-ctf/diag-ctf-version-f.d @@ -2,4 +2,4 @@ #source: diag-ctf-version-f.s #ld: -shared #name: Diagnostics - Unsupported CTF version -#warning: CTF section .* not loaded; its types will be discarded: .CTF dict version is too new for libctf. +#warning: CTF section .* not loaded; its types will be discarded: CTF dict version is too new for libctf diff --git a/ld/testsuite/ld-ctf/diag-cttname-invalid.d b/ld/testsuite/ld-ctf/diag-cttname-invalid.d index de4aedc..8755c7e 100644 --- a/ld/testsuite/ld-ctf/diag-cttname-invalid.d +++ b/ld/testsuite/ld-ctf/diag-cttname-invalid.d @@ -2,4 +2,4 @@ #source: diag-cttname-invalid.s #ld: -shared #name: Diagnostics - Invalid type name. -#warning: CTF section in .*not loaded; its types will be discarded: .String name offset is corrupt. +#warning: CTF section in .*not loaded; its types will be discarded: String name offset is corrupt diff --git a/ld/testsuite/ld-ctf/diag-decompression-failure.d b/ld/testsuite/ld-ctf/diag-decompression-failure.d index c4567e1..1deeea1 100644 --- a/ld/testsuite/ld-ctf/diag-decompression-failure.d +++ b/ld/testsuite/ld-ctf/diag-decompression-failure.d @@ -2,4 +2,4 @@ #source: diag-decompression-failure.s #ld: -shared #name: Diagnostics - Decompression failure -#warning: CTF section.* not loaded; its types will be discarded: .Failed to decompress CTF data. +#warning: CTF section.* not loaded; its types will be discarded: Failed to decompress CTF data diff --git a/ld/testsuite/ld-ctf/diag-parname.d b/ld/testsuite/ld-ctf/diag-parname.d index d2ce9aa..f8fcd68 100644 --- a/ld/testsuite/ld-ctf/diag-parname.d +++ b/ld/testsuite/ld-ctf/diag-parname.d @@ -2,4 +2,4 @@ #source: diag-parname.s #ld: -shared --ctf-variables #name: Diagnostics - No parent dictionary -#warning: CTF linking failed; output will have no CTF section: .The parent CTF dictionary is unavailable. +#warning: CTF linking failed; output will have no CTF section: The parent CTF dictionary is unavailable diff --git a/ld/testsuite/ld-ctf/diag-unsupported-flag.d b/ld/testsuite/ld-ctf/diag-unsupported-flag.d index 8797817..6b7a359 100644 --- a/ld/testsuite/ld-ctf/diag-unsupported-flag.d +++ b/ld/testsuite/ld-ctf/diag-unsupported-flag.d @@ -2,4 +2,4 @@ #source: diag-unsupported-flag.s #ld: -shared #name: Diagnostics - Unsupported flag -#warning: CTF section.* not loaded; its types will be discarded: .CTF header contains flags unknown to libctf. +#warning: CTF section.* not loaded; its types will be discarded: CTF header contains flags unknown to libctf diff --git a/ld/testsuite/ld-ctf/diag-wrong-magic-number-mixed.d b/ld/testsuite/ld-ctf/diag-wrong-magic-number-mixed.d index 5415c13..1e33447 100644 --- a/ld/testsuite/ld-ctf/diag-wrong-magic-number-mixed.d +++ b/ld/testsuite/ld-ctf/diag-wrong-magic-number-mixed.d @@ -3,7 +3,7 @@ #source: B.c #ld: -shared --ctf-variables #name: Diagnostics - Wrong magic number mixed with valid CTF sections -#warning: CTF section in .* not loaded; its types will be discarded: .Buffer does not contain CTF data. +#warning: CTF section in .* not loaded; its types will be discarded: Buffer does not contain CTF data .*: +file format .* diff --git a/ld/testsuite/ld-ctf/diag-wrong-magic-number.d b/ld/testsuite/ld-ctf/diag-wrong-magic-number.d index 847bbf8..0c27b0c 100644 --- a/ld/testsuite/ld-ctf/diag-wrong-magic-number.d +++ b/ld/testsuite/ld-ctf/diag-wrong-magic-number.d @@ -2,4 +2,4 @@ #source: diag-wrong-magic-number.s #ld: -shared #name: Diagnostics - Wrong magic number -#warning: CTF section in .* not loaded; its types will be discarded: .Buffer does not contain CTF data. +#warning: CTF section in .* not loaded; its types will be discarded: Buffer does not contain CTF data -- cgit v1.1