Eliminate repeated allocation of return nodes

NREL · Jan 23, 2019 · 0ac3fd3 · 0ac3fd3 · nmerket · Feb 7, 2019
1 parent f0a4a37
commit 0ac3fd3
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/src/EnergyPlus/ZoneEquipmentManager.cc b/src/EnergyPlus/ZoneEquipmentManager.cc
@@ -4943,14 +4943,24 @@ namespace ZoneEquipmentManager {
                              Real64 &FinalTotalReturnMassFlow // Final total return air mass flow rate
     )
     {
+        static Array1D_bool fixedReturn;
+        static bool AllocateFlag = true;
+        if (AllocateFlag) {
+            AllocateFlag = false;
+            int maxReturnNodes = 0;
+            for (auto & z : ZoneEquipConfig) {
+                maxReturnNodes = max(maxReturnNodes, z.NumReturnNodes);
+            }
+            fixedReturn.allocate(maxReturnNodes);
+        }
         auto &thisZoneEquip(ZoneEquipConfig(ZoneNum));
         int numRetNodes = thisZoneEquip.NumReturnNodes;
         Real64 totReturnFlow = 0.0; // Total flow to all return nodes in the zone (kg/s)
         Real64 totVarReturnFlow =
             0.0; // Total variable return flow, for return nodes connected to an airloop with an OA system or not with specified flow (kg/s)
         Real64 returnSchedFrac = ScheduleManager::GetCurrentScheduleValue(thisZoneEquip.ReturnFlowSchedPtrNum);
-        Array1D_bool fixedReturn; // If true, this return flow may not be adjusted
-        fixedReturn.allocate(numRetNodes);
+//        Array1D_bool fixedReturn; // If true, this return flow may not be adjusted
+//        fixedReturn.allocate(numRetNodes);
         fixedReturn = false;
         FinalTotalReturnMassFlow = 0.0;