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

Warn the user if contracts are unused #2741

Open
JustusAdam opened this issue Sep 6, 2023 · 0 comments
Open

Warn the user if contracts are unused #2741

JustusAdam opened this issue Sep 6, 2023 · 0 comments
Assignees
Labels
[C] Feature / Enhancement A new feature request or enhancement to an existing feature.

Comments

@JustusAdam
Copy link
Contributor

The contracts as added by #2655 ignore unused contracts. However a scenario is conceivable where a user specifies a contract for a function and expect this to be checked/enforced automatically as is the case in many other verifiers.

@feliperodri suggested we should warn the user if an "unused" contract is encountered.. This issue should serve as a discussion board for when such a warning should be issued.

Some initial questions:

  • Should issuing the warning depend on reachability of the contracted code?
  • What it --harness is passed?
  • Should we only warn that a check is missing or also that replacement could be used?
  • Should the warning depend on whether -Zfunction-contracts is enabled?
@JustusAdam JustusAdam added the [C] Feature / Enhancement A new feature request or enhancement to an existing feature. label Sep 6, 2023
@feliperodri feliperodri self-assigned this Feb 28, 2024
@feliperodri feliperodri added this to the Function Contracts MVP milestone Jun 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C] Feature / Enhancement A new feature request or enhancement to an existing feature.
Projects
None yet
Development

No branches or pull requests

2 participants