From 380d23b5d4b9fb8f5f0ebf178590f61528b2483e Mon Sep 17 00:00:00 2001 From: Huon Wilson Date: Fri, 20 Feb 2015 17:33:11 +1100 Subject: Remove `'static` bound from sync::mpsc, Mutex and RwLock. Adds some basic tests to check that the types still catch the most glaring errors that could occur. cc #22444. --- .../compile-fail/send-is-not-static-std-sync-2.rs | 46 ++++++++++++++++++ .../compile-fail/send-is-not-static-std-sync.rs | 56 ++++++++++++++++++++++ 2 files changed, 102 insertions(+) create mode 100644 src/test/compile-fail/send-is-not-static-std-sync-2.rs create mode 100644 src/test/compile-fail/send-is-not-static-std-sync.rs (limited to 'src/test') diff --git a/src/test/compile-fail/send-is-not-static-std-sync-2.rs b/src/test/compile-fail/send-is-not-static-std-sync-2.rs new file mode 100644 index 00000000000..d9d3706586b --- /dev/null +++ b/src/test/compile-fail/send-is-not-static-std-sync-2.rs @@ -0,0 +1,46 @@ +// 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 or the MIT license +// , at your +// option. This file may not be copied, modified, or distributed +// except according to those terms. + +// basic tests to see that certain "obvious" errors are caught by +// these types no longer requiring `'static` (RFC 458) + +#![allow(dead_code)] + +use std::sync::{Mutex, RwLock, mpsc}; + +fn mutex() { + let lock = { + let x = 1; + Mutex::new(&x) //~ ERROR does not live long enough + }; + + let _dangling = *lock.lock().unwrap(); +} + +fn rwlock() { + let lock = { + let x = 1; + RwLock::new(&x) //~ ERROR does not live long enough + }; + let _dangling = *lock.read().unwrap(); +} + +fn channel() { + let (_tx, rx) = { + let x = 1; + let (tx, rx) = mpsc::channel(); + let _ = tx.send(&x); //~ ERROR does not live long enough + (tx, rx) + }; + + let _dangling = rx.recv(); +} + +fn main() {} diff --git a/src/test/compile-fail/send-is-not-static-std-sync.rs b/src/test/compile-fail/send-is-not-static-std-sync.rs new file mode 100644 index 00000000000..8ec2fe8a1ec --- /dev/null +++ b/src/test/compile-fail/send-is-not-static-std-sync.rs @@ -0,0 +1,56 @@ +// 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 or the MIT license +// , at your +// option. This file may not be copied, modified, or distributed +// except according to those terms. + +// basic tests to see that certain "obvious" errors are caught by +// these types no longer requiring `'static` (RFC 458) + +#![allow(dead_code)] + +use std::sync::{Mutex, RwLock, mpsc}; + +fn mutex() { + let x = 1; + let y = Box::new(1); + let lock = Mutex::new(&x); + *lock.lock().unwrap() = &*y; + drop(y); //~ ERROR cannot move out + { + let z = 2; + *lock.lock().unwrap() = &z; //~ ERROR does not live long enough + } +} + +fn rwlock() { + let x = 1; + let y = Box::new(1); + let lock = RwLock::new(&x); + *lock.write().unwrap() = &*y; + drop(y); //~ ERROR cannot move out + { + let z = 2; + *lock.write().unwrap() = &z; //~ ERROR does not live long enough + } +} + +fn channel() { + let x = 1; + let y = Box::new(1); + let (tx, rx) = mpsc::channel(); + + tx.send(&x).unwrap(); + tx.send(&*y); + drop(y); //~ ERROR cannot move out + { + let z = 2; + tx.send(&z).unwrap(); //~ ERROR does not live long enough + } +} + +fn main() {} -- cgit 1.4.1-3-g733a5