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

Jonas Devlieghere via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Nov 4 10:58:07 PST 2019


JDevlieghere added inline comments.


================
Comment at: lldb/unittests/ScriptInterpreter/Python/PythonDataObjectsTests.cpp:764
   }
 
 #if PY_MAJOR_VERSION >= 3 && PY_MINOR_VERSION >= 3
----------------
lawrence_danna wrote:
> JDevlieghere wrote:
> > Should we check `has_varargs` here as well?
> nope, it's an implementation detail now.   not available here.
Makes sense, sorry for the noise! 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69742/new/

https://reviews.llvm.org/D69742





More information about the lldb-commits mailing list