From c4c6d9367d96bdcf886b26839069ab486100eab6 Mon Sep 17 00:00:00 2001 From: Brian Gianforcaro Date: Wed, 5 Aug 2020 04:47:33 -0700 Subject: [PATCH] Kernel: Fix build break from missing KResult [[nodiscard]] suppressions Missed this somehow in previous change. --- Kernel/FileSystem/BlockBasedFileSystem.cpp | 6 ++++-- Kernel/Process.cpp | 3 ++- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/Kernel/FileSystem/BlockBasedFileSystem.cpp b/Kernel/FileSystem/BlockBasedFileSystem.cpp index c14a95d19d..cc6e9e261f 100644 --- a/Kernel/FileSystem/BlockBasedFileSystem.cpp +++ b/Kernel/FileSystem/BlockBasedFileSystem.cpp @@ -261,7 +261,8 @@ void BlockBasedFS::flush_specific_block_if_needed(unsigned index) if (entry.is_dirty && entry.block_index == index) { u32 base_offset = static_cast(entry.block_index) * static_cast(block_size()); file_description().seek(base_offset, SEEK_SET); - file_description().write(entry.data, block_size()); + // FIXME: Should this error path be surfaced somehow? + (void)file_description().write(entry.data, block_size()); entry.is_dirty = false; } }); @@ -278,7 +279,8 @@ void BlockBasedFS::flush_writes_impl() return; u32 base_offset = static_cast(entry.block_index) * static_cast(block_size()); file_description().seek(base_offset, SEEK_SET); - file_description().write(entry.data, block_size()); + // FIXME: Should this error path be surfaced somehow? + (void)file_description().write(entry.data, block_size()); ++count; entry.is_dirty = false; }); diff --git a/Kernel/Process.cpp b/Kernel/Process.cpp index e00d511f36..9cd084175a 100644 --- a/Kernel/Process.cpp +++ b/Kernel/Process.cpp @@ -639,7 +639,8 @@ void Process::finalize() if (!description_or_error.is_error()) { auto& description = description_or_error.value(); auto json = m_perf_event_buffer->to_json(m_pid, m_executable ? m_executable->absolute_path() : ""); - description->write(json.data(), json.size()); + // FIXME: Should this error path be surfaced somehow? + (void)description->write(json.data(), json.size()); } }