diff options
author | Nick Alcock <nick.alcock@oracle.com> | 2020-07-29 13:50:10 +0100 |
---|---|---|
committer | Nick Alcock <nick.alcock@oracle.com> | 2020-08-27 13:16:39 +0100 |
commit | 5e9b84f7a2e13af9ed00e262ad844e192b88ea8c (patch) | |
tree | 3d66a4b83d070f5be4c923871a9dda4fe3cebff6 | |
parent | 926c9e7665707d2f2eeb705b410e485f6930ab38 (diff) | |
download | gdb-5e9b84f7a2e13af9ed00e262ad844e192b88ea8c.zip gdb-5e9b84f7a2e13af9ed00e262ad844e192b88ea8c.tar.gz gdb-5e9b84f7a2e13af9ed00e262ad844e192b88ea8c.tar.bz2 |
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 <nick.alcock@oracle.com>
* objdump.c (dump_ctf_errs): Unquote CTF error messages.
* readelf.c (dump_ctf_errs): Likewise.
ld/ChangeLog
2020-08-27 Nick Alcock <nick.alcock@oracle.com>
* 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.
-rw-r--r-- | binutils/ChangeLog | 5 | ||||
-rw-r--r-- | binutils/objdump.c | 2 | ||||
-rw-r--r-- | binutils/readelf.c | 2 | ||||
-rw-r--r-- | ld/ChangeLog | 14 | ||||
-rw-r--r-- | ld/ldlang.c | 8 | ||||
-rw-r--r-- | ld/testsuite/ld-ctf/diag-ctf-version-f.d | 2 | ||||
-rw-r--r-- | ld/testsuite/ld-ctf/diag-cttname-invalid.d | 2 | ||||
-rw-r--r-- | ld/testsuite/ld-ctf/diag-decompression-failure.d | 2 | ||||
-rw-r--r-- | ld/testsuite/ld-ctf/diag-parname.d | 2 | ||||
-rw-r--r-- | ld/testsuite/ld-ctf/diag-unsupported-flag.d | 2 | ||||
-rw-r--r-- | ld/testsuite/ld-ctf/diag-wrong-magic-number-mixed.d | 2 | ||||
-rw-r--r-- | ld/testsuite/ld-ctf/diag-wrong-magic-number.d | 2 |
12 files changed, 32 insertions, 13 deletions
diff --git a/binutils/ChangeLog b/binutils/ChangeLog index 05423bd..44c2898 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,5 +1,10 @@ 2020-08-27 Nick Alcock <nick.alcock@oracle.com> + * objdump.c (dump_ctf_errs): Unquote CTF error messages. + * readelf.c (dump_ctf_errs): Likewise. + +2020-08-27 Nick Alcock <nick.alcock@oracle.com> + * objdump.c (dump_ctf_archive_member): Move error- reporting... (dump_ctf_errs): ... into this separate function. diff --git a/binutils/objdump.c b/binutils/objdump.c index c184896..c7ad51e 100644 --- a/binutils/objdump.c +++ b/binutils/objdump.c @@ -4083,7 +4083,7 @@ dump_ctf_errs (ctf_file_t *fp) /* Dump accumulated errors and warnings. */ while ((errtext = ctf_errwarning_next (fp, &it, &is_warning, &err)) != NULL) { - non_fatal (_("%s: `%s'"), is_warning ? _("warning"): _("error"), + non_fatal (_("%s: %s"), is_warning ? _("warning"): _("error"), errtext); free (errtext); } diff --git a/binutils/readelf.c b/binutils/readelf.c index c5796e8..d4dcab8 100644 --- a/binutils/readelf.c +++ b/binutils/readelf.c @@ -14249,7 +14249,7 @@ dump_ctf_errs (ctf_file_t *fp) /* Dump accumulated errors and warnings. */ while ((errtext = ctf_errwarning_next (fp, &it, &is_warning, &err)) != NULL) { - error (_("%s: `%s'"), is_warning ? _("warning"): _("error"), + error (_("%s: %s"), is_warning ? _("warning"): _("error"), errtext); free (errtext); } 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 <nick.alcock@oracle.com> + * 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 <nick.alcock@oracle.com> + * 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 |