about summary refs log tree commit diff
path: root/src/libsync
diff options
context:
space:
mode:
authorNiko Matsakis <niko@alum.mit.edu>2014-08-27 14:53:23 -0400
committerNiko Matsakis <niko@alum.mit.edu>2014-08-27 15:06:44 -0400
commit5c82f484db29e37e26064e853d8a303cc13bc78c (patch)
treeb5262ac98c1f356484a9438d8d9f57e9de16c839 /src/libsync
parent18d6eefadbe03ca913cd3a049e900024c4c680d6 (diff)
downloadrust-5c82f484db29e37e26064e853d8a303cc13bc78c.tar.gz
rust-5c82f484db29e37e26064e853d8a303cc13bc78c.zip
Remove invalid test -- this test reads freed memory, from what I can tell
Diffstat (limited to 'src/libsync')
-rw-r--r--src/libsync/spsc_queue.rs21
1 files changed, 0 insertions, 21 deletions
diff --git a/src/libsync/spsc_queue.rs b/src/libsync/spsc_queue.rs
index 578e518cb8f..32b77be78a4 100644
--- a/src/libsync/spsc_queue.rs
+++ b/src/libsync/spsc_queue.rs
@@ -315,27 +315,6 @@ mod test {
         assert_eq!(consumer.pop(), None);
     }
 
-    // This behaviour is blocked by the type system if using the safe constructor
-    #[test]
-    fn pop_peeked_unchecked() {
-        let q = unsafe { Queue::new(0) };
-        q.push(vec![1i]);
-        q.push(vec![2]);
-        let peeked = q.peek().unwrap();
-
-        assert_eq!(*peeked, vec![1]);
-        assert_eq!(q.pop(), Some(vec![1]));
-
-        assert_eq!(*peeked, vec![1]);
-        q.push(vec![7]);
-
-        // Note: This should actually expect 1, but this test is to highlight
-        // the unsafety allowed by the unchecked usage. A Rust user would not
-        // expect their peeked value to mutate like this without the type system
-        // complaining.
-        assert_eq!(*peeked, vec![7]);
-    }
-
     #[test]
     fn peek() {
         let (mut consumer, mut producer) = queue(0);