[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lmi-commits] [lmi] master 8ca13877 1/5: Remove a redundant condition
From: |
Greg Chicares |
Subject: |
[lmi-commits] [lmi] master 8ca13877 1/5: Remove a redundant condition |
Date: |
Fri, 2 Sep 2022 14:23:16 -0400 (EDT) |
branch: master
commit 8ca138772098a77133116bc831ba83a403a7f7ce
Author: Gregory W. Chicares <gchicares@sbcglobal.net>
Commit: Gregory W. Chicares <gchicares@sbcglobal.net>
Remove a redundant condition
Within this loop:
for(int j = 0; j < BasicValues::Length; ++j)
the middle expression is always true, and repeating it thus:
j < std::min(yare_input_.SolveEndYear, BasicValues::Length)
i.e.
j < yare_input_.SolveEndYear && j < BasicValues::Length
was redundant.
---
ihs_avstrtgy.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ihs_avstrtgy.cpp b/ihs_avstrtgy.cpp
index 57d2869f..8719a520 100644
--- a/ihs_avstrtgy.cpp
+++ b/ihs_avstrtgy.cpp
@@ -139,7 +139,7 @@ void AccountValue::PerformSpecAmtStrategy()
(
mce_solve_specamt == yare_input_.SolveType
&& yare_input_.SolveBeginYear <= j
- && j < std::min(yare_input_.SolveEndYear, BasicValues::Length)
+ && j < yare_input_.SolveEndYear
)
{
strategy = mce_sa_input_scalar;