---

yaml
---
r: 15458
b: refs/heads/try
c: a1b305c
h: refs/heads/master
v: v3

Author: catamorphism

[refs]

@@ -2,5 +2,5 @@
 refs/heads/master: 61b1875c16de39c166b0f4d54bba19f9c6777d1a
 refs/heads/snap-stage1: e33de59e47c5076a89eadeb38f4934f58a3618a6
 refs/heads/snap-stage3: 4a81779abd786ff22d71434c6d9a5917ea4cdfff
-refs/heads/try: ea0063788bd3de6eeccdf575c3f6cc95d7a5d2c8
+refs/heads/try: a1b305c8e526700e3e16e51d576691ac4ebdc335
 refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105

branches/try/src/libcore/str.rs

@@ -445,7 +445,7 @@ fn split_inner(s: str, sepfn: fn(cc: char) -> bool, count: uint,
     result
 }
 
-// FIXME use Boyer-Moore
+// See Issue #1932 for why this is a naive search
 fn iter_matches(s: str, sep: str, f: fn(uint, uint)) {
     let sep_len = len(sep), l = len(s);
     assert sep_len > 0u;
@@ -581,7 +581,7 @@ pure fn le(&&a: str, &&b: str) -> bool { a <= b }
 #[doc = "String hash function"]
 fn hash(&&s: str) -> uint {
     // djb hash.
-    // FIXME: replace with murmur.
+    // FIXME: replace with murmur. (see #859 and #1616)
     let mut u: uint = 5381u;
     for each(s) {|c| u *= 33u; u += c as uint; }
     ret u;
@@ -1072,7 +1072,7 @@ or equal to `len(s)`.
 "]
 fn find_str_between(haystack: str, needle: str, start: uint, end:uint)
   -> option<uint> {
-    // FIXME: Boyer-Moore should be significantly faster
+    // See Issue #1932 for why this is a naive search
     assert end <= len(haystack);
     let needle_len = len(needle);
     if needle_len == 0u { ret some(start); }
@@ -1600,16 +1600,11 @@ fn capacity(&&s: str) -> uint unsafe {
 #[doc = "Unsafe operations"]
 mod unsafe {
    export
-      // FIXME: stop exporting several of these
       from_buf,
-      from_buf_len,
       from_c_str,
-      from_c_str_len,
       from_bytes,
-      from_byte,
       slice_bytes,
       push_byte,
-      push_bytes,
       pop_byte,
       shift_byte,
       set_len;
@@ -1740,6 +1735,15 @@ mod unsafe {
         let null = ptr::mut_offset(ptr::mut_addr_of((*repr).data), new_len);
         *null = 0u8;
     }
+
+    #[test]
+    fn test_from_buf_len() unsafe {
+        let a = [65u8, 65u8, 65u8, 65u8, 65u8, 65u8, 65u8, 0u8];
+        let b = vec::unsafe::to_ptr(a);
+        let c = from_buf_len(b, 3u);
+        assert (c == "AAA");
+    }
+
 }
 
 #[doc = "Extension methods for strings"]
@@ -2439,14 +2443,6 @@ mod tests {
         assert (c == "AAAAAAA");
     }
 
-    #[test]
-    fn test_from_buf_len() unsafe {
-        let a = [65u8, 65u8, 65u8, 65u8, 65u8, 65u8, 65u8, 0u8];
-        let b = vec::unsafe::to_ptr(a);
-        let c = unsafe::from_buf_len(b, 3u);
-        assert (c == "AAA");
-    }
-
     #[test]
     #[ignore(cfg(target_os = "win32"))]
     #[should_fail]