From patchwork Mon Mar 20 14:56:38 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alexandra Petlanova Hajkova X-Patchwork-Id: 66641 Return-Path: X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id DD2F9385F012 for ; Mon, 20 Mar 2023 14:57:09 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org DD2F9385F012 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1679324229; bh=UH3Taz3s1xm6laWU/53x/haEutfksd58Mj6MSo6G4e4=; h=To:Subject:Date:List-Id:List-Unsubscribe:List-Archive:List-Post: List-Help:List-Subscribe:From:Reply-To:From; b=D7y0CLAMxNaom72lzIMExsobNBaj5+0lKYibXZxB5DPpk/70R6FWtFSw+bhPKsuqn syHDyMBnvIjfltbJ4T08xrMweApmFC+yos7TWVnLgHqQlVLmC5oPlGlFdv/l32bxiy lDfVSeNJSk3K+uwPo6E8Y4lUVtgxh+xA85TOaGQo= X-Original-To: gdb-patches@sourceware.org Delivered-To: gdb-patches@sourceware.org Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by sourceware.org (Postfix) with ESMTPS id 0F3AF38515E4 for ; Mon, 20 Mar 2023 14:56:45 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 0F3AF38515E4 Received: from mimecast-mx02.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-400-U_umNgVRN6CpJlSLN5gUmw-1; Mon, 20 Mar 2023 10:56:42 -0400 X-MC-Unique: U_umNgVRN6CpJlSLN5gUmw-1 Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.rdu2.redhat.com [10.11.54.6]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 40EAE3C16E85 for ; Mon, 20 Mar 2023 14:56:42 +0000 (UTC) Received: from fedora.redhat.com (unknown [10.45.242.11]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 5729A2166B2B for ; Mon, 20 Mar 2023 14:56:41 +0000 (UTC) To: gdb-patches@sourceware.org Subject: [PATCH v2] Use styled_string when defering warnings when loading separate debug files Date: Mon, 20 Mar 2023 15:56:38 +0100 Message-Id: <20230320145638.1202335-1-ahajkova@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.1 on 10.11.54.6 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com X-Spam-Status: No, score=-10.6 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, KAM_SHORT, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H2, SPF_HELO_NONE, SPF_NONE, TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-Patchwork-Original-From: =?utf-8?q?Alexandra_H=C3=A1jkov=C3=A1_via_Gdb-patches?= From: Alexandra Petlanova Hajkova Reply-To: =?utf-8?q?Alexandra_H=C3=A1jkov=C3=A1?= Errors-To: gdb-patches-bounces+patchwork=sourceware.org@sourceware.org Sender: "Gdb-patches" This improves commit 6647f05df023b63bbe056e9167e9e234172fa2ca, the filenames used in the warnings are styled properly now. --- This version uses ui_file rather instead of vector of callback functions that emit warnings. Addresses various cosmetic issues. gdb/build-id.c | 16 ++++++++++------ gdb/build-id.h | 4 ++-- gdb/coffread.c | 4 ++-- gdb/elfread.c | 4 ++-- gdb/symfile.c | 26 ++++++++++++++++---------- gdb/symfile.h | 4 ++-- gdb/ui-file.h | 7 +++++++ 7 files changed, 41 insertions(+), 24 deletions(-) diff --git a/gdb/build-id.c b/gdb/build-id.c index 00250c20ae9..5d5114e2ace 100644 --- a/gdb/build-id.c +++ b/gdb/build-id.c @@ -18,6 +18,7 @@ along with this program. If not, see . */ #include "defs.h" +#include "cli/cli-style.h" #include "bfd.h" #include "gdb_bfd.h" #include "build-id.h" @@ -209,7 +210,7 @@ build_id_to_exec_bfd (size_t build_id_len, const bfd_byte *build_id) std::string find_separate_debug_file_by_buildid (struct objfile *objfile, - std::vector *warnings_vector) + std::vector *warnings_vector) { const struct bfd_build_id *build_id; @@ -228,12 +229,15 @@ find_separate_debug_file_by_buildid (struct objfile *objfile, && filename_cmp (bfd_get_filename (abfd.get ()), objfile_name (objfile)) == 0) { - std::string msg - = string_printf (_("\"%s\": separate debug info file has no " - "debug info"), bfd_get_filename (abfd.get ())); + string_file warnings (true); + warnings.printf ("\"%ps\":separate debug info file has no debug info", + styled_string (file_name_style.style (), + bfd_get_filename (abfd.get ()))); if (separate_debug_file_debug) - gdb_printf (gdb_stdlog, "%s", msg.c_str ()); - warnings_vector->emplace_back (std::move (msg)); + /* Avoid styling for the debug output. */ + gdb_printf (gdb_stdlog, _("%s: separate debug info file has no debug info"), + bfd_get_filename (abfd.get ())); + warnings_vector->emplace_back (std::move (warnings)); } else if (abfd != NULL) return std::string (bfd_get_filename (abfd.get ())); diff --git a/gdb/build-id.h b/gdb/build-id.h index 191720ddf28..0c2c4181b4e 100644 --- a/gdb/build-id.h +++ b/gdb/build-id.h @@ -52,13 +52,13 @@ extern gdb_bfd_ref_ptr build_id_to_exec_bfd (size_t build_id_len, separate debug file, otherwise, return an empty string. Any warnings that are generated by the lookup process should be added to - WARNINGS_VECTOR, one std::string per warning. If some other mechanism can + WARNINGS_VECTOR, one string_file per warning. If some other mechanism can be used to lookup the debug information then the warning will not be shown, however, if GDB fails to find suitable debug information using any approach, then any warnings will be printed. */ extern std::string find_separate_debug_file_by_buildid - (struct objfile *objfile, std::vector *warnings_vector); + (struct objfile *objfile, std::vector *warnings_vector); /* Return an hex-string representation of BUILD_ID. */ diff --git a/gdb/coffread.c b/gdb/coffread.c index e993b17db09..32705c215fd 100644 --- a/gdb/coffread.c +++ b/gdb/coffread.c @@ -729,7 +729,7 @@ coff_symfile_read (struct objfile *objfile, symfile_add_flags symfile_flags) /* Try to add separate debug file if no symbols table found. */ if (!objfile->has_partial_symbols ()) { - std::vector warnings_vector; + std::vector warnings_vector; std::string debugfile = find_separate_debug_file_by_buildid (objfile, &warnings_vector); @@ -747,7 +747,7 @@ coff_symfile_read (struct objfile *objfile, symfile_add_flags symfile_flags) /* If all the methods to collect the debuginfo failed, print any warnings that were collected. */ if (debugfile.empty () && !warnings_vector.empty ()) - for (const std::string &w : warnings_vector) + for (const string_file &w : warnings_vector) warning ("%s", w.c_str ()); } } diff --git a/gdb/elfread.c b/gdb/elfread.c index b414da9ed21..4bc89d09d65 100644 --- a/gdb/elfread.c +++ b/gdb/elfread.c @@ -1217,7 +1217,7 @@ elf_symfile_read_dwarf2 (struct objfile *objfile, && objfile->separate_debug_objfile == NULL && objfile->separate_debug_objfile_backlink == NULL) { - std::vector warnings_vector; + std::vector warnings_vector; std::string debugfile = find_separate_debug_file_by_buildid (objfile, &warnings_vector); @@ -1269,7 +1269,7 @@ elf_symfile_read_dwarf2 (struct objfile *objfile, /* If all the methods to collect the debuginfo failed, print the warnings, if there're any. */ if (debugfile.empty () && !has_dwarf2 && !warnings_vector.empty ()) - for (const std::string &w : warnings_vector) + for (const string_file &w : warnings_vector) warning ("%s", w.c_str ()); } diff --git a/gdb/symfile.c b/gdb/symfile.c index bb9981a4634..64d29ea71f9 100644 --- a/gdb/symfile.c +++ b/gdb/symfile.c @@ -1244,7 +1244,7 @@ bool separate_debug_file_debug = false; static int separate_debug_file_exists (const std::string &name, unsigned long crc, struct objfile *parent_objfile, - std::vector *warnings_vector) + std::vector *warnings_vector) { unsigned long file_crc; int file_crc_p; @@ -1337,13 +1337,19 @@ separate_debug_file_exists (const std::string &name, unsigned long crc, if (verified_as_different || parent_crc != file_crc) { - std::string msg - = string_printf (_("the debug information found in \"%s\"" - " does not match \"%s\" (CRC mismatch).\n"), - name.c_str (), objfile_name (parent_objfile)); + string_file warnings (true); + warnings.printf (_("the debug information found in \"%ps\"" + " does not match \"%ps\" (CRC mismatch)."), + styled_string (file_name_style.style (), + name.c_str ()), + styled_string (file_name_style.style (), + objfile_name (parent_objfile))); + /* Avoid styling for the debug output. */ if (separate_debug_file_debug) - gdb_printf (gdb_stdlog, "%s", msg.c_str ()); - warnings_vector->emplace_back (std::move (msg)); + gdb_printf (gdb_stdlog, _("the debug information found in %s" + " does not match %s (CRC mismatch).\n"), + name.c_str(), objfile_name (parent_objfile)); + warnings_vector->emplace_back (std::move (warnings)); } return 0; @@ -1379,14 +1385,14 @@ show_debug_file_directory (struct ui_file *file, int from_tty, string. Any warnings generated as part of the lookup process are added to - WARNINGS_VECTOR, one std::string per warning. */ + WARNINGS_VECTOR, one string_file per warning. */ static std::string find_separate_debug_file (const char *dir, const char *canon_dir, const char *debuglink, unsigned long crc32, struct objfile *objfile, - std::vector *warnings_vector) + std::vector *warnings_vector) { if (separate_debug_file_debug) gdb_printf (gdb_stdlog, @@ -1534,7 +1540,7 @@ terminate_after_last_dir_separator (char *path) std::string find_separate_debug_file_by_debuglink - (struct objfile *objfile, std::vector *warnings_vector) + (struct objfile *objfile, std::vector *warnings_vector) { unsigned long crc32; diff --git a/gdb/symfile.h b/gdb/symfile.h index b433e2be31a..041f6f74f9c 100644 --- a/gdb/symfile.h +++ b/gdb/symfile.h @@ -245,10 +245,10 @@ extern void symbol_file_add_separate (const gdb_bfd_ref_ptr &, const char *, Returns pathname, or an empty string. Any warnings generated as part of this lookup are added to - WARNINGS_VECTOR, one std::string per warning. */ + WARNINGS_VECTOR, one string_file per warning. */ extern std::string find_separate_debug_file_by_debuglink - (struct objfile *objfile, std::vector *warnings_vector); + (struct objfile *objfile, std::vector *warnings_vector); /* Build (allocate and populate) a section_addr_info struct from an existing section table. */ diff --git a/gdb/ui-file.h b/gdb/ui-file.h index de24620e247..4444c8adcdb 100644 --- a/gdb/ui-file.h +++ b/gdb/ui-file.h @@ -212,6 +212,13 @@ class string_file : public ui_file return *this; } + string_file(string_file &&other) + : m_string (std::move (other.c_str ())), + m_term_out (other.m_term_out) + { + + } + /* Provide a few convenience methods with the same API as the underlying std::string. */ const char *data () const { return m_string.data (); }