diff options
| author | Tatsuhiro Tsujikawa <tatsuhiro.t@gmail.com> | 2013-06-26 23:56:43 +0900 |
|---|---|---|
| committer | Tatsuhiro Tsujikawa <tatsuhiro.t@gmail.com> | 2013-06-26 23:56:43 +0900 |
| commit | da7400ef5c62ee144fc14ba7ca4745349f0b0c72 (patch) | |
| tree | b0d023eecfcfcca2a88eb3466baf878bdff9f4c1 /src/DownloadEngine.h | |
| parent | 47402c5f29cb806f73aae2ed18aa2dde21956700 (diff) | |
Return std::unique_ptr member as const ref
Returning raw pointer has a risk that it may be stolen by
std::shared_ptr in accident.
Diffstat (limited to 'src/DownloadEngine.h')
| -rw-r--r-- | src/DownloadEngine.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/DownloadEngine.h b/src/DownloadEngine.h index 0f2e67fd..00f1aab2 100644 --- a/src/DownloadEngine.h +++ b/src/DownloadEngine.h @@ -302,7 +302,7 @@ public: uint16_t port, const std::string& username); - CookieStorage* getCookieStorage() const; + const std::unique_ptr<CookieStorage>& getCookieStorage() const; #ifdef ENABLE_BITTORRENT const std::shared_ptr<BtRegistry>& getBtRegistry() const @@ -333,7 +333,7 @@ public: void setAuthConfigFactory(std::unique_ptr<AuthConfigFactory> factory); - AuthConfigFactory* getAuthConfigFactory() const; + const std::unique_ptr<AuthConfigFactory>& getAuthConfigFactory() const; void setRefreshInterval(int64_t interval); |