Merge pull request #10773 from NREL/CustomMeterFix

Fix Meter:Custom with mix of valid and invalid names
NREL · Sep 30, 2024 · b422061 · b422061 · nrel-bot-2c · Sep 30, 2024
2 parents 76fb9f8 + 93b50a9
commit b422061
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/src/EnergyPlus/OutputProcessor.cc b/src/EnergyPlus/OutputProcessor.cc
@@ -803,7 +803,7 @@ namespace OutputProcessor {
 
                     // Has to be a summed variable
                     if (srcDDVar->storeType != StoreType::Sum) {
-                        ShowWarningCustomMessage(state,
+                        ShowWarningCustomMessage(state, // Is clang-format formatting things like this? This is gross.
                                                  eoh,
                                                  format(R"(Meter:Custom="{}", variable not summed variable {}="{}".)",
                                                         ipsc->cAlphaArgs(1),
@@ -867,8 +867,7 @@ namespace OutputProcessor {
                                             ipsc->cAlphaFieldNames(fldIndex + 1),
                                             ipsc->cAlphaArgs(fldIndex + 1)));
                     ShowContinueError(state, "...will not be shown with the Meter results.");
-                    foundBadSrc = true;
-                    break;
+                    // Not setting the foundBadSrc flag here.
                 }
 
             } // for (fldIndex)