diff options
| author | Lioncash <mathew1800@gmail.com> | 2015-12-06 23:15:51 -0500 |
|---|---|---|
| committer | Lioncash <mathew1800@gmail.com> | 2015-12-07 05:57:37 -0500 |
| commit | edbbf493f807ed94b9b32cfb7463f9b7f8f27852 (patch) | |
| tree | 1babef3faeba508560effc1db476cbb722a9a709 /Source/Core/DiscIO/CompressedBlob.cpp | |
| parent | a0ac2b867368aa6f82ee3c471fea2cf1a450b37c (diff) | |
DiscIO: Make factory methods return unique_ptrs
Rather than rely on the developer to do the right thing,
just make the default behavior safely deallocate resources.
If shared semantics are ever needed in the future, the
constructor that takes a unique_ptr for shared_ptr can
be used.
Diffstat (limited to 'Source/Core/DiscIO/CompressedBlob.cpp')
| -rw-r--r-- | Source/Core/DiscIO/CompressedBlob.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/Source/Core/DiscIO/CompressedBlob.cpp b/Source/Core/DiscIO/CompressedBlob.cpp index 7b01aeed5d..845d969344 100644 --- a/Source/Core/DiscIO/CompressedBlob.cpp +++ b/Source/Core/DiscIO/CompressedBlob.cpp @@ -55,12 +55,12 @@ CompressedBlobReader::CompressedBlobReader(const std::string& filename) : m_file memset(m_zlib_buffer, 0, m_zlib_buffer_size); } -CompressedBlobReader* CompressedBlobReader::Create(const std::string& filename) +std::unique_ptr<CompressedBlobReader> CompressedBlobReader::Create(const std::string& filename) { if (IsGCZBlob(filename)) - return new CompressedBlobReader(filename); - else - return nullptr; + return std::unique_ptr<CompressedBlobReader>(new CompressedBlobReader(filename)); + + return nullptr; } CompressedBlobReader::~CompressedBlobReader() |
