From cd5d483bbd48e37203c2871dab7bb76884465b08 Mon Sep 17 00:00:00 2001 From: Andreas Kling Date: Tue, 7 Sep 2021 14:48:13 +0200 Subject: Kernel: Use KResultOr and TRY() for ThreadTracer Also make the constructor private, since it's only called by the static factory function. --- Kernel/ThreadTracer.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'Kernel/ThreadTracer.h') diff --git a/Kernel/ThreadTracer.h b/Kernel/ThreadTracer.h index c7a66a1e90..beec018b5c 100644 --- a/Kernel/ThreadTracer.h +++ b/Kernel/ThreadTracer.h @@ -15,7 +15,7 @@ namespace Kernel { class ThreadTracer { public: - static OwnPtr create(ProcessID tracer) { return try_make(tracer); } + static KResultOr> try_create(ProcessID tracer) { return adopt_nonnull_own_or_enomem(new (nothrow) ThreadTracer(tracer)); } ProcessID tracer_pid() const { return m_tracer_pid; } bool has_pending_signal(u32 signal) const { return m_pending_signals & (1 << (signal - 1)); } @@ -34,9 +34,9 @@ public: return m_regs.value(); } +private: explicit ThreadTracer(ProcessID); -private: ProcessID m_tracer_pid { -1 }; // This is a bitmap for signals that are sent from the tracer to the tracee -- cgit v1.2.3