Clean up directory listing interface.

runtime/bin/directory.dart

 // Copyright (c) 2011, 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.
 
 interface Directory factory _Directory {
   /**
    * Creates a directory object. The path is either a full path or
    * relative to the directory in which the Dart VM was
-   * started. Throws an exception if the path does not specify a
-   * directory.
+   * started.
    */
-  Directory.open(String dir);
-
-  /**
-   * Close this [Directory]. Terminates listing operation if one is in
-   * progress. Returns a boolean indicating whether the close operation
-   * succeeded.
-   */
-  bool close();
+  Directory(String dir);
 
   /**
    * List the sub-directories and files of this
    * [Directory]. Optionally recurse into sub-directories. For each
    * file and directory, the file or directory handler is called. When
    * all directories have been listed the done handler is called. If
-   * the listing operation is recursive, the directory error handler
-   * is called if a subdirectory cannot be opened for listing.
+   * the listing operation is recursive, the error handler is called
+   * if a subdirectory cannot be opened for listing.
    */
   void list([bool recursive]);
 
   /**
    * Sets the directory handler that is called for all directories
    * during listing operations. The directory handler is called with
    * the full path of the directory.
    */
   void setDirHandler(void dirHandler(String dir));
 
   /**
-   * Sets the file handler that is called for all directories during
-   * listing operations. The directory handler is called with the full
-   * path of the file.
+   * Sets the file handler that is called for all files during listing
+   * operations. The file handler is called with the full path of the
+   * file.
    */
   void setFileHandler(void fileHandler(String file));
 
   /**
-   * Set the done handler that is called either when a directory
-   * listing is completed or when the close method is called.
+   * Set the done handler that is called when a directory listing is
+   * done. The handler is called with an indication of whether or not
+   * the listing operation completed.

[Søren Gjesse, 2011/10/12 09:30:30]

What does "not completed" actually mean? Will the first error trigger done? Will
recurse continue even if some sub-directories cannot be opened?

[Mads Ager (google), 2011/10/12 10:36:28]

Good question. :)

Currently, done means that no errors that could effect the list of files and
directories occurred. Therefore, the listing is done and it completed (so you
got all the callbacks for everything that was encountered). If, for instance, we
found a directory that could not be opened, we don't list the contents of that
directory and completed will be false.

    */
   void setDoneHandler(void doneHandler(bool completed));
 
   /**
-   * Set the directory error handler that is called for all
-   * directories that cannot be opened for listing during recursive
-   * listing.
+   * Sets the error handler that is called on errors listing
+   * directories.
    */
-  void setDirErrorHandler(void errorHandler(String dir));
+  void setErrorHandler(void errorHandler(String error));
 }