[Lldb-commits] [PATCH] D69742: [LLDB][Python] remove ArgInfo::count

Lawrence D'Anna via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Nov 1 17:38:50 PDT 2019


lawrence_danna created this revision.
lawrence_danna added reviewers: labath, mgorny, JDevlieghere.
Herald added a project: LLDB.

This patch updates the last user of ArgInfo::count and deletes 
it.   I also delete `GetNumInitArguments()` and `GetInitArgInfo()`. 
Classess are callables and `GetArgInfo()` should work on them.

On python 3 it already works, of course. `inspect` is good.

On python 2 we have to add yet another special case.   But hey if 
python 2 wasn't crufty we wouln't need python 3.

I also delete `is_bound_method` becuase it is unused.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D69742

Files:
  lldb/scripts/Python/python-wrapper.swig
  lldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.cpp
  lldb/source/Plugins/ScriptInterpreter/Python/PythonDataObjects.h
  lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.cpp
  lldb/unittests/ScriptInterpreter/Python/PythonDataObjectsTests.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D69742.227555.patch
Type: text/x-patch
Size: 12610 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20191102/86d1c7d8/attachment.bin>


More information about the lldb-commits mailing list