Fix operator __ne__

__ne__ was defined as != which means it would just recurse calling itself.
Thriftpy · Dec 11, 2014 · dcfe18a · hit9 · Jan 13, 2015 · dcfe18a
1 parent f5fdb83
commit dcfe18a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/tests/test_container.py b/tests/test_container.py
@@ -17,6 +17,9 @@ def test_list_item():
     l_item2 = deserialize(container.ListItem(), b)
     assert l_item == l_item2
 
+    l_item3 = container.ListItem()
+    assert l_item != l_item3
+
 
 def test_map_item():
     m_item = container.MapItem()

diff --git a/thriftpy/thrift.py b/thriftpy/thrift.py
@@ -107,7 +107,7 @@ def __hash__(self):
         return super(TPayload, self).__hash__()
 
     def __ne__(self, other):
-        return self != other
+        return not self.__eq__(other)
 
 
 class TClient(object):