diff options
| author | Cory Fields <[email protected]> | 2014-09-25 19:25:47 -0400 |
|---|---|---|
| committer | Cory Fields <[email protected]> | 2014-09-26 13:27:40 -0400 |
| commit | 6eb67b0ed2b350b772f7edb67aee1bcf09c91b0b (patch) | |
| tree | f0e5a6c11a01cbfc861ed86fcd122ff7253a5948 /src | |
| parent | autofile: don't copy CAutoFile by value (diff) | |
| download | discoin-6eb67b0ed2b350b772f7edb67aee1bcf09c91b0b.tar.xz discoin-6eb67b0ed2b350b772f7edb67aee1bcf09c91b0b.zip | |
autofile: Disallow by-value copies of CAutoFile
One might assume that CAutoFile would be ref-counted so that a copied object
would delay closing the underlying file until all copies have gone out of
scope. Since that's not the case with CAutoFile, explicitly disable copying.
Diffstat (limited to 'src')
| -rw-r--r-- | src/serialize.h | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/serialize.h b/src/serialize.h index 447d808de..7f8f93328 100644 --- a/src/serialize.h +++ b/src/serialize.h @@ -1154,7 +1154,7 @@ public: -/** RAII wrapper for FILE*. +/** Non-refcounted RAII wrapper for FILE*. * * Will automatically close the file when it goes out of scope if not null. * If you're returning the file pointer, return file.release(). @@ -1162,6 +1162,10 @@ public: */ class CAutoFile { +private: + // Disallow copies + CAutoFile(const CAutoFile&); + CAutoFile& operator=(const CAutoFile&); protected: FILE* file; public: |