From db2e67fd53facb5120f565bff761abf9a6d99cb6 Mon Sep 17 00:00:00 2001
From: Andreas Kling <kling@serenityos.org>
Date: Tue, 7 Sep 2021 12:27:10 +0200
Subject: [PATCH] Kernel: Tidy up ProcessProcFSTraits construction a bit more

Let the constructor take a Process& instead of a WeakPtr<Process> and
avoid a bunch of WeakPtr copying.
---
 Kernel/Process.cpp | 2 +-
 Kernel/Process.h   | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/Kernel/Process.cpp b/Kernel/Process.cpp
index 356b7513c6..e1c75f3ad2 100644
--- a/Kernel/Process.cpp
+++ b/Kernel/Process.cpp
@@ -268,7 +268,7 @@ KResult Process::attach_resources(NonnullOwnPtr<Memory::AddressSpace>&& prealloc
         first_thread->detach();
     }
 
-    m_procfs_traits = TRY(ProcessProcFSTraits::try_create({}, make_weak_ptr()));
+    m_procfs_traits = TRY(ProcessProcFSTraits::try_create({}, *this));
     return KSuccess;
 }
 
diff --git a/Kernel/Process.h b/Kernel/Process.h
index e51224f8bd..5aed32eeb3 100644
--- a/Kernel/Process.h
+++ b/Kernel/Process.h
@@ -718,7 +718,7 @@ public:
 
     class ProcessProcFSTraits : public ProcFSExposedComponent {
     public:
-        static KResultOr<NonnullRefPtr<ProcessProcFSTraits>> try_create(Badge<Process>, WeakPtr<Process> process)
+        static KResultOr<NonnullRefPtr<ProcessProcFSTraits>> try_create(Badge<Process>, Process& process)
         {
             return adopt_nonnull_ref_or_enomem(new (nothrow) ProcessProcFSTraits(process));
         }
@@ -732,8 +732,8 @@ public:
         virtual GroupID owner_group() const override;
 
     private:
-        ProcessProcFSTraits(WeakPtr<Process> process)
-            : m_process(process)
+        explicit ProcessProcFSTraits(Process& process)
+            : m_process(process.make_weak_ptr())
         {
         }