Branch: refs/heads/main
Revision: a5cef8ec079e0fb540e18458beb2163ff24a8e22
Author: vasslitvinov
Link: Fix more fallouts from #23992 by vasslitvinov · Pull Request #23999 · chapel-lang/chapel · GitHub
Log Message:
Fix more fallouts from #23992 (#23999)
This reverts the change to Errors.chpl made in #23992's commit
cfebf149d2. That change switched two asserts from displaying
someLocale.id
to displaying someLocale
, which threw off a lot of
tests due to problems with resolution order. For example, the test
errhandling/parallel/forall-wrap-first-error
started reporting "error:
module-scope variable 'count' may be used before it is initialized".
Using someLocale.id
in the assertions avoids these problems.
Correspondingly, this reverts the change to
test/modules/bradc/userInsteadOfStandard/ChapelIO.chpl
as no longer
needed.
Merging without a review for expediency.
Diff:
M modules/standard/Errors.chpl
M test/errhandling/errorMessages/taskerrors-remote-1.good
M test/errhandling/errorMessages/taskerrors-remote-2.good
M test/modules/bradc/userInsteadOfStandard/ChapelIO.chpl
https://github.com/chapel-lang/chapel/pull/23999.diff