auto: Make ~fn non-copyable, make &fn copyable, split barefn/closure types #4810

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
Contributor

nikomatsakis commented Feb 6, 2013

Part of #3678. Fixes #2828, #3904.

r? @brson

Contributor

brson commented Feb 6, 2013

So many improvements in one patch. This is going to make actor-oriented code much less error prone.

I'm sad to see managed boxes in rustdoc, but rustdoc is going to take some effort to rehab anyway.

Contributor

nikomatsakis commented Feb 6, 2013

Yeah I was sad to add the managed boxes but I couldn't find another way to adapt the code without rewriting it more completely.

@nikomatsakis nikomatsakis Make ~fn non-copyable, make &fn copyable, split barefn/closure types,
correct handling of moves for struct-record update.

Part of #3678.  Fixes #2828, #3904, #4719.
a32498d
Owner

nikomatsakis commented on a32498d Feb 7, 2013

r+ r=brson

Contributor

bors replied Feb 7, 2013

saw approval from nikomatsakis
at nikomatsakis/rust@a32498d

Contributor

bors replied Feb 7, 2013

merging nikomatsakis/rust/owned-fn-noncopyable = a32498d into auto

Contributor

bors replied Feb 7, 2013

nikomatsakis/rust/owned-fn-noncopyable = a32498d merged ok, testing candidate = 3764cfb

Contributor

bors replied Feb 7, 2013

fast-forwarding incoming to auto = 3764cfb

@bors bors added a commit that referenced this pull request Feb 7, 2013

@bors bors auto merge of #4810 : nikomatsakis/rust/owned-fn-noncopyable, r=nikom…
…atsakis

Part of #3678.  Fixes #2828, #3904.

r? @brson
3764cfb

bors closed this Feb 7, 2013

nikomatsakis deleted the nikomatsakis:owned-fn-noncopyable branch Mar 30, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment