mirror of
https://github.com/LadybirdBrowser/ladybird.git
synced 2025-01-24 02:03:06 -05:00
bc319d9e88
Okay, I've spent a whole day on this now, and it finally kinda works! With this patch, CObject and all of its derived classes are reference counted instead of tree-owned. The previous, Qt-like model was nice and familiar, but ultimately also outdated and difficult to reason about. CObject-derived types should now be stored in RefPtr/NonnullRefPtr and each class can be constructed using the forwarding construct() helper: auto widget = GWidget::construct(parent_widget); Note that construct() simply forwards all arguments to an existing constructor. It is inserted into each class by the C_OBJECT macro, see CObject.h to understand how that works. CObject::delete_later() disappears in this patch, as there is no longer a single logical owner of a CObject.
48 lines
1.2 KiB
C++
48 lines
1.2 KiB
C++
#include <LibThread/BackgroundAction.h>
|
|
#include <LibThread/Thread.h>
|
|
#include <LibThread/Lock.h>
|
|
#include <AK/Queue.h>
|
|
|
|
static LibThread::Lockable<Queue<Function<void()>>>* s_all_actions;
|
|
static LibThread::Thread* s_background_thread;
|
|
|
|
static int background_thread_func()
|
|
{
|
|
while (true) {
|
|
Function<void()> work_item;
|
|
{
|
|
LOCKER(s_all_actions->lock());
|
|
|
|
if (!s_all_actions->resource().is_empty())
|
|
work_item = s_all_actions->resource().dequeue();
|
|
}
|
|
if (work_item)
|
|
work_item();
|
|
else
|
|
sleep(1);
|
|
}
|
|
|
|
ASSERT_NOT_REACHED();
|
|
}
|
|
|
|
static void init()
|
|
{
|
|
s_all_actions = new LibThread::Lockable<Queue<Function<void()>>>();
|
|
s_background_thread = &LibThread::Thread::construct(background_thread_func).leak_ref();
|
|
s_background_thread->set_name("Background thread");
|
|
s_background_thread->start();
|
|
}
|
|
|
|
LibThread::Lockable<Queue<Function<void()>>>& LibThread::BackgroundActionBase::all_actions()
|
|
{
|
|
if (s_all_actions == nullptr)
|
|
init();
|
|
return *s_all_actions;
|
|
}
|
|
|
|
LibThread::Thread& LibThread::BackgroundActionBase::background_thread()
|
|
{
|
|
if (s_background_thread == nullptr)
|
|
init();
|
|
return *s_background_thread;
|
|
}
|