================
@@ -188,8 +188,13 @@ class ScriptedPythonInterface : virtual public
ScriptedInterface {
// This addresses the cases where the embedded interpreter session
// dictionary is passed to the extension initializer which is not used
// most of the time.
+ // Note, though none of our API's suggest defining the interfaces with
+ // varargs, we have some extant clients that were doing that. To keep
+ // from breaking them, we just say putting a varargs in these signatures
+ // turns off argument checking.
size_t num_args = sizeof...(Args);
- if (num_args != arg_info->max_positional_args) {
+ if (arg_info->max_positional_args != PythonCallable::ArgInfo::UNBOUNDED
&&
----------------
jimingham wrote:
I don't think that's necessary. Someone did this once and then sadly that got
copied around and so there are a few versions I need to maintain support for.
I'm not sure this is something anyone else is likely to think is a good idea.
https://github.com/llvm/llvm-project/pull/166883
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
