public inbox for gdb-cvs@sourceware.org
help / color / mirror / Atom feed
* [binutils-gdb] Fix printcmds.exp failure for wide strings tests.
@ 2019-07-09 17:39 Philippe Waroquiers
0 siblings, 0 replies; only message in thread
From: Philippe Waroquiers @ 2019-07-09 17:39 UTC (permalink / raw)
To: gdb-cvs
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=0826779b99b49f7f8df5d186f3c481b6007010d4
commit 0826779b99b49f7f8df5d186f3c481b6007010d4
Author: Philippe Waroquiers <philippe.waroquiers@skynet.be>
Date: Tue Jul 9 19:36:17 2019 +0200
Fix printcmds.exp failure for wide strings tests.
wchar_t type must be known to create wide strings.
As this type is predefined when current GDB language is C++,
switch to c++ for the wide strings tests.
Problem analysis and fix by Sergio.
Diff:
---
gdb/testsuite/ChangeLog | 6 ++++++
gdb/testsuite/gdb.base/printcmds.exp | 29 ++++++++++++++---------------
2 files changed, 20 insertions(+), 15 deletions(-)
diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog
index a7d09cb..3df5e58 100644
--- a/gdb/testsuite/ChangeLog
+++ b/gdb/testsuite/ChangeLog
@@ -1,3 +1,9 @@
+2019-07-09 Sergio Durigan Junior <sergiodj@redhat.com>
+
+ * gdb.base/printcmds.exp (test_printf_convenience_var): Set
+ language to "c++" before dealing with wchar_t. Remove DO_WSTRING
+ argument and update callers.
+
2019-07-09 Andrew Burgess <andrew.burgess@embecosm.com>
* gdb.arch/amd64-break-on-asm-line.S: New file.
diff --git a/gdb/testsuite/gdb.base/printcmds.exp b/gdb/testsuite/gdb.base/printcmds.exp
index 0e3126b..edd5b31 100644
--- a/gdb/testsuite/gdb.base/printcmds.exp
+++ b/gdb/testsuite/gdb.base/printcmds.exp
@@ -935,11 +935,8 @@ proc test_repeat_bytes {} {
# Test printf of convenience variables.
# These tests can be done with or without a running inferior.
# PREFIX ensures uniqueness of test names.
-# DO_WSTRING 1 tells to test printf of wide strings. Wide strings tests
-# must be skipped (DO_WSTRING 0) if the wchar_t type is not yet known by
-# GDB, as this type is needed to create wide strings.
-proc test_printf_convenience_var {prefix do_wstring} {
+proc test_printf_convenience_var {prefix} {
with_test_prefix "conv var: $prefix" {
gdb_test_no_output "set var \$cstr = \"abcde\"" "set \$cstr"
@@ -958,12 +955,16 @@ proc test_printf_convenience_var {prefix do_wstring} {
gdb_test_no_output "set language auto" "set language auto"
gdb_test "printf \"astr val = %s\\n\", \$astr" "astr val = fghij" \
"printf \$astr, auto language"
- if {$do_wstring} {
- gdb_test_no_output "set var \$wstr = L\"facile\"" \
- "set \$wstr"
- gdb_test "printf \"wstr val = %ls\\n\", \$wstr" \
- "wstr val = facile" "printf \$wstr"
- }
+ # Wide strings can only be created when wchar_t type is known.
+ # Switch to c++ for the wide strings tests, as wchar_t is predefined
+ # when current language is c++.
+ # See above "set language ada" about why we use gdb_test.
+ gdb_test "set language c++" ".*" "set language c++"
+ gdb_test_no_output "set var \$wstr = L\"facile\"" \
+ "set \$wstr"
+ gdb_test "printf \"wstr val = %ls\\n\", \$wstr" \
+ "wstr val = facile" "printf \$wstr"
+ gdb_test_no_output "set language auto" "set language auto, wstring"
}
}
@@ -985,9 +986,7 @@ gdb_test "print \$cvar = \"abc\"" " = \"abc\""
gdb_test "print sizeof (\$cvar)" " = 4"
# Similarly, printf of a string convenience var should work without a target.
-# At this point, we cannot create a wide string convenience var, as the
-# wchar_t type is not yet known, so skip the wide string tests.
-test_printf_convenience_var "no target" 0
+test_printf_convenience_var "no target"
# GDB used to complete the explicit location options even when
# printing expressions.
@@ -1019,11 +1018,11 @@ if ![runto_main] then {
}
# With a running target, printf convenience vars should of course work.
-test_printf_convenience_var "with target" 1
+test_printf_convenience_var "with target"
# It should also work when inferior function calls are forbidden.
gdb_test_no_output "set may-call-functions off"
-test_printf_convenience_var "with target, may-call-functions off" 1
+test_printf_convenience_var "with target, may-call-functions off"
gdb_test_no_output "set may-call-functions on"
test_integer_literals_accepted
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2019-07-09 17:39 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-07-09 17:39 [binutils-gdb] Fix printcmds.exp failure for wide strings tests Philippe Waroquiers
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).