diff options
| author | bors <bors@rust-lang.org> | 2015-10-21 02:39:59 +0000 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2015-10-21 02:39:59 +0000 |
| commit | fddf2f75747faf5e0e2fa9e10befe30120554e02 (patch) | |
| tree | ffc16d493a298dc851371210594a038f5ced2bcb /src | |
| parent | 4826f9625b589b9b0ec61791b040b51e131347b5 (diff) | |
| parent | 2b712c78cfb88960aeb1928d008291881d2c7341 (diff) | |
| download | rust-fddf2f75747faf5e0e2fa9e10befe30120554e02.tar.gz rust-fddf2f75747faf5e0e2fa9e10befe30120554e02.zip | |
Auto merge of #29186 - pnkfelix:fsk-fix-issue-29166, r=alexcrichton
Add dropck unsafe escape hatch (UGEH) to vec::IntoIter. Fix #29166
Diffstat (limited to 'src')
| -rw-r--r-- | src/libcollections/vec.rs | 1 | ||||
| -rw-r--r-- | src/test/run-pass/issue-29166.rs | 30 |
2 files changed, 31 insertions, 0 deletions
diff --git a/src/libcollections/vec.rs b/src/libcollections/vec.rs index d374c0959f3..40e1222c884 100644 --- a/src/libcollections/vec.rs +++ b/src/libcollections/vec.rs @@ -1577,6 +1577,7 @@ impl<T> ExactSizeIterator for IntoIter<T> {} #[stable(feature = "rust1", since = "1.0.0")] impl<T> Drop for IntoIter<T> { + #[unsafe_destructor_blind_to_params] fn drop(&mut self) { // destroy the remaining elements for _x in self {} diff --git a/src/test/run-pass/issue-29166.rs b/src/test/run-pass/issue-29166.rs new file mode 100644 index 00000000000..193f977ff99 --- /dev/null +++ b/src/test/run-pass/issue-29166.rs @@ -0,0 +1,30 @@ +// Copyright 2015 The Rust Project Developers. See the COPYRIGHT +// file at the top-level directory of this distribution and at +// http://rust-lang.org/COPYRIGHT. +// +// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or +// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license +// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your +// option. This file may not be copied, modified, or distributed +// except according to those terms. + +// This test ensures that vec.into_iter does not overconstrain element lifetime. + +pub fn main() { + original_report(); + revision_1(); + revision_2(); +} + +fn original_report() { + drop(vec![&()].into_iter()) +} + +fn revision_1() { + // below is what above `vec!` expands into at time of this writing. + drop(<[_]>::into_vec(::std::boxed::Box::new([&()])).into_iter()) +} + +fn revision_2() { + drop((match (Vec::new(), &()) { (mut v, b) => { v.push(b); v } }).into_iter()) +} |
