aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKeith Seitz <keiths@redhat.com>2018-08-30 07:47:03 -0700
committerKeith Seitz <keiths@redhat.com>2018-08-30 07:47:03 -0700
commitfdad7678b74a170ead1ef967007be087435715f7 (patch)
treee7f0836e901518feb4d9fcad056b3a5be01838f8
parent4c971803c4fb225b029420f812229941beabd42e (diff)
downloadgdb-fdad7678b74a170ead1ef967007be087435715f7.zip
gdb-fdad7678b74a170ead1ef967007be087435715f7.tar.gz
gdb-fdad7678b74a170ead1ef967007be087435715f7.tar.bz2
Use host_address_to_string in compile_cplus_instance::enter_scope
This patch fixes a problem being reported by the buildbot with an invalid argument to a "%p" printf format. Instead of "%p", the debug output is changed to use "%s" and host_address_to_string. gdb/ChangeLog * compile/compile-cplus-types.c (compile_cplus_instance::enter_scope): Use "%s" and host_address_to_string instead of "%p" in printf.
-rw-r--r--gdb/ChangeLog5
-rw-r--r--gdb/compile/compile-cplus-types.c5
2 files changed, 9 insertions, 1 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 43e22e2..2e59caf 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2018-08-30 Keith Seitz <keiths@redhat.com>
+
+ * compile/compile-cplus-types.c (compile_cplus_instance::enter_scope):
+ Use "%s" and host_address_to_string instead of "%p" in printf.
+
2018-08-29 Keith Seitz <keiths@redhat.com>
* Makefile.in (SUBDIR_GCC_COMPILE_SRCS): Add compile-cplus-symbols.c
diff --git a/gdb/compile/compile-cplus-types.c b/gdb/compile/compile-cplus-types.c
index 9425fc6..e86a573 100644
--- a/gdb/compile/compile-cplus-types.c
+++ b/gdb/compile/compile-cplus-types.c
@@ -259,7 +259,10 @@ compile_cplus_instance::enter_scope (compile_scope &new_scope)
if (must_push)
{
if (debug_compile_cplus_scopes)
- fprintf_unfiltered (gdb_stdlog, "entering new scope %p\n", new_scope);
+ {
+ fprintf_unfiltered (gdb_stdlog, "entering new scope %s\n",
+ host_address_to_string (&new_scope));
+ }
/* Push the global namespace. */
plugin ().push_namespace ("");