summaryrefslogtreecommitdiff
path: root/Kernel/VM
diff options
context:
space:
mode:
authorLenny Maiorani <lenny@colorado.edu>2020-12-20 16:09:48 -0700
committerGitHub <noreply@github.com>2020-12-21 00:09:48 +0100
commit765936ebaedfaa3a339d99a9865b555ddd7c23e2 (patch)
treee092294ec99ca5b3ba9c92139f847dcd9a8a20bc /Kernel/VM
parent4421d98e30763424055eefe729c9cab28abdf19d (diff)
downloadserenity-765936ebaedfaa3a339d99a9865b555ddd7c23e2.zip
Everywhere: Switch from (void) to [[maybe_unused]] (#4473)
Problem: - `(void)` simply casts the expression to void. This is understood to indicate that it is ignored, but this is really a compiler trick to get the compiler to not generate a warning. Solution: - Use the `[[maybe_unused]]` attribute to indicate the value is unused. Note: - Functions taking a `(void)` argument list have also been changed to `()` because this is not needed and shows up in the same grep command.
Diffstat (limited to 'Kernel/VM')
-rw-r--r--Kernel/VM/InodeVMObject.cpp4
1 files changed, 1 insertions, 3 deletions
diff --git a/Kernel/VM/InodeVMObject.cpp b/Kernel/VM/InodeVMObject.cpp
index c63417fab0..970dec24a9 100644
--- a/Kernel/VM/InodeVMObject.cpp
+++ b/Kernel/VM/InodeVMObject.cpp
@@ -89,10 +89,8 @@ void InodeVMObject::inode_size_changed(Badge<Inode>, size_t old_size, size_t new
});
}
-void InodeVMObject::inode_contents_changed(Badge<Inode>, off_t offset, ssize_t size, const UserOrKernelBuffer& data)
+void InodeVMObject::inode_contents_changed(Badge<Inode>, off_t offset, [[maybe_unused]] ssize_t size, [[maybe_unused]] const UserOrKernelBuffer& data)
{
- (void)size;
- (void)data;
InterruptDisabler disabler;
ASSERT(offset >= 0);