From 7543c960b00ddea23f6105eff4358ec214a7f93c Mon Sep 17 00:00:00 2001 From: Hannes Domani Date: Fri, 8 Dec 2023 18:19:42 +0100 Subject: Use pretty printers for struct member stubs PR29079 shows that pretty printers can be used for an incomplete type (stub), but only when printing it directly, not if it's part of another struct: ``` (gdb) p s $1 = {pp m_i = 5} (gdb) p s2 $2 = {m_s = , m_l = 20} ``` The reason is simply that in common_val_print the check for stubs is before any pretty printer is tried. It works if the pretty printer is tried before the stub check: ``` (gdb) p s $1 = {pp m_i = 5} (gdb) p s2 $2 = {m_s = {pp m_i = 10}, m_l = 20} ``` Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=29079 Approved-By: Tom Tromey --- gdb/testsuite/gdb.python/py-prettyprint-stub.exp | 59 ++++++++++++++++++++++++ 1 file changed, 59 insertions(+) create mode 100644 gdb/testsuite/gdb.python/py-prettyprint-stub.exp (limited to 'gdb/testsuite/gdb.python/py-prettyprint-stub.exp') diff --git a/gdb/testsuite/gdb.python/py-prettyprint-stub.exp b/gdb/testsuite/gdb.python/py-prettyprint-stub.exp new file mode 100644 index 0000000..79492ec --- /dev/null +++ b/gdb/testsuite/gdb.python/py-prettyprint-stub.exp @@ -0,0 +1,59 @@ +# Copyright (C) 2023 Free Software Foundation, Inc. + +# This program is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 3 of the License, or +# (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program. If not, see . + +# This file is part of the GDB testsuite. +# It tests Python-based pretty-printing of stubs. + +load_lib gdb-python.exp + +require allow_python_tests + +standard_testfile .cc py-prettyprint-stub-2.cc + +set srcfiles [list $srcfile $srcfile2] + +if { [build_executable_from_specs \ + "failed to prepare" \ + $testfile {c++} \ + $srcfile {c++ debug} \ + $srcfile2 {c++}] == -1 } { + return -1 +} + +# Start with a fresh gdb. +clean_restart $testfile + +if {![runto_main]} { + return +} + +set remote_python_file [gdb_remote_download host \ + ${srcdir}/${subdir}/${testfile}.py] + +gdb_breakpoint [gdb_get_line_number "Break here."] +gdb_continue_to_breakpoint "Break here" ".*Break here.*" + +# First test without pretty printer. +gdb_test "print s" " = " +gdb_test "print s2" " = {m_s = , m_l = 20}" + +# Load pretty printer. +gdb_test_no_output "source ${remote_python_file}" "load python file" + +# Test with pretty printer. +with_test_prefix pp { + gdb_test "print s" " = {pp m_i = 5}" + gdb_test "print s2" " = {m_s = {pp m_i = 10}, m_l = 20}" +} -- cgit v1.1