Fix arity check for function arguments.

This commit is contained in:
Mikaël Capelle
2023-09-27 20:22:34 +02:00
parent eb2c874a39
commit 328b6e7710
2 changed files with 5 additions and 4 deletions
+3 -3
View File
@@ -554,12 +554,12 @@ namespace mo2::python {
py::return_value_policy::reference) py::return_value_policy::reference)
.def("onAboutToRun", .def("onAboutToRun",
py::overload_cast<std::function<bool(QString const&)> const&>( py::overload_cast<std::function<bool(QString const&, const QDir&,
const QString&)> const&>(
&IOrganizer::onAboutToRun), &IOrganizer::onAboutToRun),
"callback"_a) "callback"_a)
.def("onAboutToRun", .def("onAboutToRun",
py::overload_cast<std::function<bool(QString const&, const QDir&, py::overload_cast<std::function<bool(QString const&)> const&>(
const QString&)> const&>(
&IOrganizer::onAboutToRun), &IOrganizer::onAboutToRun),
"callback"_a) "callback"_a)
.def("onFinishedRun", &IOrganizer::onFinishedRun, "callback"_a) .def("onFinishedRun", &IOrganizer::onFinishedRun, "callback"_a)
+2 -1
View File
@@ -21,7 +21,8 @@ namespace mo2::python::detail {
auto required_count = args_count - defaults_count; auto required_count = args_count - defaults_count;
return required_count <= arity // cannot require more parameters than given, return required_count <= arity // cannot require more parameters than given,
&& (args_count >= arity || varargs); // must accept enough parameters. && (args_count >= arity ||
!varargs.is_none()); // must accept enough parameters.
} }
} // namespace mo2::python::detail } // namespace mo2::python::detail