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

Region inference member constraints #344

Merged

Conversation

nikomatsakis
Copy link
Contributor

I started working on expanding the chapter on region inference, so as to better document rust-lang/rust#61775. Didn't really get to member constraints yet.

@mark-i-m
Copy link
Member

@nikomatsakis Are you saying that we should wait for you to push more?

@nikomatsakis
Copy link
Contributor Author

Yes.

@mark-i-m mark-i-m added the S-waiting-on-author Status: this PR is waiting for additional action by the OP label Jun 18, 2019
@nikomatsakis
Copy link
Contributor Author

OK, I pushed a lot more content now.

'd: 'c
```

Here we have two SCCs: S0 contains `'a` and `'b`, and S1 contains `'c`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Here we have two SCCs: S0 contains `'a` and `'b`, and S1 contains `'c`
Here we have two SCCs: `S0` contains `'a` and `'b`, and S1 contains `'c`

@lqd
Copy link
Member

lqd commented Jun 20, 2019

wow, these are so good !

Copy link
Member

@mark-i-m mark-i-m left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! I was wondering what the SCC stuff was.

There are a bunch of nits/suggestions, but they are mostly typos, minor things.

@@ -0,0 +1,3 @@
# Reporting region errors

TODO: we should discuss how to generate errors from the results of these analyses.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can actually write a bit here from working on rust-lang/rust#58281

src/borrow_check/region_inference/member_constraints.md Outdated Show resolved Hide resolved
nikomatsakis and others added 24 commits June 24, 2019 14:32
@mark-i-m
Copy link
Member

@nikomatsakis r=me when CI is green. There is one long line (in the ToC, lol):

src/SUMMARY.md
	82 :             - [Placeholders and universes](./borrow_check/region_inference/placeholders_and_universes.md)

@mark-i-m mark-i-m merged commit abf512f into rust-lang:master Jun 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-author Status: this PR is waiting for additional action by the OP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants