diff options
author | Reid Spencer <rspencer@reidspencer.com> | 2005-07-07 18:21:42 +0000 |
---|---|---|
committer | Reid Spencer <rspencer@reidspencer.com> | 2005-07-07 18:21:42 +0000 |
commit | c7f083297cd796c2fadb534307e4ef406fd68945 (patch) | |
tree | 9b2229594545a12585fe9e49e955595c4f5c5725 /lib/System/Unix | |
parent | 8d4b9eddc00ad2c1c8f5bada32750c72f06c1517 (diff) | |
download | llvm-c7f083297cd796c2fadb534307e4ef406fd68945.tar.gz llvm-c7f083297cd796c2fadb534307e4ef406fd68945.tar.bz2 llvm-c7f083297cd796c2fadb534307e4ef406fd68945.tar.xz |
For PR495:
Change interface to Path class:
readable -> canRead
writable -> canWrite
executable -> canExecute
More (incremental) changes coming to close 495.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@22345 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/System/Unix')
-rw-r--r-- | lib/System/Unix/Path.inc | 12 | ||||
-rw-r--r-- | lib/System/Unix/Program.inc | 6 |
2 files changed, 9 insertions, 9 deletions
diff --git a/lib/System/Unix/Path.inc b/lib/System/Unix/Path.inc index b371d00e69..f871adb2c7 100644 --- a/lib/System/Unix/Path.inc +++ b/lib/System/Unix/Path.inc @@ -168,14 +168,14 @@ static void getPathList(const char*path, std::vector<sys::Path>& Paths) { while( delim != 0 ) { std::string tmp(at, size_t(delim-at)); if (tmpPath.setDirectory(tmp)) - if (tmpPath.readable()) + if (tmpPath.canRead()) Paths.push_back(tmpPath); at = delim + 1; delim = strchr(at, ':'); } if (*at != 0) if (tmpPath.setDirectory(std::string(at))) - if (tmpPath.readable()) + if (tmpPath.canRead()) Paths.push_back(tmpPath); } @@ -205,7 +205,7 @@ Path::GetBytecodeLibraryPaths(std::vector<sys::Path>& Paths) { { Path tmpPath; if (tmpPath.setDirectory(LLVM_LIBDIR)) - if (tmpPath.readable()) + if (tmpPath.canRead()) Paths.push_back(tmpPath); } #endif @@ -305,17 +305,17 @@ Path::exists() const { } bool -Path::readable() const { +Path::canRead() const { return 0 == access(path.c_str(), F_OK | R_OK ); } bool -Path::writable() const { +Path::canWrite() const { return 0 == access(path.c_str(), F_OK | W_OK ); } bool -Path::executable() const { +Path::canExecute() const { struct stat st; int r = stat(path.c_str(), &st); if (r != 0 || !S_ISREG(st.st_mode)) diff --git a/lib/System/Unix/Program.inc b/lib/System/Unix/Program.inc index de5ae7f197..774bd02e88 100644 --- a/lib/System/Unix/Program.inc +++ b/lib/System/Unix/Program.inc @@ -46,7 +46,7 @@ Program::FindProgramByName(const std::string& progName) { return Path(); // FIXME: have to check for absolute filename - we cannot assume anything // about "." being in $PATH - if (temp.executable()) // already executable as is + if (temp.canExecute()) // already executable as is return temp; // At this point, the file name is valid and its not executable @@ -66,7 +66,7 @@ Program::FindProgramByName(const std::string& progName) { Path FilePath; if (FilePath.setDirectory(std::string(PathStr,Colon))) { FilePath.appendFile(progName); - if (FilePath.executable()) + if (FilePath.canExecute()) return FilePath; // Found the executable! } @@ -109,7 +109,7 @@ Program::ExecuteAndWait(const Path& path, const Path** redirects, unsigned secondsToWait ) { - if (!path.executable()) + if (!path.canExecute()) throw path.toString() + " is not executable"; #ifdef HAVE_SYS_WAIT_H |