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.

Sign up for GitHub

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

Jump to bottom

Rename or remove {:layerQuantifier} attribute #35

Closed
richardlford opened this issue Sep 27, 2016 · 1 comment · Fixed by #2816
Closed

Rename or remove {:layerQuantifier} attribute #35

richardlford opened this issue Sep 27, 2016 · 1 comment · Fixed by #2816
Labels
misc: cleanup Cleanups in the implementation or in corners of the language part: verifier Translation from Dafny to Boogie (translator) status: designed Issues that have a complete story on how to implement this feature and why we want it

Comments

@richardlford
Copy link
Collaborator

richardlford commented Sep 27, 2016 •
edited
Loading

Rustan has commented that, if the {:layerQuantifier} is to stay at all, that it should be renamed to {:fuelQuantifier}. In addition, if this is kept, the reference manual needs a more complete description of the purpose and effect of this attribute. If it is only used internally its name should start with a leading underscore.
@acioc acioc added kind: enhancement Enhancements, feature requests, etc. These are NOT bugs, but ways we can improve Dafny and removed code-cleanup labels Jun 22, 2020
@acioc acioc closed this as completed Jul 8, 2020
camrein added a commit that referenced this issue Apr 8, 2021
@camrein
Merge pull request #35 from dafny-lang/improvement/upgrade-dafny-soure
95ca117 
Upgrade the dafny submodule to cb3b874
@MikaelMayer MikaelMayer reopened this Mar 18, 2022
@MikaelMayer MikaelMayer mentioned this issue Mar 18, 2022
Merged
@cpitclaudel cpitclaudel added part: verifier Translation from Dafny to Boogie (translator) misc: cleanup Cleanups in the implementation or in corners of the language status: designed Issues that have a complete story on how to implement this feature and why we want it and removed kind: enhancement Enhancements, feature requests, etc. These are NOT bugs, but ways we can improve Dafny labels May 10, 2022
@davidcok
Copy link
Collaborator

davidcok commented Sep 29, 2022 •
edited
Loading

The only place this attribute is mentioned is checking for its presence in ./Source/DafnyCore/Verifier/Translator.ExpressionTranslator.cs.

There are no tests that try it out. I suggest it be removed from the code and the RM.

All CI tests pass if the code mentioning this attribute is removed.

@davidcok davidcok mentioned this issue Sep 29, 2022
Merged
davidcok added a commit that referenced this issue Sep 30, 2022
@davidcok
Remove support for :layerQuantifier (#2816)
bfd25c5 
Fixes #35


<small>By submitting this pull request, I confirm that my contribution
is made under the terms of the [MIT
license](https://github.com/dafny-lang/dafny/blob/master/LICENSE.txt).</small>

Co-authored-by: davidcok <davidcok@github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
misc: cleanup Cleanups in the implementation or in corners of the language part: verifier Translation from Dafny to Boogie (translator) status: designed Issues that have a complete story on how to implement this feature and why we want it
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove support for :layerQuantifier davidcok/dafny
6 participants
@seanmcl @MikaelMayer @acioc @davidcok @richardlford @cpitclaudel