Skip to content

Commit

Permalink
Update test to cover #18769
Browse files Browse the repository at this point in the history
  • Loading branch information
bkoropoff committed Nov 9, 2014
1 parent fb4e618 commit c0a7d55
Showing 1 changed file with 11 additions and 1 deletion.
12 changes: 11 additions & 1 deletion src/test/run-pass/unboxed-closures-move-mutable.rs
Expand Up @@ -14,7 +14,9 @@
// Test that mutating a mutable upvar in a capture-by-value unboxed
// closure does not ice (issue #18238) and marks the upvar as used
// mutably so we do not get a spurious warning about it not needing to
// be declared mutable (issue #18336).
// be declared mutable (issue #18336 and #18769)

fn set(x: &mut uint) { *x = 42; }

fn main() {
{
Expand All @@ -25,4 +27,12 @@ fn main() {
let mut x = 0u;
move |:| x += 1;
}
{
let mut x = 0u;
move |&mut:| set(&mut x);
}
{
let mut x = 0u;
move |:| set(&mut x);
}
}

5 comments on commit c0a7d55

@bors
Copy link
Contributor

@bors bors commented on c0a7d55 Nov 10, 2014

Choose a reason for hiding this comment

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

saw approval from luqmana
at bkoropoff@c0a7d55

@bors
Copy link
Contributor

@bors bors commented on c0a7d55 Nov 10, 2014

Choose a reason for hiding this comment

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

merging bkoropoff/rust/issue-18769 = c0a7d55 into auto

@bors
Copy link
Contributor

@bors bors commented on c0a7d55 Nov 10, 2014

Choose a reason for hiding this comment

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

bkoropoff/rust/issue-18769 = c0a7d55 merged ok, testing candidate = a30b72b

@bors
Copy link
Contributor

@bors bors commented on c0a7d55 Nov 10, 2014

Choose a reason for hiding this comment

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

@bors
Copy link
Contributor

@bors bors commented on c0a7d55 Nov 10, 2014

Choose a reason for hiding this comment

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

fast-forwarding master to auto = a30b72b

Please sign in to comment.