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

move remaining vectors, boxes in chalk-ir into associated types on Interner #369

Closed
3 of 5 tasks
nikomatsakis opened this issue Mar 31, 2020 · 7 comments · Fixed by #386
Closed
3 of 5 tasks

move remaining vectors, boxes in chalk-ir into associated types on Interner #369

nikomatsakis opened this issue Mar 31, 2020 · 7 comments · Fixed by #386
Assignees
Labels
current-sprint Being worked on in the current sprint

Comments

@nikomatsakis
Copy link
Contributor

nikomatsakis commented Mar 31, 2020

In general, we have been trying to remove hard-coded vectors and boxes from the chalk-ir APIs, and moving them instead to Interner associated types. See the chalk book for more background on the Interner trait and its design.

This issue tracks the removal of the final vectors etc:

Here are some example PRs showing how this removal can be done:

This issue has been assigned to @crlf0710 via this comment.

@nikomatsakis nikomatsakis added the current-sprint Being worked on in the current sprint label Apr 1, 2020
@crlf0710
Copy link
Member

crlf0710 commented Apr 3, 2020

Will have a try this weekend.

@crlf0710
Copy link
Member

crlf0710 commented Apr 4, 2020

Edit: Seems #370 is in flight.

@nikomatsakis
Copy link
Contributor Author

@crlf0710 there are still other issues to claim -- e.g., QuantifiedWhereClauses, ParameterKinds, etc. Feel free to claim one of those.

@crlf0710
Copy link
Member

crlf0710 commented Apr 7, 2020

Changed Vec<Parameter> to Substitution in Fn in #378.

I'll have a try on Vec<QuantifiedWhereClause<I>> next.

@crlf0710
Copy link
Member

crlf0710 commented Apr 7, 2020

Introduced QuantifiedWhereClauses in #379.

@crlf0710
Copy link
Member

Given the rest a try in #386

@crlf0710
Copy link
Member

@rustbot claim

@rustbot rustbot self-assigned this Apr 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
current-sprint Being worked on in the current sprint
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants