dart:io | Add FileSystemEntity.parent, make File.directory call it.

sdk/lib/io/file_system_entity.dart

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 part of dart.io;
 
 class FileSystemEntityType {
   static const FILE = const FileSystemEntityType._internal(0);
   static const DIRECTORY = const FileSystemEntityType._internal(1);
   static const LINK = const FileSystemEntityType._internal(2);
@@ skipping 512 matching lines @@
    * Synchronously checks if typeSync(path) returns
    * FileSystemEntityType.DIRECTORY.
    */
   static bool isDirectorySync(String path) =>
       (_getTypeSync(path, true) == FileSystemEntityType.DIRECTORY._type);
 
   external static _getType(String path, bool followLinks);
   external static _identical(String path1, String path2);
   external static _resolveSymbolicLinks(String path);
 
+  /**
+   * The directory containing [this].  If [this] is a root
+   * directory, returns [this].
+   */
+  Directory get parent {

[Søren Gjesse, 2013/10/04 12:20:16]

How about

return new Directory(new Uri.file(path).resolve('..').toFilePath());

[Bill Hesse, 2013/10/04 12:41:06]

This will not give the desired result, since it will remove all '..' and '.'
segments, and it will do different things depending on whether there are
trailing slashes.

+    if (Platform.isWindows) {
+      int rootEnd = -1;
+      if (path.startsWith(_absoluteWindowsPathPattern)) {
+        // Root ends at first / or \ after the first two characters.
+        rootEnd = path.indexOf(new RegExp(r'[/\\]'), 2);
+        if (rootEnd == -1) return new Directory(path);
+      }
+      // Ignore trailing slashes - find a slash followed by a non-slash.
+      int pos = path.lastIndexOf(new RegExp(r'[/\\][^/\\]'));
+      if (pos <= rootEnd) {
+        if (rootEnd == -1) {
+          return new Directory('.');
+        } else {
+          return new Directory(path.substring(0, rootEnd + 1));
+        }
+      }
+      while (pos > rootEnd + 1 &&
+             (path[pos - 1] == '\\' || path[pos - 1] == '/')) {
+        pos--;
+      }
+      return new Directory(path.substring(0, pos));
+    } else {
+      // Ignore trailing slashes - find a slash followed by a non-slash.
+      int pos = path.lastIndexOf(new RegExp(r'/[^/]'));
+      if (pos < 0) return new Directory('.');
+      while (pos > 0 && path[pos - 1] == '/') --pos;
+      if (pos == 0) return new Directory('/');
+      return new Directory(path.substring(0, pos));
+    }
+  }
+
   static int _getTypeSync(String path, bool followLinks) {
     var result = _getType(path, followLinks);
     _throwIfError(result, 'Error getting type of FileSystemEntity');
     return result;
   }
 
   static Future<int> _getTypeAsync(String path, bool followLinks) {
-    return _IOService.dispatch(_FILE_TYPE, [path, followLinks]).then((response) {
-      if (_isErrorResponse(response)) {
-        throw _exceptionFromResponse(response, "Error getting type", path);
-      }
-      return response;
-    });
+    return _IOService.dispatch(_FILE_TYPE, [path, followLinks])
+      .then((response) {
+        if (_isErrorResponse(response)) {
+          throw _exceptionFromResponse(response, "Error getting type", path);
+        }
+        return response;
+      });
   }
 
   static _throwIfError(Object result, String msg, [String path]) {
     if (result is OSError) {
       throw new FileException(msg, path, result);
     } else if (result is ArgumentError) {
       throw result;
     }
   }
 
@@ skipping 105 matching lines @@
   }
 }
 
 
 abstract class _FileSystemWatcher {
   external factory _FileSystemWatcher(String path, int events, bool recursive);
   external static bool get isSupported;
 
   Stream<FileSystemEvent> get stream;
 }