diff options
author | Max Horn | 2006-04-26 08:29:32 +0000 |
---|---|---|
committer | Max Horn | 2006-04-26 08:29:32 +0000 |
commit | 8bb9ae92ad0721adca944d8a8197f3e5d3944bd6 (patch) | |
tree | 290f15bb4d1cc00d7ae9ef20ed1ffb10529cb0f8 /common | |
parent | f42bb14ea3f0c4e4801a026d6ff9cd881d0999df (diff) | |
download | scummvm-rg350-8bb9ae92ad0721adca944d8a8197f3e5d3944bd6.tar.gz scummvm-rg350-8bb9ae92ad0721adca944d8a8197f3e5d3944bd6.tar.bz2 scummvm-rg350-8bb9ae92ad0721adca944d8a8197f3e5d3944bd6.zip |
Yet another revision of File::exists. I now believe the function really is 'wrong' right now (it has to fulfill too many roles right now). Need to correctly fix this later
svn-id: r22171
Diffstat (limited to 'common')
-rw-r--r-- | common/file.cpp | 28 |
1 files changed, 27 insertions, 1 deletions
diff --git a/common/file.cpp b/common/file.cpp index 90acdb45d3..267b993276 100644 --- a/common/file.cpp +++ b/common/file.cpp @@ -263,8 +263,34 @@ bool File::open(const String &filename, AccessMode mode, const char *directory) } bool File::exists(const String &filename) { + // First try to find the file it via a FilesystemNode (in case an absolute + // path was passed). But we only use this to filter out directories. FilesystemNode file(filename); - return (file.isValid() && !file.isDirectory()); + if (file.isValid() && file.isDirectory()) + return false; + + // Next, try to locate the file by *opening* it in read mode. This has + // multiple effects: + // 1) It takes _filesMap and _defaultDirectories into consideration -> good + // 2) It returns true if and only if File::open is possible on the file -> good + // 3) If this method is misused, it could lead to an fopen call on a directory + // -> bad! + // 4) It also checks whether we can read the file. This is not 100% + // desirable; after all, even when we can't read it, the file is present. + // Since this method is often used to check whether a file should be + // re-created, that's not nice. + // + // TODO/FIXME: We should clarify the semantics of this method, and then + // maybe should introduce several new methods: + // fileExistsAndReadable + // fileExists + // fileExistsAtPath + // dirExists + // dirExistsAtPath + // or maybe only 1-2 methods which take some params :-). + + File tmp; + return tmp.open(filename, kFileReadMode); } void File::close() { |