diff options
author | Johannes Schickel | 2007-09-18 23:15:53 +0000 |
---|---|---|
committer | Johannes Schickel | 2007-09-18 23:15:53 +0000 |
commit | d5a2010197df1017e49f1b52072d440b8c02c67b (patch) | |
tree | 580cb633930c20232ac9569fa7960b0b8da43c1d | |
parent | f1091ec8bb7b0f1a9892ade6c04fa445b6d52391 (diff) | |
download | scummvm-rg350-d5a2010197df1017e49f1b52072d440b8c02c67b.tar.gz scummvm-rg350-d5a2010197df1017e49f1b52072d440b8c02c67b.tar.bz2 scummvm-rg350-d5a2010197df1017e49f1b52072d440b8c02c67b.zip |
Clarified the FIXME in Common::File::exists a bit.
svn-id: r28954
-rw-r--r-- | common/file.cpp | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/common/file.cpp b/common/file.cpp index 4d6fd5a69f..3f475e3792 100644 --- a/common/file.cpp +++ b/common/file.cpp @@ -446,9 +446,13 @@ bool File::exists(const String &filename) { // FIXME: since (as stated in the comment above) FilesystemNode creation // just works for absolute paths, and we use this to tell if a file - // exsists in any of the setup paths we can't use: + // exsists in any of the setup paths so we ca not use: //return (!file.isDirectory() && file.exists()); - // if file.exsits() fails + // + // I.e.: + // FilesystemNode("foofile"); would fail for most (even all?) + // implementations where the file 'foofile' does not exist in the CWD, + // so we can not rely on FilesystemNode::exists, which would return false. if (file.exists()) return !file.isDirectory(); |