Skip to content

Commit bcd16ac

Browse files
fix: clarify gap-detection comment in bindKeywordArguments
Agent-Logs-Url: https://github.com/Ultimaker/CuraFormulaeEngine/sessions/bb29b144-a878-4363-8aec-f3d1835aad79 Co-authored-by: casperlamboo <6638028+casperlamboo@users.noreply.github.com>
1 parent cd1de48 commit bcd16ac

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

src/ast/fn_application_expr.cpp

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,8 +64,9 @@ std::optional<std::vector<eval::Value>> bindKeywordArguments(
6464
bound.push_back(slot.value());
6565
}
6666

67-
// If any slot beyond the bound range has a value there is a gap (a required
68-
// earlier argument is missing while a later one was supplied via keyword).
67+
// If any slot beyond the consecutively-filled slots from the start has a
68+
// value there is a gap (a required earlier argument is missing while a
69+
// later one was supplied via keyword).
6970
// Return nullopt so the caller can report InvalidNumberOfArguments.
7071
for (size_t i = bound.size(); i < slots.size(); ++i)
7172
{

0 commit comments

Comments
 (0)