This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[RFA/mi] Remove varobj enumeration XFAIL
- From: Keith Seitz <keiths at redhat dot com>
- To: "gdb-patches at sourceware dot org ml" <gdb-patches at sourceware dot org>
- Date: Fri, 18 Nov 2011 12:33:12 -0800
- Subject: [RFA/mi] Remove varobj enumeration XFAIL
Hi,
mi{,2}-var-display.exp contains an XFAIL for the following test:
# Test: c_variable-7.51
# Desc: value of e
mi_gdb_test "-var-evaluate-expression e" \
"\\^done,value=\"FIXME\"" \
"eval variable e"
There is no bug filed for this (or for any of the other twenty-six
setup_xfail calls in gdb.mi/), and I could find no history of why this
was expected to fail. AFAICT, it has been this way from the beginning.
I suspect that this was xfailed because the test originally stopped in
the beginning of the function, and there was no way to (easily?) get the
value of the initialized enum.
But that all changed with this commit:
http://sourceware.org/ml/gdb-patches/2008-04/msg00007.html
This xfail no longer seems necessary, so I've attached a patch to fix
the test case and remove the XFAIL.
Keith
testsuite/ChangeLog
2011-11-18 Keith Seitz <keiths@redhat.com>
* gdb.mi/mi-var-display.exp: Remove XFAIL for c_variable-7.51,
-var-evaluate-expression of an enumeration and fix expected result.
* gdb.mi/mi2-varvdisplay.exp: Likewise.
diff --git a/gdb/testsuite/gdb.mi/mi-var-display.exp b/gdb/testsuite/gdb.mi/mi-var-display.exp
index ff27407..ad5bd23 100644
--- a/gdb/testsuite/gdb.mi/mi-var-display.exp
+++ b/gdb/testsuite/gdb.mi/mi-var-display.exp
@@ -529,13 +529,11 @@ mi_list_varobj_children anons {
# Desc: create enum e
mi_create_varobj e e "create local variable e"
-setup_xfail "*-*-*"
# Test: c_variable-7.51
# Desc: value of e
mi_gdb_test "-var-evaluate-expression e" \
- "\\^done,value=\"FIXME\"" \
+ "\\^done,value=\"bar\"" \
"eval variable e"
-clear_xfail "*-*-*"
# Test: c_variable-7.52
# Desc: type of e
diff --git a/gdb/testsuite/gdb.mi/mi2-var-display.exp b/gdb/testsuite/gdb.mi/mi2-var-display.exp
index 828614b..2fb0f47 100644
--- a/gdb/testsuite/gdb.mi/mi2-var-display.exp
+++ b/gdb/testsuite/gdb.mi/mi2-var-display.exp
@@ -528,13 +528,11 @@ mi_list_varobj_children anons {
# Desc: create enum e
mi_create_varobj e e "create local variable e"
-setup_xfail "*-*-*"
# Test: c_variable-7.51
# Desc: value of e
mi_gdb_test "-var-evaluate-expression e" \
- "\\^done,value=\"FIXME\"" \
+ "\\^done,value=\"bar\"" \
"eval variable e"
-clear_xfail "*-*-*"
# Test: c_variable-7.52
# Desc: type of e