---

yaml
---
r: 2387
b: refs/heads/master
c: 50ac893
h: refs/heads/master
i:
  2385: 31c62ca
  2383: 2a470f2
v: v3

Author: graydon

[refs]

@@ -1,2 +1,2 @@
 ---
-refs/heads/master: 9a1f0977a227a14fa521f8e0efe5edcbf81eb597
+refs/heads/master: 50ac893b00853fa832a0f689b81104bd82e2a9b5

trunk/src/lib/fs.rs

@@ -9,14 +9,9 @@ fn path_sep() -> str {
 type path = str;
 
 fn dirname(path p) -> path {
-    auto sep = path_sep();
-    assert (_str.byte_len(sep) == 1u);
-    let int i = _str.rindex(p, sep.(0));
+    let int i = _str.rindex(p, os_fs.path_sep as u8);
     if (i == -1) {
-        // FIXME: the '/' character is a path separator on all 3 platforms we
-        // support. This should probably be generalized a bit more in the
-        // future, but for now this should work.
-        i = _str.rindex(p, '/' as u8);
+        i = _str.rindex(p, os_fs.alt_path_sep as u8);
         if (i == -1) {
             ret p;
         }

trunk/src/lib/posix_fs.rs

@@ -20,6 +20,7 @@ fn list_dir(str path) -> vec[str] {
 }
 
 const char path_sep = '/';
+const char alt_path_sep = '/';
 
 // Local Variables:
 // mode: rust;

trunk/src/lib/win32_fs.rs

@@ -14,6 +14,7 @@ fn list_dir(str path) -> vec[str] {
  * tag type.
  */
 const char path_sep = '/';
+const char alt_path_sep = '\\';
 
 // Local Variables:
 // mode: rust;