<div id="reply-content">Okay, I'll do that when I get home. Do you have more specific version requirements?
    </div><div id="reply-content"><br></div><div id="reply-content">Regards,</div><div id="reply-content"><br></div><div id="reply-content">  Filipe</div>
    <div id="1E2233F9B9854E3286A3E08E1B062788"><div><br></div>-- <br>Filipe Cabecinhas<br><div>Sent with <a href="http://www.sparrowmailapp.com/?sig">Sparrow</a></div><div><br></div></div>
     
    <p style="color: #A0A0A8;">On Thursday, 10 de May de 2012 at 18:51, Johnny Chen wrote:</p>
    <blockquote type="cite" style="border-left-style:solid;border-width:1px;margin-left:0px;padding-left:10px;">
        <div id="quoted-message-content"><div><div>Hi Filipe,</div><div><br></div><div>Can you try 1.3-ish swig to see whether your patch is backward compatible?</div><div><br></div><div>On May 10, 2012, at 6:45 AM, Filipe Cabecinhas <filcab@gmail.com> wrote:</div><div><br></div><blockquote type="cite"><div><div>Hi all, </div><div><br></div><div>My test suite was erring out (on trunk) on some tests. I tracked it and found out that swig (at least version 2.0.6, that I installed with homebrew) always converts ints and longs into PyLongObject. which will fail in any of lldb's __len__() methods, since these methods _must_ return a PyIntObject. I get an exception in  TestSBData.SBDataAPICase.test_with_dsym_and_run_command saying this: </div><div><br></div><div>ERROR: test_with_dsym_and_run_command (TestSBData.SBDataAPICase)</div><div>Test the SBData APIs.</div><div>----------------------------------------------------------------------</div><div>Traceback (most recent call last):</div><div>File "/Users/filcab/dev/lldb-mine/test/lldbtest.py", line 376, in wrapper</div><div>return func(self, *args, **kwargs)</div><div>File "/Users/filcab/dev/lldb-mine/test/lldbtest.py", line 410, in wrapper</div><div>return func(self, *args, **kwargs)</div><div>File "/Users/filcab/dev/lldb-mine/test/python_api/sbdata/TestSBData.py", line 20, in test_with_dsym_and_run_command</div><div>self.data_api()</div><div>File "/Users/filcab/dev/lldb-mine/test/python_api/sbdata/TestSBData.py", line 235, in data_api</div><div>self.assertTrue(data2.uint64s == [1,2,3,4,5], 'read_data_helper failure: data2 == [1,2,3,4,5]')</div><div>File "/Users/filcab/dev/lldb-mine/build/Debug/LLDB.framework/Resources/Python/lldb/__init__.py", line 2198, in _read_all_uint64</div><div>return self._make_helper_uint64().all()</div><div>File "/Users/filcab/dev/lldb-mine/build/Debug/LLDB.framework/Resources/Python/lldb/__init__.py", line 2153, in all</div><div>return self[0:len(self)]</div><div>TypeError: __len__() should return an int</div><div>Config=i386-clang</div><div>----------------------------------------------------------------------</div><div><br></div><div><br></div><div><br></div><div><br></div><div>I'm attaching for review a patch that converts every __len__() return to a PyInt, which makes this test pass.</div><div>I'm also attaching for review a patch that makes SBProcess::ReadMemory accept PyLongObjects for the byteSize argument, as well as PyIntObjects.</div><div><br></div><div>Regards,</div><div><br></div><div> Filipe</div><div><br></div><div><lldb-SBProcess_ReadMemory-bugfix.patch><lldb-fix-__len__-methods.patch>_______________________________________________</div><div>lldb-commits mailing list</div><div>lldb-commits@cs.uiuc.edu</div><div>http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</div></div></blockquote></div></div>
         
         
         
         
    </blockquote>
     
    <div>
        <br>
    </div>