mirror of
https://github.com/LadybirdBrowser/ladybird.git
synced 2024-11-25 00:50:22 +00:00
AK: Hoist the assertion in Retainable::release() to RetainableBase.
This means we don't have to generate a __PRETTY_FUNCTION__ symbol there for each and every specialization.
This commit is contained in:
parent
3ebb5fbb87
commit
dd5ca1940a
Notes:
sideshowbarker
2024-07-19 15:01:43 +09:00
Author: https://github.com/awesomekling Commit: https://github.com/SerenityOS/serenity/commit/dd5ca1940a1
1 changed files with 7 additions and 2 deletions
|
@ -49,6 +49,12 @@ protected:
|
|||
ASSERT(!m_retain_count);
|
||||
}
|
||||
|
||||
void release_base()
|
||||
{
|
||||
ASSERT(m_retain_count);
|
||||
--m_retain_count;
|
||||
}
|
||||
|
||||
int m_retain_count { 1 };
|
||||
};
|
||||
|
||||
|
@ -57,8 +63,7 @@ class Retainable : public RetainableBase {
|
|||
public:
|
||||
void release()
|
||||
{
|
||||
ASSERT(m_retain_count);
|
||||
--m_retain_count;
|
||||
release_base();
|
||||
if (m_retain_count == 0) {
|
||||
call_will_be_destroyed_if_present(static_cast<T*>(this));
|
||||
delete static_cast<T*>(this);
|
||||
|
|
Loading…
Reference in a new issue