Add support for getting OS error information for creating temporary directories

runtime/bin/directory_impl.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.
 
 
+class _OSStatus {
+  int _errorCode;  // Set to OS error code if process start failed.

[Bill Hesse, 2011/12/13 13:51:03]

_OSStatus is no longer just used for process starting.

[Søren Gjesse, 2011/12/13 14:07:40]

This creates _OSStatus and keeps _ProcessStartStatus. I will consolidate in a
separate change.

+  String _errorMessage;  // Set to OS error message if process start failed.
+}
+
+
 class _DirectoryListingIsolate extends Isolate {
 
   _DirectoryListingIsolate() : super.heavy();
 
   void main() {
     port.receive((message, replyTo) {
       bool started = _list(message['dir'],
                            message['recursive'],
                            message['dirPort'],
                            message['filePort'],
@@ skipping 13 matching lines @@
 }
 
 
 class _DirectoryCreateTempIsolate extends Isolate {
 
   _DirectoryCreateTempIsolate() : super.heavy();
 
   void main() {
     port.receive((path, replyTo) {
       // Call function to get file name
-      replyTo.send(_Directory._createTemp(path, (Math.random() * 0x8000000).toInt()));
+      var status = new _OSStatus();
+      var result = _Directory._createTemp(path,
+                                          (Math.random() * 0x8000000).toInt(),
+                                          status);
+      if (result == null) {
+        replyTo.send(status);

[Bill Hesse, 2011/12/13 13:51:03]

Can we send arbitrary objects between isolates now?  I thought it was just
values, lists, and maps.

[Søren Gjesse, 2011/12/13 14:07:40]

I surely hope so. The tests work.

+      } else {
+        replyTo.send(result);
+      }
       port.close();
     });
   }
 }
 
 
 class _Directory implements Directory {
 
   _Directory(String this._path);
 
-  static String _createTemp(String template, int num) native "Directory_CreateTemp";
+  static String _createTemp(String template,
+                            int num,
+                            _OSStatus status) native "Directory_CreateTemp";
 
   bool existsSync() {
     int exists = _exists(_path);
     if (exists < 0) {
       throw new DirectoryException("Diretory exists test failed: $_path");
     }
     return (exists == 1);
   }
 
   void createSync() {
     if (!_create(_path)) {
       throw new DirectoryException("Directory creation failed: $_path");
     }
   }
 
   void createTemp() {
     new _DirectoryCreateTempIsolate().spawn().then((port) {
       port.call(_path).receive((result, ignored) {
-        if (result != '') {
+        if (result is !_OSStatus) {
           _path = result;
           if (_createTempHandler !== null) {
             _createTempHandler();
           }
         } else {
           if (_errorHandler !== null) {
-            _errorHandler("Could not create temporary directory: $_path");
+            _errorHandler("Could not create temporary directory: " +
+                          "${result._errorMessage}");
           }
         }
       });
     });
   }
 
   void createTempSync() {
-    var result = _createTemp(path, (Math.random() * 0x8000000).toInt());
-    if (result != '') {
+    var status = new _OSStatus();
+    var result = _createTemp(path, (Math.random() * 0x8000000).toInt(), status);
+    if (result != null) {
       _path = result;
     } else {
-      throw "createTempSync failed";
+      throw new DirectoryException(status._errorMessage, status._errorCode);
     }
   }
 
   void deleteSync() {
     if (!_delete(_path)) {
       throw new DirectoryException("Directory deletion failed: $_path");
     }
   }
 
   void list([bool recursive = false]) {
@@ skipping 93 matching lines @@
   bool _delete(String path) native "Directory_Delete";
 
   var _dirHandler;
   var _fileHandler;
   var _doneHandler;
   var _createTempHandler;
   var _errorHandler;
 
   String _path;
 }