about summary refs log tree commit diff
path: root/src/libstd/vec.rs
diff options
context:
space:
mode:
authorBrian Anderson <banderson@mozilla.com>2013-09-05 15:55:59 -0700
committerBrian Anderson <banderson@mozilla.com>2013-09-05 15:56:22 -0700
commitfec6716eec69cf0db650d5b86e51a32a530ae57b (patch)
treeb6ed836a1b77fff18d224b3b3ad75e043786d9aa /src/libstd/vec.rs
parentd1dde99e4b3d43d044cdead7240bbd5f0d7a0ce5 (diff)
downloadrust-fec6716eec69cf0db650d5b86e51a32a530ae57b.tar.gz
rust-fec6716eec69cf0db650d5b86e51a32a530ae57b.zip
std: Remove push_fast from OwnedVector. Closes #8769
This is an unsafe implementation detail of `push`.
Diffstat (limited to 'src/libstd/vec.rs')
-rw-r--r--src/libstd/vec.rs43
1 files changed, 21 insertions, 22 deletions
diff --git a/src/libstd/vec.rs b/src/libstd/vec.rs
index f607d1612ad..4cc5c4f14ff 100644
--- a/src/libstd/vec.rs
+++ b/src/libstd/vec.rs
@@ -1201,8 +1201,6 @@ pub trait OwnedVector<T> {
     fn shrink_to_fit(&mut self);
 
     fn push(&mut self, t: T);
-    unsafe fn push_fast(&mut self, t: T);
-
     fn push_all_move(&mut self, rhs: ~[T]);
     fn pop(&mut self) -> T;
     fn pop_opt(&mut self) -> Option<T>;
@@ -1334,7 +1332,7 @@ impl<T> OwnedVector<T> for ~[T] {
                     self.reserve_at_least(new_len);
                 }
 
-                self.push_fast(t);
+                push_fast(self, t);
             } else {
                 let repr: **Vec<()> = cast::transmute(&mut *self);
                 let fill = (**repr).fill;
@@ -1343,29 +1341,30 @@ impl<T> OwnedVector<T> for ~[T] {
                     self.reserve_at_least(new_len);
                 }
 
-                self.push_fast(t);
+                push_fast(self, t);
             }
         }
-    }
 
-    // This doesn't bother to make sure we have space.
-    #[inline] // really pretty please
-    unsafe fn push_fast(&mut self, t: T) {
-        if contains_managed::<T>() {
-            let repr: **mut Box<Vec<u8>> = cast::transmute(self);
-            let fill = (**repr).data.fill;
-            (**repr).data.fill += sys::nonzero_size_of::<T>();
-            let p = to_unsafe_ptr(&((**repr).data.data));
-            let p = ptr::offset(p, fill as int) as *mut T;
-            intrinsics::move_val_init(&mut(*p), t);
-        } else {
-            let repr: **mut Vec<u8> = cast::transmute(self);
-            let fill = (**repr).fill;
-            (**repr).fill += sys::nonzero_size_of::<T>();
-            let p = to_unsafe_ptr(&((**repr).data));
-            let p = ptr::offset(p, fill as int) as *mut T;
-            intrinsics::move_val_init(&mut(*p), t);
+        // This doesn't bother to make sure we have space.
+        #[inline] // really pretty please
+        unsafe fn push_fast<T>(this: &mut ~[T], t: T) {
+            if contains_managed::<T>() {
+                let repr: **mut Box<Vec<u8>> = cast::transmute(this);
+                let fill = (**repr).data.fill;
+                (**repr).data.fill += sys::nonzero_size_of::<T>();
+                let p = to_unsafe_ptr(&((**repr).data.data));
+                let p = ptr::offset(p, fill as int) as *mut T;
+                intrinsics::move_val_init(&mut(*p), t);
+            } else {
+                let repr: **mut Vec<u8> = cast::transmute(this);
+                let fill = (**repr).fill;
+                (**repr).fill += sys::nonzero_size_of::<T>();
+                let p = to_unsafe_ptr(&((**repr).data));
+                let p = ptr::offset(p, fill as int) as *mut T;
+                intrinsics::move_val_init(&mut(*p), t);
+            }
         }
+
     }
 
     /// Takes ownership of the vector `rhs`, moving all elements into