Skip to content

Commit

Permalink
Remove more unused variables
Browse files Browse the repository at this point in the history
  • Loading branch information
rraustad committed May 16, 2019
1 parent b2f4106 commit 5bd6ae1
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 7 deletions.
7 changes: 2 additions & 5 deletions src/EnergyPlus/HVACVariableRefrigerantFlow.cc
Original file line number Diff line number Diff line change
Expand Up @@ -8537,7 +8537,7 @@ namespace HVACVariableRefrigerantFlow {
for (int i = 1; i <= TerminalUnitList(TUListNum).NumTUInList; i++) {
int VRFTUNum = TerminalUnitList(TUListNum).ZoneTUPtr(i);
// analyze the conditions of each IU
VRFTU(VRFTUNum).CalcVRFIUVariableTeTc(VRFTUNum, EvapTemp(i), CondTemp(i));
VRFTU(VRFTUNum).CalcVRFIUVariableTeTc(EvapTemp(i), CondTemp(i));

// select the Te/Tc that can satisfy all the zones
IUMinEvapTemp = min(IUMinEvapTemp, EvapTemp(i), this->IUEvapTempHigh);
Expand All @@ -8554,8 +8554,7 @@ namespace HVACVariableRefrigerantFlow {
}
}

void VRFTerminalUnitEquipment::CalcVRFIUVariableTeTc(int const VRFTUNum, // Index to VRF terminal unit
Real64 &EvapTemp, // evaporating temperature
void VRFTerminalUnitEquipment::CalcVRFIUVariableTeTc(Real64 &EvapTemp, // evaporating temperature
Real64 &CondTemp // condensing temperature
)
{
Expand Down Expand Up @@ -8599,10 +8598,8 @@ namespace HVACVariableRefrigerantFlow {
// na

// SUBROUTINE LOCAL VARIABLE DECLARATIONS:
int const Mode(1); // Performance mode for MultiMode DX coil. Always 1 for other coil types
int CoolCoilNum; // index to the VRF Cooling DX coil to be simulated
int HeatCoilNum; // index to the VRF Heating DX coil to be simulated
int OAMixNode; // index to the mix node of OA mixer
int IndexToTUInTUList; // index to TU in specific list for the VRF system
int TUListIndex; // index to TU list for this VRF system
int VRFNum; // index to VRF that the VRF Terminal Unit serves
Expand Down
3 changes: 1 addition & 2 deletions src/EnergyPlus/HVACVariableRefrigerantFlow.hh
Original file line number Diff line number Diff line change
Expand Up @@ -729,8 +729,7 @@ namespace HVACVariableRefrigerantFlow {
// Note: the argument VRFTUNum should be removed later in the deeper OO re-factor. Now this argument may be used by other functions that are
// not member functions of this class.

void CalcVRFIUVariableTeTc(int const VRFTUNum, // Index to VRF terminal unit
Real64 &EvapTemp, // evaporating temperature
void CalcVRFIUVariableTeTc(Real64 &EvapTemp, // evaporating temperature
Real64 &CondTemp // condensing temperature
);

Expand Down

7 comments on commit 5bd6ae1

@nrel-bot-2b
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

6301-VRF-Terminal-unit-flow-balance-problems (rraustad) - x86_64-Linux-Ubuntu-18.04-cppcheck: OK (0 of 0 tests passed, 0 test warnings)

Build Badge

@nrel-bot-2b
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

6301-VRF-Terminal-unit-flow-balance-problems (rraustad) - x86_64-Linux-Ubuntu-18.04-custom_check: OK (6 of 6 tests passed, 0 test warnings)

Build Badge

@nrel-bot-2c
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

6301-VRF-Terminal-unit-flow-balance-problems (rraustad) - x86_64-Linux-Ubuntu-18.04-gcc-7.4: OK (2443 of 2446 tests passed, 0 test warnings)

Messages:\n

  • 3 tests had: EIO diffs.
  • 2 tests had: ERR diffs.
  • 3 tests had: ESO big diffs.
  • 3 tests had: MTR big diffs.
  • 3 tests had: Table big diffs.

Failures:\n

regression Test Summary

  • Passed: 672
  • Failed: 3

Build Badge Test Badge

@nrel-bot-3
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

6301-VRF-Terminal-unit-flow-balance-problems (rraustad) - x86_64-MacOS-10.13-clang: OK (2403 of 2406 tests passed, 0 test warnings)

Messages:\n

  • 3 tests had: EIO diffs.
  • 2 tests had: ERR diffs.
  • 3 tests had: ESO big diffs.
  • 3 tests had: MTR big diffs.
  • 3 tests had: Table big diffs.

Failures:\n

regression Test Summary

  • Passed: 652
  • Failed: 3

Build Badge Test Badge

@nrel-bot-2b
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

6301-VRF-Terminal-unit-flow-balance-problems (rraustad) - x86_64-Linux-Ubuntu-18.04-gcc-7.4-UnitTestsCoverage-Debug: OK (1094 of 1094 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

@nrel-bot-2c
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

6301-VRF-Terminal-unit-flow-balance-problems (rraustad) - x86_64-Linux-Ubuntu-18.04-gcc-7.4-IntegrationCoverage-Debug: OK (660 of 660 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

@nrel-bot
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

6301-VRF-Terminal-unit-flow-balance-problems (rraustad) - Win64-Windows-10-VisualStudio-16: OK (2403 of 2406 tests passed, 0 test warnings)

Messages:\n

  • 3 tests had: EIO diffs.
  • 2 tests had: ERR diffs.
  • 3 tests had: ESO big diffs.
  • 3 tests had: MTR big diffs.
  • 3 tests had: Table big diffs.

Failures:\n

regression Test Summary

  • Passed: 652
  • Failed: 3

Build Badge Test Badge

Please sign in to comment.