aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNick Alcock <nick.alcock@oracle.com>2020-07-29 13:50:10 +0100
committerNick Alcock <nick.alcock@oracle.com>2020-08-27 13:16:39 +0100
commit5e9b84f7a2e13af9ed00e262ad844e192b88ea8c (patch)
tree3d66a4b83d070f5be4c923871a9dda4fe3cebff6
parent926c9e7665707d2f2eeb705b410e485f6930ab38 (diff)
downloadgdb-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/ChangeLog5
-rw-r--r--binutils/objdump.c2
-rw-r--r--binutils/readelf.c2
-rw-r--r--ld/ChangeLog14
-rw-r--r--ld/ldlang.c8
-rw-r--r--ld/testsuite/ld-ctf/diag-ctf-version-f.d2
-rw-r--r--ld/testsuite/ld-ctf/diag-cttname-invalid.d2
-rw-r--r--ld/testsuite/ld-ctf/diag-decompression-failure.d2
-rw-r--r--ld/testsuite/ld-ctf/diag-parname.d2
-rw-r--r--ld/testsuite/ld-ctf/diag-unsupported-flag.d2
-rw-r--r--ld/testsuite/ld-ctf/diag-wrong-magic-number-mixed.d2
-rw-r--r--ld/testsuite/ld-ctf/diag-wrong-magic-number.d2
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