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

Remove "assert" keyword from CN syntax in most places #99

Open
bcpierce00 opened this issue Jun 9, 2024 · 2 comments
Open

Remove "assert" keyword from CN syntax in most places #99

bcpierce00 opened this issue Jun 9, 2024 · 2 comments
Labels
CN Issues related to the CN tool

Comments

@bcpierce00
Copy link
Collaborator

Summary

New CN users may be confused about where "assert" is required and where it is forbidden. Dhruv and I (Benjamin) discussed this a bit, leading to the following proposal:

  • Keep "assert" in the CN statements that are sometimes used within C function bodies.
  • Remove it everywhere else

(If removing it is difficult, then making it required everywhere that it is currently forbidden would be an acceptable alternative.)

@dc-mak dc-mak added the CN Issues related to the CN tool label Jun 10, 2024
@dc-mak dc-mak unassigned cp526 and dc-mak Jun 18, 2024
@ZippeyKeys12
Copy link
Collaborator

I strongly favor requiring assert everywhere. Having bare expressions in the middle of predicates would hurt readability for me.

@bcpierce00
Copy link
Collaborator Author

bcpierce00 commented Jun 18, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CN Issues related to the CN tool
Projects
None yet
Development

No branches or pull requests

4 participants