From 8a11512eb8030de09ed806d8b40b7f54c2f204ab Mon Sep 17 00:00:00 2001
From: Marcel Braun <marcel.braun@ost.ch>
Date: Mon, 11 May 2026 14:48:46 +0200
Subject: Refactor get_full_path remove recursion

---
 kernel/src/filesystem/dentry.cpp | 34 +++++++++++++++++++++++++---------
 1 file changed, 25 insertions(+), 9 deletions(-)

(limited to 'kernel/src/filesystem/dentry.cpp')

diff --git a/kernel/src/filesystem/dentry.cpp b/kernel/src/filesystem/dentry.cpp
index 1cf8730..a3bd7f4 100644
--- a/kernel/src/filesystem/dentry.cpp
+++ b/kernel/src/filesystem/dentry.cpp
@@ -6,6 +6,7 @@
 
 #include <kstd/memory>
 #include <kstd/string>
+#include <kstd/vector>
 
 #include <algorithm>
 #include <cstdint>
@@ -39,20 +40,35 @@ namespace kernel::filesystem
     return m_name.view();
   }
 
-  // NOLINTNEXTLINE(misc-no-recursion)
   auto dentry::get_full_path() const -> kstd::string
   {
-    if (m_parent)
+    if (m_name == "/")
     {
-      auto parent_path = m_parent->get_full_path();
-      if (parent_path != "/")
-      {
-        parent_path += '/';
-      }
-      return parent_path + m_name.view();
+      return "/";
     }
 
-    return m_name.view();
+    kstd::vector<std::string_view> components;
+    components.push_back(m_name.view());
+    components.push_back("/");
+
+    auto parent = m_parent;
+
+    while (parent && parent->m_name != "/")
+    {
+      components.push_back(parent->m_name.view());
+      components.push_back("/");
+      parent = parent->get_parent();
+    }
+
+    std::ranges::reverse(components);
+
+    kstd::string full_path = "";
+    for (auto const & view : components)
+    {
+      full_path += view;
+    }
+
+    return full_path;
   }
 
   auto dentry::add_child(kstd::shared_ptr<dentry> const & child) -> void
-- 
cgit v1.2.3


From 7ea0aa798e3062dea20a317e2b19d7cf879611ca Mon Sep 17 00:00:00 2001
From: Marcel Braun <marcel.braun@ost.ch>
Date: Mon, 11 May 2026 18:46:10 +0200
Subject: Refactor dentry

---
 kernel/src/filesystem/dentry.cpp | 40 ++++++++++++++++++----------------------
 1 file changed, 18 insertions(+), 22 deletions(-)

(limited to 'kernel/src/filesystem/dentry.cpp')

diff --git a/kernel/src/filesystem/dentry.cpp b/kernel/src/filesystem/dentry.cpp
index a3bd7f4..7617b28 100644
--- a/kernel/src/filesystem/dentry.cpp
+++ b/kernel/src/filesystem/dentry.cpp
@@ -23,6 +23,10 @@ namespace kernel::filesystem
     {
       kapi::system::panic("[FILESYSTEM] dentry constructed with null inode.");
     }
+    if (m_name.empty())
+    {
+      kapi::system::panic("[FILESYSTEM] dentry constructed with empty name.");
+    }
   }
 
   auto dentry::get_inode() const -> kstd::shared_ptr<inode> const &
@@ -42,33 +46,25 @@ namespace kernel::filesystem
 
   auto dentry::get_full_path() const -> kstd::string
   {
-    if (m_name == "/")
-    {
-      return "/";
-    }
-
-    kstd::vector<std::string_view> components;
-    components.push_back(m_name.view());
-    components.push_back("/");
+    kstd::string path = m_name;
 
     auto parent = m_parent;
-
-    while (parent && parent->m_name != "/")
-    {
-      components.push_back(parent->m_name.view());
-      components.push_back("/");
-      parent = parent->get_parent();
-    }
-
-    std::ranges::reverse(components);
-
-    kstd::string full_path = "";
-    for (auto const & view : components)
+    while (parent)
     {
-      full_path += view;
+      auto parent_name = parent->m_name;
+      if (parent_name == "/")
+      {
+        path = "/" + path;
+      }
+      else
+      {
+        path = parent_name + "/" + path;
+      }
+
+      parent = parent->m_parent;
     }
 
-    return full_path;
+    return path;
   }
 
   auto dentry::add_child(kstd::shared_ptr<dentry> const & child) -> void
-- 
cgit v1.2.3


From a4c9b9bdb768ce20c9f9eac353a34598e4a422a9 Mon Sep 17 00:00:00 2001
From: Marcel Braun <marcel.braun@ost.ch>
Date: Mon, 11 May 2026 18:54:27 +0200
Subject: Rename get_full_path to get_absolute_path

---
 kernel/src/filesystem/dentry.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'kernel/src/filesystem/dentry.cpp')

diff --git a/kernel/src/filesystem/dentry.cpp b/kernel/src/filesystem/dentry.cpp
index 7617b28..59fd89e 100644
--- a/kernel/src/filesystem/dentry.cpp
+++ b/kernel/src/filesystem/dentry.cpp
@@ -44,7 +44,7 @@ namespace kernel::filesystem
     return m_name.view();
   }
 
-  auto dentry::get_full_path() const -> kstd::string
+  auto dentry::get_absolute_path() const -> kstd::string
   {
     kstd::string path = m_name;
 
-- 
cgit v1.2.3


From c958b72922b89fff35c0b8e0bbf21ad42a667022 Mon Sep 17 00:00:00 2001
From: Marcel Braun <marcel.braun@ost.ch>
Date: Mon, 11 May 2026 20:27:01 +0200
Subject: Remove unused include

---
 kernel/src/filesystem/dentry.cpp | 1 -
 1 file changed, 1 deletion(-)

(limited to 'kernel/src/filesystem/dentry.cpp')

diff --git a/kernel/src/filesystem/dentry.cpp b/kernel/src/filesystem/dentry.cpp
index 59fd89e..a77ce23 100644
--- a/kernel/src/filesystem/dentry.cpp
+++ b/kernel/src/filesystem/dentry.cpp
@@ -6,7 +6,6 @@
 
 #include <kstd/memory>
 #include <kstd/string>
-#include <kstd/vector>
 
 #include <algorithm>
 #include <cstdint>
-- 
cgit v1.2.3