diff options
| author | Lukas Oesch <lukasoesch20@gmail.com> | 2026-05-31 13:49:35 +0200 |
|---|---|---|
| committer | Felix Morgner <felix.morgner@ost.ch> | 2026-07-14 20:21:58 +0200 |
| commit | 34079d19ab57a38dad67bcd001ff2467e7bd9e99 (patch) | |
| tree | 9fd1c74bfde4f5a80c12a43fd98a6fae7975c1b5 | |
| parent | efdb44b9ca98ccea925a4fbf51b0b1a9960570b2 (diff) | |
| download | kernel-34079d19ab57a38dad67bcd001ff2467e7bd9e99.tar.xz kernel-34079d19ab57a38dad67bcd001ff2467e7bd9e99.zip | |
refactoring
| -rw-r--r-- | kernel/kernel/filesystem/ext2/filesystem.cpp | 1 | ||||
| -rw-r--r-- | kernel/kernel/filesystem/ext2/inode.tests.cpp | 10 |
2 files changed, 6 insertions, 5 deletions
diff --git a/kernel/kernel/filesystem/ext2/filesystem.cpp b/kernel/kernel/filesystem/ext2/filesystem.cpp index 3aa07945..f29b3868 100644 --- a/kernel/kernel/filesystem/ext2/filesystem.cpp +++ b/kernel/kernel/filesystem/ext2/filesystem.cpp @@ -507,4 +507,5 @@ namespace kernel::filesystem::ext2 { return block_size() == 1024 ? 2 * block_size() : block_size(); } + } // namespace kernel::filesystem::ext2 diff --git a/kernel/kernel/filesystem/ext2/inode.tests.cpp b/kernel/kernel/filesystem/ext2/inode.tests.cpp index 8341dbbf..4d5038f7 100644 --- a/kernel/kernel/filesystem/ext2/inode.tests.cpp +++ b/kernel/kernel/filesystem/ext2/inode.tests.cpp @@ -319,28 +319,28 @@ SCENARIO_METHOD(kernel::tests::filesystem::storage_boot_module_fixture, "Ext2 in auto const text = std::string_view{reinterpret_cast<char const *>(read_buffer.data()), *bytes_read}; REQUIRE(text == "Hello World!"); - auto inode_data = static_cast<kernel::filesystem::ext2::inode *>(file.get())->data(); + auto inode_data = static_cast<kernel::filesystem::ext2::inode *>(file.value().get())->data(); REQUIRE(inode_data.size == bytes_read); } THEN("writing allocates new blocks and updates the inode size") { - auto inode_data = static_cast<kernel::filesystem::ext2::inode *>(file.get())->data(); + auto inode_data = static_cast<kernel::filesystem::ext2::inode *>(file.value().get())->data(); auto const block_size = fs.block_size(); auto const expected_allocated_blocks = 32 * fs.inode_block_count(inode_data); auto write_buffer = kstd::vector<std::byte>(block_size * expected_allocated_blocks, std::byte{'A'}); - auto const bytes_written = file->write(write_buffer.data(), 0, write_buffer.size()); + auto const bytes_written = file.value()->write(write_buffer.data(), 0, write_buffer.size()); REQUIRE(bytes_written == block_size * expected_allocated_blocks); auto read_buffer = kstd::vector<std::byte>(block_size * expected_allocated_blocks, std::byte{0x00}); - auto const bytes_read = file->read(read_buffer.data(), 0, read_buffer.size()); + auto const bytes_read = file.value()->read(read_buffer.data(), 0, read_buffer.size()); REQUIRE(bytes_read == block_size * expected_allocated_blocks); REQUIRE(std::ranges::all_of(read_buffer, [](std::byte c) { return c == std::byte{'A'}; })); - auto new_inode_data = static_cast<kernel::filesystem::ext2::inode *>(file.get())->data(); + auto new_inode_data = static_cast<kernel::filesystem::ext2::inode *>(file.value().get())->data(); REQUIRE(new_inode_data.size == write_buffer.size()); REQUIRE(fs.inode_block_count(new_inode_data) == expected_allocated_blocks); } |
