Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade reads and modifies error messages #4485

Open
MikaelMayer opened this issue Aug 29, 2023 · 0 comments
Open

Upgrade reads and modifies error messages #4485

MikaelMayer opened this issue Aug 29, 2023 · 0 comments
Labels
area: error-reporting Clarity of the error reporting part: documentation Dafny's reference manual, tutorial, and other materials

Comments

@MikaelMayer
Copy link
Member

What change in documentation do you suggest?

Currently, reads errors look like this:

Error: insufficient reads clause to invoke function

which is very hard to decipher, has no actionable item, does not seem linked to verification.
I think an error message like this would help disambiguate what's going on:

Inside methods
Error: could not prove that the function's reads consists only of fresh objects or object/fields that the enclosing method declares to read

Inside functions
Error: could not prove that the function's reads consists only of object/fields that the enclosing function declares to read

@MikaelMayer MikaelMayer added part: documentation Dafny's reference manual, tutorial, and other materials area: error-reporting Clarity of the error reporting labels Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: error-reporting Clarity of the error reporting part: documentation Dafny's reference manual, tutorial, and other materials
Projects
None yet
Development

No branches or pull requests

1 participant