Revert "fail with the first unmet constraint to avoid running into type mismatches"

This reverts commit aee67d58f0.

I believe the original intention of this code was to fall through to
later constraints, even if earlier constraints were not satisfied. This
is not captured by any of the tests in the test suite, which is an
oversight on my part.
This commit is contained in:
Tristan Penman 2023-08-24 09:18:24 +10:00
parent aee67d58f0
commit 92b9514bde

View File

@ -120,7 +120,7 @@ public:
{
bool allTrue = true;
for (auto &&constraint : m_constraints) {
allTrue = allTrue && applyFunction(*constraint);
allTrue = applyFunction(*constraint) && allTrue;
}
return allTrue;