RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-05-31 07:08:10 +00:00
Code Issues Activity Actions

Utilities: Use lround() instead of casting round() to long

Browse source 
And one case where we previously cast to int, since the extra precision
does not matter.
This commit is contained in:
Sam Atkins 2023-03-10 13:26:16 +00:00 • committed by Andreas Kling
parent f2ae25deee
commit b3cd91d26d
3 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

2
Userland/Utilities/adjtime.cpp
View file

@ -21,7 +21,7 @@ ErrorOr<int> serenity_main(Main::Arguments arguments)
args_parser.parse(arguments); args_parser.parse(arguments);
if (delta.has_value()) { if (delta.has_value()) {
long delta_us = static_cast<long>(round(*delta * 1'000'000)); long delta_us = lround(*delta * 1'000'000);
timeval delta_timeval; timeval delta_timeval;
delta_timeval.tv_sec = delta_us / 1'000'000; delta_timeval.tv_sec = delta_us / 1'000'000;
delta_timeval.tv_usec = delta_us % 1'000'000; delta_timeval.tv_usec = delta_us % 1'000'000;

2
Userland/Utilities/asctl.cpp
View file

@ -69,7 +69,7 @@ ErrorOr<int> serenity_main(Main::Arguments arguments)
for (auto to_print : values_to_print) { for (auto to_print : values_to_print) {
switch (to_print) { switch (to_print) {
case AudioVariable::Volume: { case AudioVariable::Volume: {
auto volume = static_cast<int>(round(audio_client->get_main_mix_volume() * 100)); auto volume = lround(audio_client->get_main_mix_volume() * 100);
if (human_mode) if (human_mode)
outln("Volume: {}%", volume); outln("Volume: {}%", volume);
else else

2
Userland/Utilities/ntpquery.cpp
View file

@ -306,7 +306,7 @@ ErrorOr<int> serenity_main(Main::Arguments arguments)
outln("Offset: {}", offset_s); outln("Offset: {}", offset_s);
if (adjust_time) { if (adjust_time) {
long delta_us = static_cast<long>(round(offset_s * 1'000'000)); long delta_us = lround(offset_s * 1'000'000);
timeval delta_timeval; timeval delta_timeval;
delta_timeval.tv_sec = delta_us / 1'000'000; delta_timeval.tv_sec = delta_us / 1'000'000;
delta_timeval.tv_usec = delta_us % 1'000'000; delta_timeval.tv_usec = delta_us % 1'000'000;