Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

some clang-tidy 19 fixes #1664

Merged
merged 19 commits into from
Nov 25, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
e77907e
add a slightly more extensive nova network
zingale Aug 27, 2021
6ae3dcf
Merge branch 'development' of ssh://github.com/StarKiller-astro/Micro…
zingale Oct 6, 2021
1e3b012
Merge branch 'development' of ssh://github.com/StarKiller-astro/Micro…
zingale Oct 13, 2021
cf0a29c
Merge branch 'development' of ssh://github.com/StarKiller-astro/Micro…
zingale Feb 16, 2022
e4fbb2c
Merge branch 'development' of ssh://github.com/StarKiller-astro/Micro…
zingale Apr 7, 2022
645baad
Merge branch 'development' of ssh://github.com/StarKiller-astro/Micro…
zingale Aug 22, 2022
b754391
Merge branch 'development' of ssh://github.com/StarKiller-astro/Micro…
zingale Feb 23, 2023
d398920
Merge branch 'development' of ssh://github.com/StarKiller-astro/Micro…
zingale Jun 1, 2023
991b355
Merge branch 'development' of ssh://github.com/StarKiller-astro/Micro…
zingale Jun 9, 2023
03de257
Merge branch 'development' of ssh://github.com/StarKiller-astro/Micro…
zingale Nov 10, 2023
6a8ade9
Merge branch 'development' of ssh://github.com/StarKiller-astro/Micro…
zingale Nov 17, 2023
14be14d
Merge branch 'development' of ssh://github.com/StarKiller-astro/Micro…
zingale Dec 1, 2023
86cc226
Merge branch 'development' of ssh://github.com/StarKiller-astro/Micro…
zingale Apr 23, 2024
1be585f
Merge branch 'development' of ssh://github.com/StarKiller-astro/Micro…
zingale Sep 30, 2024
9a6d8b8
Merge branch 'development' of github.com:amrex-astro/Microphysics int…
zingale Nov 13, 2024
894094e
some clang-tidy 19 fixes
zingale Nov 13, 2024
d567749
fix spelling
zingale Nov 13, 2024
6d53f9c
Merge branch 'development' into clang-tidy-19-again
zingale Nov 14, 2024
9b3e9e6
Merge branch 'development' into clang-tidy-19-again
zingale Nov 20, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions .clang-tidy
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ Checks: >
-misc-include-cleaner,
-misc-non-private-member-variables-in-classes,
-misc-use-anonymous-namespace,
-misc-use-internal-linkage,
modernize-*,
-modernize-avoid-c-arrays,
-modernize-use-trailing-return-type,
Expand All @@ -29,12 +30,14 @@ Checks: >
-readability-avoid-const-params-in-decls,
-readability-braces-around-statements,
-readability-else-after-return,
-readability-enum-initial-value,
-readability-function-cognitive-complexity,
-readability-function-size,
-readability-identifier-length,
-readability-implicit-bool-conversion,
-readability-isolate-declaration,
-readability-magic-numbers,
-readability-math-missing-parentheses,
-readability-named-parameter,
-readability-simplify-boolean-expr,
mpi-*,
Expand Down
10 changes: 3 additions & 7 deletions integration/VODE/vode_dvhin.H
Original file line number Diff line number Diff line change
Expand Up @@ -122,17 +122,13 @@ void dvhin (BurnT& state, DvodeT& vstate, amrex::Real& H0, int& NITER, int& IER)

}

// Iteration done. Apply bounds, bias factor, and sign. Then exit.
// Iteration done. Apply bounds and bias factor. Then exit.
H0 = hnew * 0.5_rt;
if (H0 < HLB) {
H0 = HLB;
}
if (H0 > HUB) {
H0 = HUB;
}
H0 = std::clamp(H0, HLB, HUB);

}

// apply sign
H0 = std::copysign(H0, vstate.tout - vstate.t);
NITER = iter;
IER = 0;
Expand Down
2 changes: 1 addition & 1 deletion networks/aprox13/actual_network.H
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,7 @@ namespace NSE_INDEX
#endif

namespace Rates {
enum NetworkRates {
enum NetworkRates: std::uint8_t {
He4_He4_He4_to_C12 = 1,
C12_He4_to_O16,
C12_C12_to_Ne20_He4,
Expand Down
2 changes: 1 addition & 1 deletion networks/aprox21/actual_network.H
Original file line number Diff line number Diff line change
Expand Up @@ -152,7 +152,7 @@ namespace NSE_INDEX
#endif

namespace Rates {
enum NetworkRates {
enum NetworkRates : std::uint8_t {
H1_H1_to_He3 = 1,
H1_H1_H1_to_He3,
P_to_N,
Expand Down
2 changes: 1 addition & 1 deletion networks/iso7/actual_network.H
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ namespace NSE_INDEX
#endif

namespace Rates {
enum NetworkRates {
enum NetworkRates : std::uint8_t {
C12_He4_to_O16 = 1,
He4_He4_He4_to_C12,
C12_C12_to_Ne20_He4,
Expand Down
2 changes: 1 addition & 1 deletion networks/rprox/actual_network.H
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,7 @@ namespace network

namespace Rates
{
enum NetworkRates
enum NetworkRates : std::uint8_t
{
He4_He4_He4_to_C12 = 1,
C12_H1_to_N13,
Expand Down
Loading