xenia/home
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

RefPtr: use inconditional noexcept specification on destructor

Browse Source 
The conditional specification could end up being recursive,
assume destructors must be (as is the C++ default) noexcept.
This commit is contained in:
Maxime Coste 2018-02-28 15:04:00 +11:00
parent 0d838f80a0
commit bebc81ebe1
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/ref_ptr.hh
View File

@ -22,7 +22,7 @@ struct RefCountable
struct RefCountablePolicy
{
static void inc_ref(RefCountable* r, void*) noexcept { ++r->refcount; }
static void dec_ref(RefCountable* r, void*) noexcept(noexcept(r->~RefCountable())) { if (--r->refcount == 0) delete r; }
static void dec_ref(RefCountable* r, void*) noexcept { if (--r->refcount == 0) delete r; }
static void ptr_moved(RefCountable*, void*, void*) noexcept {}
};
@ -31,7 +31,7 @@ struct RefPtr
{
RefPtr() = default;
explicit RefPtr(T* ptr) : m_ptr(ptr) { acquire(); }
~RefPtr() noexcept(noexcept(std::declval<RefPtr>().release())) { release(); }
~RefPtr() noexcept { release(); }
RefPtr(const RefPtr& other) : m_ptr(other.m_ptr) { acquire(); }
RefPtr(RefPtr&& other)
noexcept(noexcept(std::declval<RefPtr>().moved(nullptr)))
@ -102,8 +102,7 @@ private:
}
[[gnu::always_inline]]
void release()
noexcept(noexcept(Policy::dec_ref(nullptr, nullptr)))
void release() noexcept
{
if (m_ptr)
Policy::dec_ref(m_ptr, this);