---

yaml
---
r: 145207
b: refs/heads/try2
c: e9acdd9
h: refs/heads/master
i:
  145205: 2084e1b
  145203: 1d6e9ed
  145199: b1c1654
v: v3

Author: olsonjeffery

[refs]

@@ -5,7 +5,7 @@ refs/heads/snap-stage3: 78a7676898d9f80ab540c6df5d4c9ce35bb50463
 refs/heads/try: 519addf6277dbafccbb4159db4b710c37eaa2ec5
 refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105
 refs/heads/ndm: f3868061cd7988080c30d6d5bf352a5a5fe2460b
-refs/heads/try2: 52840a5bbc7fa3bc4bf93dacc95d0db523812639
+refs/heads/try2: e9acdd93920f126d733d525d98234cd2df5706f1
 refs/heads/dist-snap: ba4081a5a8573875fed17545846f6f6902c8ba8d
 refs/tags/release-0.2: c870d2dffb391e14efb05aa27898f1f6333a9596
 refs/tags/release-0.3: b5f0d0f648d9a6153664837026ba1be43d3e2503

branches/try2/src/libstd/rt/io/file.rs

@@ -17,7 +17,7 @@ use rt::rtio::{RtioFileStream, IoFactory, IoFactoryObject};
 use rt::io::{io_error, read_error, EndOfFile,
             FileMode, FileAccess, FileStat, IoError,
             PathAlreadyExists, PathDoesntExist,
-            MismatchedFileTypeForOperation};
+            MismatchedFileTypeForOperation, ignore_io_error};
 use rt::local::Local;
 use option::{Some, None};
 use path::Path;
@@ -248,18 +248,6 @@ impl Seek for FileStream {
     }
 }
 
-// helper for grabbing a stat and ignoring any
-// error.. used in Info wrappers
-fn suppressed_stat(cb: &fn() -> Option<FileStat>) -> Option<FileStat> {
-    do io_error::cond.trap(|_| {
-        // just swallow the error.. downstream users
-        // who can make a decision based on a None result
-        // won't care
-    }).inside {
-        cb()
-    }
-}
-
 /// Shared functionality between `FileInfo` and `DirectoryInfo`
 pub trait FileSystemInfo {
     /// Get the filesystem path that this instance points at,
@@ -277,7 +265,7 @@ pub trait FileSystemInfo {
     /// returns `true` if the location pointed at by the enclosing
     /// exists on the filesystem
     fn exists(&self) -> bool {
-        match suppressed_stat(|| self.stat()) {
+        match ignore_io_error(|| self.stat()) {
             Some(_) => true,
             None => false
         }
@@ -306,7 +294,7 @@ pub trait FileInfo : FileSystemInfo {
     /// false for paths to non-existent locations or directories or
     /// other non-regular files (named pipes, etc).
     fn is_file(&self) -> bool {
-        match suppressed_stat(|| self.stat()) {
+        match ignore_io_error(|| self.stat()) {
             Some(s) => s.is_file,
             None => false
         }
@@ -315,7 +303,7 @@ pub trait FileInfo : FileSystemInfo {
     /// Attempts to open a regular file for reading/writing based
     /// on provided inputs
     fn open_stream(&self, mode: FileMode, access: FileAccess) -> Option<FileStream> {
-        match suppressed_stat(|| self.stat()) {
+        match ignore_io_error(|| self.stat()) {
             Some(s) => match s.is_file {
                 true => open(self.get_path(), mode, access),
                 false => None
@@ -364,7 +352,7 @@ trait DirectoryInfo : FileSystemInfo {
     /// false for paths to non-existent locations or if the item is
     /// not a directory (eg files, named pipes, links, etc)
     fn is_dir(&self) -> bool {
-        match suppressed_stat(|| self.stat()) {
+        match ignore_io_error(|| self.stat()) {
             Some(s) => s.is_dir,
             None => false
         }
@@ -375,7 +363,7 @@ trait DirectoryInfo : FileSystemInfo {
     /// at that location or if some other error occurs during
     /// the mkdir operation
     fn mkdir(&self) {
-        match suppressed_stat(|| self.stat()) {
+        match ignore_io_error(|| self.stat()) {
             Some(_) => {
                 io_error::cond.raise(IoError {
                     kind: PathAlreadyExists,
@@ -391,7 +379,7 @@ trait DirectoryInfo : FileSystemInfo {
     /// the type underlying the given `DirectoryInfo`. Will fail
     /// if there is no directory at the given location or if
     fn rmdir(&self) {
-        match suppressed_stat(|| self.stat()) {
+        match ignore_io_error(|| self.stat()) {
             Some(s) => {
                 match s.is_dir {
                     true => rmdir(self.get_path()),

branches/try2/src/libstd/rt/io/mod.rs

@@ -401,6 +401,18 @@ condition! {
     pub read_error: super::IoError -> ();
 }
 
+/// Helper for wrapper calls where you want to
+/// ignore any io_errors that might be raised
+pub fn ignore_io_error<T>(cb: &fn() -> T) -> T {
+    do io_error::cond.trap(|_| {
+        // just swallow the error.. downstream users
+        // who can make a decision based on a None result
+        // won't care
+    }).inside {
+        cb()
+    }
+}
+
 pub trait Reader {
     /// Read bytes, up to the length of `buf` and place them in `buf`.
     /// Returns the number of bytes read. The number of bytes read my