[lmi-commits] [lmi] master 8ca13877 1/5: Remove a redundant condition

[Greg Chicares]

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;