diff options
author | Wonsik Kim <wonsik@google.com> | 2014-03-20 15:42:03 +0900 |
---|---|---|
committer | Wonsik Kim <wonsik@google.com> | 2014-03-24 11:40:36 +0900 |
commit | c4cc584bbd031b94c37912d8c5e340b24ff31b57 (patch) | |
tree | a70d9c987788b1503ecfe2fd5936d07d681a59cc /include/utils/NativeHandle.h | |
parent | a2910877a6781f57a7ef3eea7343fa46bc3e1bd0 (diff) | |
download | system_core-c4cc584bbd031b94c37912d8c5e340b24ff31b57.tar.gz system_core-c4cc584bbd031b94c37912d8c5e340b24ff31b57.tar.bz2 system_core-c4cc584bbd031b94c37912d8c5e340b24ff31b57.zip |
Clarify ownership for NativeHandle::mHandle
Change-Id: I0835278df1aa78f10d5493d7ef2c9e4a15c0fee9
Diffstat (limited to 'include/utils/NativeHandle.h')
-rw-r--r-- | include/utils/NativeHandle.h | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/include/utils/NativeHandle.h b/include/utils/NativeHandle.h index dc49b5181..b82516879 100644 --- a/include/utils/NativeHandle.h +++ b/include/utils/NativeHandle.h @@ -26,9 +26,11 @@ namespace android { class NativeHandle: public LightRefBase<NativeHandle> { public: - // Create a refcounted wrapper around a native_handle_t. + // Create a refcounted wrapper around a native_handle_t, and declare + // whether the wrapper owns the handle (so that it should clean up the + // handle upon destruction) or not. // If handle is NULL, no NativeHandle will be created. - static sp<NativeHandle> create(native_handle_t* handle); + static sp<NativeHandle> create(native_handle_t* handle, bool ownsHandle); const native_handle_t* handle() const { return mHandle; @@ -38,10 +40,11 @@ private: // for access to the destructor friend class LightRefBase<NativeHandle>; - NativeHandle(native_handle_t* handle); + NativeHandle(native_handle_t* handle, bool ownsHandle); virtual ~NativeHandle(); native_handle_t* mHandle; + bool mOwnsHandle; // non-copyable NativeHandle(const NativeHandle&); |