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

Check for a type-confusion issue in IndDef #1294

Closed

Conversation

talsewell
Copy link
Contributor

Add an additional user-input check to InductiveRelation.check_clause to catch a case where different clauses of an inductive relation would give it different types. This would lead to low-level matching errors later in the process.

Add an additional user-input check to InductiveRelation.check_clause to catch a
case where different clauses of an inductive relation would give it different
types. This would lead to low-level matching errors later in the process.
@talsewell
Copy link
Contributor Author

I'm going to pull this PR, the proposal addresses an issue that I'd misunderstood.

The type inconsistency would already not be permitted if the user had typed it directly. It appears (deep in the code) because the inner machinery has inconsistently made adjustments for "schematic" variables in the relation.

@talsewell talsewell closed this Sep 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant