The previous implementation unconditionally closed the first open upvalue when executing CLOSE_UPVALUE, which could crash when a local variable was never actually captured by a closure. Changed closeUpvalue to closeUpvalues with a stack threshold parameter, so only upvalues whose stack slot is at or above the given pointer are closed. This prevents closing nonexistent upvalues for locals that were never closed over, fixing segfaults in unused closure scenarios. Added regression tests for unused closures and for nested scopes where only some locals are captured.
This contains the automated validation suite for the VM and built-in libraries.
-
benchmark/- Performance tests. These aren't strictly pass/fail, but let us compare performance both against other languages and against previous builds of Wren itself. -
core/- Tests for the built in core library, mainly methods on the core classes. If a bug is inwren_core.corwren_value.c, it will most likely break one of these tests. -
language/- Tests of the language itself, its grammar and runtime semantics. If a bug is inwren_compiler.corwren_vm.c, it will most likely break one of these tests. This includes tests for the syntax for the literal forms of the core classes. -
limit/- Tests for various hardcoded limits. The language doesn't officially specify these limits, but the Wren implementation has them. These tests ensure that limit behavior is well-defined and tested.