mirror of
https://github.com/XRPLF/rippled.git
synced 2026-06-03 08:46:46 +00:00
Address more nitpicky AI comments
This commit is contained in:
@@ -121,8 +121,7 @@ private:
|
||||
case MantissaScale::Large:
|
||||
return 1'000'000'000'000'000'000ULL;
|
||||
default:
|
||||
// Since this can never be called outside a non-constexpr
|
||||
// context, this throw assures that the build fails if an
|
||||
// If called in a constexpr context, this throw assures that the build fails if an
|
||||
// invalid scale is used.
|
||||
throw std::runtime_error("Unknown mantissa scale");
|
||||
}
|
||||
@@ -139,8 +138,7 @@ private:
|
||||
case MantissaScale::Large:
|
||||
return CuspRoundingFix::Enabled;
|
||||
default:
|
||||
// Since this can never be called outside a non-constexpr
|
||||
// context, this throw assures that the build fails if an
|
||||
// If called in a constexpr context, this throw assures that the build fails if an
|
||||
// invalid scale is used.
|
||||
throw std::runtime_error("Unknown mantissa scale");
|
||||
}
|
||||
|
||||
@@ -42,7 +42,7 @@ setCurrentTransactionRules(std::optional<Rules> r)
|
||||
// The improved accuracy that will come from enabling large
|
||||
// mantissas is a goal separate from SAV and LP. It was originally
|
||||
// only tied to those two amendments to avoid needing a new
|
||||
// amendment of it's own, and because they require that behavior.
|
||||
// amendment of its own, and because they require that behavior.
|
||||
// Because fixCleanup3_2_0 fixes a separate bug related to the large
|
||||
// mantissas, that can take precedence and activate the large
|
||||
// mantissas even in the absence of the other two amendments.
|
||||
|
||||
Reference in New Issue
Block a user