diff options
author | Lei Zhang <thestig@chromium.org> | 2018-10-10 17:28:33 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-10-10 17:28:33 +0000 |
commit | 2c4c95d545c6e27b775a286e88130caadf8d5a7c (patch) | |
tree | b5aa06a161591f81279296c3ee0dd8ed6f8a9a6a /core/fxcrt/observable.h | |
parent | 7bf816420aa79b23c37ad433183f01dc5af653fa (diff) | |
download | pdfium-2c4c95d545c6e27b775a286e88130caadf8d5a7c.tar.xz |
Fix some random nits.
Change-Id: I735c057d4900bcd58c1041df16b885dc7c9ed27d
Reviewed-on: https://pdfium-review.googlesource.com/c/43793
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Diffstat (limited to 'core/fxcrt/observable.h')
-rw-r--r-- | core/fxcrt/observable.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/core/fxcrt/observable.h b/core/fxcrt/observable.h index e99dadfc8e..e118dc898b 100644 --- a/core/fxcrt/observable.h +++ b/core/fxcrt/observable.h @@ -25,13 +25,13 @@ class Observable { // Simple case of a self-nulling pointer. class ObservedPtr final : public Observer { public: - ObservedPtr() : m_pObservable(nullptr) {} + ObservedPtr() = default; explicit ObservedPtr(T* pObservable) : m_pObservable(pObservable) { if (m_pObservable) m_pObservable->AddObserver(this); } ObservedPtr(const ObservedPtr& that) : ObservedPtr(that.Get()) {} - ~ObservedPtr() { + ~ObservedPtr() override { if (m_pObservable) m_pObservable->RemoveObserver(this); } @@ -60,7 +60,7 @@ class Observable { T* operator->() const { return m_pObservable; } private: - T* m_pObservable; + T* m_pObservable = nullptr; }; Observable() = default; |