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

Merge WhereClause and WhereClauseGoal into a single enum #37

Closed
wants to merge 1 commit into from
Closed

Merge WhereClause and WhereClauseGoal into a single enum #37

wants to merge 1 commit into from

Conversation

scalexm
Copy link
Member

@scalexm scalexm commented May 30, 2017

The two enums were merged into a single one named WhereClause. In particular, chalk syntax is now a superset of rust syntax and WellFormed types are now allowed into the environment. This is a preparation for solving #12.

The two enums were merged into a single one named `WhereClause`.
In particular, chalk syntax is now a superset of rust syntax
and `WF` types are now allowed into the environment.
@aturon
Copy link
Member

aturon commented May 30, 2017

Note, this change is subsumed by the overhaul here.

@nikomatsakis
Copy link
Contributor

OK, I'm going to close this in favor of #36, which does indeed do a similar thing -- though it separates out unifications, which I overlooked when talking to @scalexm in the first place (unification goals is probably why I made the split between "where-clause-goal" and "where-clause" goal to start).

@nikomatsakis
Copy link
Contributor

Sorry @scalexm for leading you down a false start. =)

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.

3 participants