[Lldb-commits] [PATCH] D17545: Fix PythonDataObjectsTests for python 2

Pavel Labath via lldb-commits lldb-commits at lists.llvm.org
Tue Feb 23 08:47:16 PST 2016


labath added inline comments.

================
Comment at: unittests/ScriptInterpreter/Python/PythonDataObjectsTests.cpp:212
@@ -211,3 +211,2 @@
     PythonBytes python_bytes(PyRefType::Owned, py_bytes);
-    EXPECT_EQ(PyObjectType::Bytes, python_bytes.GetObjectType());
 
----------------
zturner wrote:
> Seems like this check is still valid?
Not on python 2, I get "string" there. We can't have checks on line 212 and 216 both be true, so I assumed "string" is intended. Note that I have simply moved this check into the python-3 branch.


http://reviews.llvm.org/D17545





More information about the lldb-commits mailing list