Fix #10899

The issue was that an if else ladder inside a loop was incorrectly refactored to a switch case:

This was the original thing
```c++
for (int i = ....) {
  SELECT_CASE_var = XXXX
  if (SELECT_CASE_var == blabla) {
    break; // THIS BREAKS THE FOR LOOP
   } else ...
}
```

The refactor:

```c++
for (int i = ....) {
  switch (XXX) {
  case blabla:
    break; // THIS DO NOT BREAK THE FOR LOOP, it just prevents other `case`s to be processed (fallthrough)
  }
  ...
 }
```

255e7e939fc?w=1#diff-79e778e07bedf79acf027aa76c1235badd89a6f3014b6f0cdadc4ada74402ae3R3817-R3827

src/EnergyPlus/OutputReportTabular.cc

@@ -3767,7 +3767,11 @@ void GatherMonthlyResultsForTimestep(EnergyPlusData &state, OutputProcessor::Tim
                 // If the hours variable is active then scan through the rest of the variables
                 // and accumulate
                 if (activeHoursShown) {
+                    bool exit_loop = false;
                     for (int kOtherColumn = jColumn + 1; kOtherColumn <= ort->MonthlyTables(iTable).numColumns; ++kOtherColumn) {
+                        if (exit_loop) {
+                            break;
+                        }
                         int const scanColumn = kOtherColumn + ort->MonthlyTables(iTable).firstColumn - 1;
                         OutputProcessor::VariableType const scanTypeOfVar = ort->MonthlyColumns(scanColumn).typeOfVar;
                         int const scanVarNum = ort->MonthlyColumns(scanColumn).varNum;
@@ -3781,6 +3785,7 @@ void GatherMonthlyResultsForTimestep(EnergyPlusData &state, OutputProcessor::Tim
                         case AggType::HoursNegative:
                         case AggType::HoursNonNegative:
                             // end scanning since these might reset
+                            exit_loop = true;
                             break; // do
                         case AggType::SumOrAverageHoursShown: {
                             // this case is when the value should be set