diff options
author | Chris Lattner <sabre@nondot.org> | 2003-12-30 02:45:16 +0000 |
---|---|---|
committer | Chris Lattner <sabre@nondot.org> | 2003-12-30 02:45:16 +0000 |
commit | 9899ae3ae85e0d5cdb1a2fad0f527e5145da7f8c (patch) | |
tree | 95ec95056b33d26997b1e8e7f25c9801f0dbf499 /include | |
parent | 6dea527b4ae8bde56814b10cb96aa9cba5493c2e (diff) | |
download | external_llvm-9899ae3ae85e0d5cdb1a2fad0f527e5145da7f8c.zip external_llvm-9899ae3ae85e0d5cdb1a2fad0f527e5145da7f8c.tar.gz external_llvm-9899ae3ae85e0d5cdb1a2fad0f527e5145da7f8c.tar.bz2 |
Further revisions of the FDHandle idea. In this version we use ownership
semantics that are the same as those used by std::auto_ptr. This allows
copying of FDHandle's, but copying transfers ownership.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@10646 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'include')
-rw-r--r-- | include/Support/FileUtilities.h | 27 | ||||
-rw-r--r-- | include/llvm/Support/FileUtilities.h | 27 |
2 files changed, 44 insertions, 10 deletions
diff --git a/include/Support/FileUtilities.h b/include/Support/FileUtilities.h index 978b61b..b17bca9 100644 --- a/include/Support/FileUtilities.h +++ b/include/Support/FileUtilities.h @@ -104,24 +104,41 @@ bool MakeFileReadable (const std::string & Filename); /// FDHandle - Simple handle class to make sure a file descriptor gets closed -/// when the object is destroyed. +/// when the object is destroyed. This handle acts similarly to an +/// std::auto_ptr, in that the copy constructor and assignment operators +/// transfer ownership of the handle. This means that FDHandle's do not have +/// value semantics. /// class FDHandle { int FD; - FDHandle(const FDHandle &); // DO NOT IMPLEMENT - void operator=(const FDHandle&); // DO NOT IMPLEMENT public: FDHandle() : FD(-1) {} FDHandle(int fd) : FD(fd) {} + FDHandle(FDHandle &RHS) : FD(RHS.FD) { + RHS.FD = -1; // Transfer ownership + } + ~FDHandle() throw(); + /// get - Get the current file descriptor, without releasing ownership of it. + int get() const { return FD; } operator int() const { return FD; } FDHandle &operator=(int fd) throw(); - /// take - Take ownership of the file descriptor away from the FDHandle + FDHandle &operator=(FDHandle &RHS) { + int fd = RHS.FD; + RHS.FD = -1; // Transfer ownership + return operator=(fd); + } + + /// release - Take ownership of the file descriptor away from the FDHandle /// object, so that the file is not closed when the FDHandle is destroyed. - int take() throw(); + int release() { + int Ret = FD; + FD = -1; + return Ret; + } }; } // End llvm namespace diff --git a/include/llvm/Support/FileUtilities.h b/include/llvm/Support/FileUtilities.h index 978b61b..b17bca9 100644 --- a/include/llvm/Support/FileUtilities.h +++ b/include/llvm/Support/FileUtilities.h @@ -104,24 +104,41 @@ bool MakeFileReadable (const std::string & Filename); /// FDHandle - Simple handle class to make sure a file descriptor gets closed -/// when the object is destroyed. +/// when the object is destroyed. This handle acts similarly to an +/// std::auto_ptr, in that the copy constructor and assignment operators +/// transfer ownership of the handle. This means that FDHandle's do not have +/// value semantics. /// class FDHandle { int FD; - FDHandle(const FDHandle &); // DO NOT IMPLEMENT - void operator=(const FDHandle&); // DO NOT IMPLEMENT public: FDHandle() : FD(-1) {} FDHandle(int fd) : FD(fd) {} + FDHandle(FDHandle &RHS) : FD(RHS.FD) { + RHS.FD = -1; // Transfer ownership + } + ~FDHandle() throw(); + /// get - Get the current file descriptor, without releasing ownership of it. + int get() const { return FD; } operator int() const { return FD; } FDHandle &operator=(int fd) throw(); - /// take - Take ownership of the file descriptor away from the FDHandle + FDHandle &operator=(FDHandle &RHS) { + int fd = RHS.FD; + RHS.FD = -1; // Transfer ownership + return operator=(fd); + } + + /// release - Take ownership of the file descriptor away from the FDHandle /// object, so that the file is not closed when the FDHandle is destroyed. - int take() throw(); + int release() { + int Ret = FD; + FD = -1; + return Ret; + } }; } // End llvm namespace |