diff options
| author |  Luca Fedeli <luca.fedeli@cea.fr>
| 2022-02-08 00:18:55 +0100 |
|---|---|---|
| committer |  GitHub <noreply@github.com>
| 2022-02-07 15:18:55 -0800 |
| commit | e29ca3a14a4d514b765fe807808e95e7ef887260 (patch) | |
| tree | ec02b15c02bd37039c0ba8bb886f83e46397914c /Source/Utils/WarpXUtil.cpp | |
| parent | 5c3e30f77fd23ad9c7fb9f99e862ddbddc6e922f (diff) | |
| download | WarpX-e29ca3a14a4d514b765fe807808e95e7ef887260.tar.gz WarpX-e29ca3a14a4d514b765fe807808e95e7ef887260.tar.zst WarpX-e29ca3a14a4d514b765fe807808e95e7ef887260.zip | |
Implement some clang-tidy suggestions (#2826)
* implement some clang-tidy suggestions
* Update loop
Co-authored-by: Axel Huebl <axel.huebl@plasma.ninja>
Diffstat (limited to 'Source/Utils/WarpXUtil.cpp')
| -rw-r--r-- | Source/Utils/WarpXUtil.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/Source/Utils/WarpXUtil.cpp b/Source/Utils/WarpXUtil.cpp index be11eb5c5..9ba842830 100644 --- a/Source/Utils/WarpXUtil.cpp +++ b/Source/Utils/WarpXUtil.cpp @@ -288,7 +288,9 @@ int safeCastToInt(const amrex::Real x, const std::string& real_name) { std::string assert_msg; // (2.0*(numeric_limits<int>::max()/2+1)) converts numeric_limits<int>::max()+1 to a real ensuring accuracy to all digits // This accepts x = 2**31-1 but rejects 2**31. - if (x < (2.0*(std::numeric_limits<int>::max()/2+1))) { + using namespace amrex::literals; + constexpr amrex::Real max_range = (2.0_rt*static_cast<amrex::Real>(std::numeric_limits<int>::max()/2+1)); + if (x < max_range) { if (std::ceil(x) >= std::numeric_limits<int>::min()) { result = static_cast<int>(x); } else { |