about summary refs log tree commit diff
diff options
context:
space:
mode:
authorBrian Anderson <banderson@mozilla.com>2011-10-28 13:56:01 -0700
committerBrian Anderson <banderson@mozilla.com>2011-10-28 13:56:01 -0700
commitad66d72e6c0f77d7c6a094cf3fc5799b65824c87 (patch)
treec7f79bfeec0948b4fa25e379ccc48ace8dd26f93
parent39b729e36f813a58de6b1f5641c94f47b7a6c277 (diff)
stdlib: Make merge_sort take [mutable? T]
-rw-r--r--src/lib/sort.rs5
-rw-r--r--src/test/stdtest/sort.rs8
2 files changed, 11 insertions, 2 deletions
diff --git a/src/lib/sort.rs b/src/lib/sort.rs
index 6d4ead4c429..a4fe4cf3f8b 100644
--- a/src/lib/sort.rs
+++ b/src/lib/sort.rs
@@ -20,7 +20,7 @@ Merge sort. Returns a new vector containing the sorted list.
 Has worst case O(n log n) performance, best case O(n), but
 is not space efficient. This is a stable sort.
 */
-fn merge_sort<T>(le: lteq<T>, v: [T]) -> [T] {
+fn merge_sort<T>(le: lteq<T>, v: [mutable? T]) -> [T] {
     fn merge<T>(le: lteq<T>, a: [T], b: [T]) -> [T] {
         let rs: [T] = [];
         let a_len: uint = len::<T>(a);
@@ -38,7 +38,8 @@ fn merge_sort<T>(le: lteq<T>, v: [T]) -> [T] {
         ret rs;
     }
     let v_len: uint = len::<T>(v);
-    if v_len <= 1u { ret v; }
+    if v_len == 0u { ret []; }
+    if v_len == 1u { ret [v[0]]; }
     let mid: uint = v_len / 2u;
     let a: [T] = slice::<T>(v, 0u, mid);
     let b: [T] = slice::<T>(v, mid, v_len);
diff --git a/src/test/stdtest/sort.rs b/src/test/stdtest/sort.rs
index ff86042e6b1..9e362f0c988 100644
--- a/src/test/stdtest/sort.rs
+++ b/src/test/stdtest/sort.rs
@@ -26,3 +26,11 @@ fn test() {
         check_sort(v1, v2);
     }
 }
+
+#[test]
+fn test_merge_sort_mutable() {
+    fn lteq(&&a: int, &&b: int) -> bool { ret a <= b; }
+    let v1 = [mutable 3, 2, 1];
+    let v2 = std::sort::merge_sort(lteq, v1);
+    assert v2 == [1, 2, 3];
+}
\ No newline at end of file