[Lldb-commits] [PATCH] D69468: [LLDB][breakpoints] ArgInfo::count -> ArgInfo::max_positional_args

[Pavel Labath via Phabricator via lldb-commits]

labath added inline comments.


================
Comment at: lldb/scripts/Python/python-wrapper.swig:64
 
+    unsigned max_positional_args = PythonCallable::ArgInfo::UNBOUNDED;
+    if (auto arg_info = pfunc.GetArgInfo()) {
----------------
lawrence_danna wrote:
> labath wrote:
> > Is there any case where fetching the argument info will fail, but we still can successfully call the target object? Should we just bail out here?
> probably not?     My thinking is that since there's no meaningful way to return an error from this function we may as well try to call it and let the exception get logged.   But I dunno.    Should I change it?
Hmm... In that case, I think it would be better to print the error which caused the ArgInfo fetching to fail. Given that, in the new way of doing things, the `PyErr_Cleaner` object is not going to work anyway, we might as well use this opportunity to create a different mechanism for printing exceptions.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69468