diff options
author | Lenny Maiorani <lenny@colorado.edu> | 2020-12-20 16:09:48 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-12-21 00:09:48 +0100 |
commit | 765936ebaedfaa3a339d99a9865b555ddd7c23e2 (patch) | |
tree | e092294ec99ca5b3ba9c92139f847dcd9a8a20bc /Kernel/Time | |
parent | 4421d98e30763424055eefe729c9cab28abdf19d (diff) | |
download | serenity-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/Time')
-rw-r--r-- | Kernel/Time/APICTimer.cpp | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/Kernel/Time/APICTimer.cpp b/Kernel/Time/APICTimer.cpp index 7f91f4a06b..2b0a80e24f 100644 --- a/Kernel/Time/APICTimer.cpp +++ b/Kernel/Time/APICTimer.cpp @@ -165,21 +165,18 @@ void APICTimer::reset_to_default_ticks_per_second() { } -bool APICTimer::try_to_set_frequency(size_t frequency) +bool APICTimer::try_to_set_frequency([[maybe_unused]] size_t frequency) { - (void)frequency; return true; } -bool APICTimer::is_capable_of_frequency(size_t frequency) const +bool APICTimer::is_capable_of_frequency([[maybe_unused]] size_t frequency) const { - (void)frequency; return false; } -size_t APICTimer::calculate_nearest_possible_frequency(size_t frequency) const +size_t APICTimer::calculate_nearest_possible_frequency([[maybe_unused]] size_t frequency) const { - (void)frequency; return 0; } |