mirrors/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-04-21 20:15:17 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 10

LibGC: Allow move-assigning RootVector instances

Browse source 
Rule of 5 - we were missing a move-assignment operator, thus all move
assignments resulted in a copy.
This commit is contained in:
Timothy Flynn 2025-04-18 10:36:08 -04:00
parent bbef0e8375
commit 4d01e54865
2 changed files with 22 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
Libraries/LibGC/RootVector.cpp
View file

@ -21,16 +21,15 @@ RootVectorBase::~RootVectorBase()
m_heap->did_destroy_root_vector({}, *this);
}
RootVectorBase& RootVectorBase::operator=(RootVectorBase const& other)
void RootVectorBase::assign_heap(Heap* heap)
{
if (m_heap != other.m_heap) {
m_heap = other.m_heap;
if (m_heap == heap)
return;
// NOTE: IntrusiveList will remove this RootVectorBase from the old heap it was part of.
m_heap->did_create_root_vector({}, *this);
}
m_heap = heap;
return *this;
// NOTE: IntrusiveList will remove this RootVectorBase from the old heap it was part of.
m_heap->did_create_root_vector({}, *this);
}
}

20
Libraries/LibGC/RootVector.h
View file

@ -24,7 +24,7 @@ protected:
explicit RootVectorBase(Heap&);
~RootVectorBase();
RootVectorBase& operator=(RootVectorBase const&);
void assign_heap(Heap*);
Heap* m_heap { nullptr };
IntrusiveListNode<RootVectorBase> m_list_node;
@ -38,6 +38,8 @@ class RootVector final
: public RootVectorBase
, public Vector<T, inline_capacity> {
using VectorBase = Vector<T, inline_capacity>;
public:
explicit RootVector(Heap& heap)
: RootVectorBase(heap)
@ -60,14 +62,24 @@ public:
RootVector(RootVector&& other)
: RootVectorBase(*other.m_heap)
, Vector<T, inline_capacity>(move(static_cast<Vector<T, inline_capacity>&>(other)))
, VectorBase(move(static_cast<VectorBase&>(other)))
{
}
RootVector& operator=(RootVector const& other)
{
Vector<T, inline_capacity>::operator=(other);
RootVectorBase::operator=(other);
if (&other == this)
return *this;
assign_heap(other.m_heap);
VectorBase::operator=(other);
return *this;
}
RootVector& operator=(RootVector&& other)
{
assign_heap(other.m_heap);
VectorBase::operator=(move(static_cast<VectorBase&>(other)));
return *this;
}