This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
FYI: fix bug with explicit operator calls
- From: Tom Tromey <tromey at redhat dot com>
- To: gdb-patches at sourceware dot org
- Date: Mon, 15 Mar 2010 11:24:48 -0600
- Subject: FYI: fix bug with explicit operator calls
- Reply-to: tromey at redhat dot com
I'm checking this in.
This adds some support for explicit calls to an operator.
Built and regtested on x86-64 (compile farm).
Some new tests are included.
Tom
2010-03-15 Tom Tromey <tromey@redhat.com>
* c-exp.y (name_not_typename): Add 'operator' clause.
2010-03-15 Tom Tromey <tromey@redhat.com>
* gdb.cp/userdef.exp: Add tests for explicit calls to operator==.
* gdb.cp/userdef.cc (operator==): New function.
(main): New locals mem1, mem2.
diff --git a/gdb/c-exp.y b/gdb/c-exp.y
index 1af76c9..2577a99 100644
--- a/gdb/c-exp.y
+++ b/gdb/c-exp.y
@@ -1236,6 +1236,14 @@ name_not_typename : NAME
context where only a name could occur, this might be useful.
| NAME_OR_INT
*/
+ | operator
+ {
+ $$.stoken = $1;
+ $$.sym = lookup_symbol ($1.ptr,
+ expression_context_block,
+ VAR_DOMAIN,
+ &$$.is_a_field_of_this);
+ }
;
%%
diff --git a/gdb/testsuite/gdb.cp/userdef.cc b/gdb/testsuite/gdb.cp/userdef.cc
index 338c58a..56a735f 100644
--- a/gdb/testsuite/gdb.cp/userdef.cc
+++ b/gdb/testsuite/gdb.cp/userdef.cc
@@ -311,6 +311,11 @@ public:
int z;
};
+bool operator== (const Member &m1, const Member &m2)
+{
+ return m1.z == m2.z;
+}
+
class Container
{
public:
@@ -330,8 +335,12 @@ int main (void)
A1 two(4,5);
A1 three(0,0);
Container c;
+ Member mem1, mem2;
int val;
+ mem1.z = 5;
+ mem2.z = 7;
+
marker1(); // marker1-returns-here
cout << one; // marker1-returns-here
cout << two;
diff --git a/gdb/testsuite/gdb.cp/userdef.exp b/gdb/testsuite/gdb.cp/userdef.exp
index 2bbf95e..25cade7 100644
--- a/gdb/testsuite/gdb.cp/userdef.exp
+++ b/gdb/testsuite/gdb.cp/userdef.exp
@@ -113,6 +113,7 @@ gdb_test "print one > two" "\\\$\[0-9\]* = 0\[\r\n\]"
gdb_test "print one >= two" "\\\$\[0-9\]* = 0\[\r\n\]"
gdb_test "print one == two" "\\\$\[0-9\]* = 0\[\r\n\]"
+gdb_test "print one.operator== (two)" "\\\$\[0-9\]* = 0\[\r\n\]"
gdb_test "print one != two" "\\\$\[0-9\]* = 1\[\r\n\]"
@@ -155,5 +156,8 @@ gdb_test "print *c" "\\\$\[0-9\]* = \\(Member &\\) @$hex: {z = .*}"
gdb_test "print &*c" "\\\$\[0-9\]* = \\(Member \\*\\) $hex"
gdb_test "ptype &*c" "type = (struct|class) Member {(\[\r\n \]+public:)?\[\r\n \]+int z;\[\r\n\]+} &\\*"
+gdb_test "print operator== (mem1, mem2)" " = false"
+gdb_test "print operator== (mem1, mem1)" " = true"
+
gdb_exit
return 0