diff options
author | Maksim Denisov <denisov.maksim@huawei.com> | 2024-10-01 12:23:17 +0200 |
---|---|---|
committer | Maksim Denisov <denisov.maksim@huawei.com> | 2024-10-07 13:37:39 +0200 |
commit | 7df9944604e2ef92abce20c00ea265793b151261 (patch) | |
tree | df758ede71200d463db1b2c05dd03e8cd0a76765 /src/buildtool/execution_api/remote/bazel/bazel_api.cpp | |
parent | f56805ddde51ffcfdd6123300b6c049764a86980 (diff) | |
download | justbuild-7df9944604e2ef92abce20c00ea265793b151261.tar.gz |
Enable bugprone-narrowing-conversions check
Diffstat (limited to 'src/buildtool/execution_api/remote/bazel/bazel_api.cpp')
-rw-r--r-- | src/buildtool/execution_api/remote/bazel/bazel_api.cpp | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/src/buildtool/execution_api/remote/bazel/bazel_api.cpp b/src/buildtool/execution_api/remote/bazel/bazel_api.cpp index c5d77194..8dbd026e 100644 --- a/src/buildtool/execution_api/remote/bazel/bazel_api.cpp +++ b/src/buildtool/execution_api/remote/bazel/bazel_api.cpp @@ -151,14 +151,16 @@ namespace { transmitted_bytes += chunk_digest.size(); } } - double transmission_factor = - (total_size > 0) ? 100.0 * transmitted_bytes / total_size - : 100.0; + double transmission_factor = 0.; + if (total_size > 0) { + transmission_factor = static_cast<double>(transmitted_bytes) / + static_cast<double>(total_size); + } return fmt::format( "Blob splitting saved {} bytes ({:.2f}%) of network traffic " "when fetching {}.\n", total_size - transmitted_bytes, - 100.0 - transmission_factor, + transmission_factor, artifact_info.ToString()); }); |