diff options
| author | tsepez <tsepez@chromium.org> | 2016-08-30 06:37:54 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-08-30 06:37:54 -0700 |
| commit | 94fb74c3137ab4f9f817f4cfb14e0a68a677422b (patch) | |
| tree | 40fb2336c3f95d66e96e532c7ceb8f18bc53fce5 /core/fxcrt/include/cfx_count_ref.h | |
| parent | 35512aa7e4acc3ceb9c6aef5d61eebfb4ae802af (diff) | |
| download | pdfium-94fb74c3137ab4f9f817f4cfb14e0a68a677422b.tar.xz | |
More better const-ness in CFX_CountRef<>
There are a couple of places where code is calling GetObject()
that should be calling GetPrivateCopy(), but works because the
value may not be shared at the time. This just makes it safer.
Review-Url: https://codereview.chromium.org/2290863002
Diffstat (limited to 'core/fxcrt/include/cfx_count_ref.h')
| -rw-r--r-- | core/fxcrt/include/cfx_count_ref.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/core/fxcrt/include/cfx_count_ref.h b/core/fxcrt/include/cfx_count_ref.h index 817ce95add..954b72bf88 100644 --- a/core/fxcrt/include/cfx_count_ref.h +++ b/core/fxcrt/include/cfx_count_ref.h @@ -30,7 +30,6 @@ class CFX_CountRef { } void SetNull() { m_pObject.Reset(); } - ObjClass* GetObject() { return m_pObject.Get(); } const ObjClass* GetObject() const { return m_pObject.Get(); } template <typename... Args> |