Link: test: change decl from `ref` to `var` and add future for `ref` version by arezaii · Pull Request #19357 · chapel-lang/chapel · GitHub
Merge pull request #19357 from arezaii/string-leak-test
test: change decl from
var and add future for
This PR updates the declaration of a variable in a test from
When troubleshooting failing tests with the dyno parser, we discovered
that this test declared a
ref to the
vararg formal, which is itself
returned as a
ref, so this should have produced a type or lifetime error
in the current compiler. However, it was compiling and running because
internally it was converting the
ref to a
const val. Redeclaring the
variable as a
var keeps tests passing.
Captured the existing test as a future referring to issue
Capturing `const ref` argument as mutable `ref` should be an error · Issue #19614 · chapel-lang/chapel · GitHub.
- [x] paratest
- [x] paratest with
reviewed by @mppf - thanks!
Signed-off-by: arezaii email@example.com
Modified Files: A test/functions/varargs/stringVarargLeakByRef.chpl