---

yaml
---
r: 11437
b: refs/heads/master
c: 2756a61
h: refs/heads/master
i:
  11435: d729c12
v: v3

Author: killerswan

[refs]

@@ -1,5 +1,5 @@
 ---
-refs/heads/master: e1d04e00621672a0e9f5424285f302077a973b37
+refs/heads/master: 2756a61e3442c378d10cf4e3bee3d35392623cec
 refs/heads/snap-stage1: e33de59e47c5076a89eadeb38f4934f58a3618a6
 refs/heads/snap-stage3: 4a81779abd786ff22d71434c6d9a5917ea4cdfff
 refs/heads/try: 2898dcc5d97da9427ac367542382b6239d9c0bbf

trunk/src/libcore/str.rs

@@ -72,8 +72,9 @@ export
    index_chars,
    byte_index,
    byte_index_from,
+   //rindex,
    rindex_chars,
-   find,
+   find_chars,
    find_bytes,
    find_from_bytes,
    contains,
@@ -861,7 +862,36 @@ Section: Searching
 
 // Function: index
 //
-// Returns the index of the first matching char
+// Returns the byte index of the first matching char
+// (as option some/none)
+fn index(ss: str, cc: char) -> option<uint> {
+    index_from(ss, cc, 0u, len_bytes(ss))
+}
+
+// Function: index_from
+//
+// Returns the byte index of the first matching char
+// (as option some/none), starting at `nn`
+fn index_from(ss: str, cc: char, start: uint, end: uint) -> option<uint> {
+    let bii = start;
+    while bii < end {
+        let {ch, next} = char_range_at(ss, bii);
+
+        // found here?
+        if ch == cc {
+            ret some(bii);
+        }
+
+        bii = next;
+    }
+
+    // wasn't found
+    ret none;
+}
+
+// Function: index_chars
+//
+// Returns the char index of the first matching char
 // (as option some/none)
 fn index_chars(ss: str, cc: char) -> option<uint> {
     let bii = 0u;
@@ -887,6 +917,7 @@ fn index_chars(ss: str, cc: char) -> option<uint> {
 //
 // Returns the index of the first matching byte
 // (as option some/none)
+// FIXME: delete
 fn byte_index(s: str, b: u8) -> option<uint> {
     byte_index_from(s, b, 0u, len_bytes(s))
 }
@@ -896,15 +927,36 @@ fn byte_index(s: str, b: u8) -> option<uint> {
 // Returns the index of the first matching byte within the range [`start`,
 // `end`).
 // (as option some/none)
+// FIXME: delete
 fn byte_index_from(s: str, b: u8, start: uint, end: uint) -> option<uint> {
     assert end <= len_bytes(s);
 
     str::as_bytes(s) { |v| vec::position_from(v, start, end) { |x| x == b } }
 }
 
+// Function: rindex
+//
+// Returns the byte index of the first matching char
+// (as option some/none)
+fn rindex(ss: str, cc: char) -> option<uint> {
+    let bii = len_bytes(ss);
+    while bii > 0u {
+        let {ch, prev} = char_range_at_reverse(ss, bii);
+        bii = prev;
+
+        // found here?
+        if ch == cc {
+            ret some(bii);
+        }
+    }
+
+    // wasn't found
+    ret none;
+}
+
 // Function: rindex_chars
 //
-// Returns the index of the first matching char
+// Returns the char index of the first matching char
 // (as option some/none)
 fn rindex_chars(ss: str, cc: char) -> option<uint> {
     let bii = len_bytes(ss);
@@ -926,15 +978,15 @@ fn rindex_chars(ss: str, cc: char) -> option<uint> {
 
 //Function: find_bytes
 //
-// Find the char position of the first instance of one string
+// Find the byte position of the first instance of one string
 // within another, or return option::none
 fn find_bytes(haystack: str, needle: str) -> option<uint> {
     find_from_bytes(haystack, needle, 0u, len_bytes(haystack))
 }
 
 //Function: find_from_bytes
 //
-// Find the char position of the first instance of one string
+// Find the byte position of the first instance of one string
 // within another, or return option::none
 //
 // FIXME: Boyer-Moore should be significantly faster
@@ -962,11 +1014,11 @@ fn find_from_bytes(haystack: str, needle: str, start: uint, end:uint)
     ret none;
 }
 
-// Function: find
+// Function: find_chars
 //
 // Find the char position of the first instance of one string
 // within another, or return option::none
-fn find(haystack: str, needle: str) -> option<uint> {
+fn find_chars(haystack: str, needle: str) -> option<uint> {
    alt find_bytes(haystack, needle) {
       none { ret none; }
       some(nn) { ret some(b2c_pos(haystack, nn)); }
@@ -1570,6 +1622,15 @@ mod tests {
         assert ( index_chars("hello", 'z') == none);
     }
 
+    #[test]
+    fn test_rindex() {
+        assert rindex("hello", 'l') == some(3u);
+        assert rindex("hello", 'o') == some(4u);
+        assert rindex("hello", 'h') == some(0u);
+        assert rindex("hello", 'z') == none;
+        assert rindex("ประเทศไทย中华Việt Nam", '华') == some(30u);
+    }
+
     #[test]
     fn test_rindex_chars() {
         assert (rindex_chars("hello", 'l') == some(3u));
@@ -1820,17 +1881,17 @@ mod tests {
     }
 
     #[test]
-    fn test_find() {
+    fn test_find_chars() {
         // char positions
-        assert (find("banana", "apple pie") == none);
-        assert (find("", "") == some(0u));
+        assert (find_chars("banana", "apple pie") == none);
+        assert (find_chars("", "") == some(0u));
 
         let data = "ประเทศไทย中华Việt Nam";
-        assert (find(data, "")     == some(0u));
-        assert (find(data, "ประเ") == some(0u));
-        assert (find(data, "ะเ")   == some(2u));
-        assert (find(data, "中华") == some(9u));
-        assert (find(data, "ไท华") == none);
+        assert (find_chars(data, "")     == some(0u));
+        assert (find_chars(data, "ประเ") == some(0u));
+        assert (find_chars(data, "ะเ")   == some(2u));
+        assert (find_chars(data, "中华") == some(9u));
+        assert (find_chars(data, "ไท华") == none);
     }
 
     #[test]