Merge branch 'refactoring' into 'develop-BA-FS26'

refactoring

See merge request teachos/kernel!38

1 files changed, 5 insertions, 5 deletions

diff --git a/kernel/src/filesystem/devfs/filesystem.cpp b/kernel/src/filesystem/devfs/filesystem.cpp
index 96e40a8..f0d8bf7 100644
--- a/kernel/src/filesystem/devfs/filesystem.cpp
+++ b/kernel/src/filesystem/devfs/filesystem.cpp
@@ -1,5 +1,6 @@
 #include <kernel/filesystem/devfs/filesystem.hpp>
 
+#include "kernel/filesystem/filesystem.hpp"
 #include <kernel/devices/storage/management.hpp>
 #include <kernel/filesystem/devfs/inode.hpp>
 #include <kernel/filesystem/device_inode.hpp>
@@ -22,7 +23,7 @@ namespace kernel::filesystem::devfs
     return operation_result::success;
   }
 
-  auto filesystem::lookup(kstd::shared_ptr<kernel::filesystem::inode> const & parent, std::string_view name)
+  auto filesystem::lookup(kstd::shared_ptr<kernel::filesystem::inode> const & parent, std::string_view name) const
       -> kstd::shared_ptr<kernel::filesystem::inode>
   {
     if (!parent || !parent->is_directory())
@@ -36,12 +37,11 @@ namespace kernel::filesystem::devfs
     }
 
     auto it = std::ranges::find_if(m_inodes, [&](auto const & dev_node) {
-      auto device_inode_ptr = static_cast<device_inode *>(dev_node.get());
-      if (!device_inode_ptr)
+      if (auto device_inode_ptr = static_cast<device_inode *>(dev_node.get()))
       {
-        return false;
+        return device_inode_ptr->device()->name() == name;
       }
-      return device_inode_ptr->device()->name() == name;
+      return false;
     });
     return (it != m_inodes.end()) ? *it : nullptr;
   }