Skip to content

Conversation

@luqmana
Copy link
Contributor

@luqmana luqmana commented Apr 20, 2013

@nikomatsakis
Copy link
Contributor

This doesn't quite look right. I believe the correct thing to do is simply to modify the mem_categorization code to categorize rvalues as mutable. Only thing is that I am not sure if there are other places that assume that they are immutable, or if there will be other follow on changes needed. (Note: I had planned to do this as part of the code for #5074)

@catamorphism
Copy link
Contributor

Needs a rebase.

@luqmana
Copy link
Contributor Author

luqmana commented May 14, 2013

Rebased this but not sure if @nikomatsakis has anything as part of the new borrowck.

@luqmana
Copy link
Contributor Author

luqmana commented May 14, 2013

Closed in favour of #6473.

@luqmana luqmana closed this May 14, 2013
@luqmana luqmana deleted the mut-temp branch May 14, 2013 14:35
bors added a commit that referenced this pull request May 16, 2013
…y, r=pcwalton

Simpler version of PR #5974 based on new borrowck.
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