Skip to content

Commit

Permalink
revert a few consts
Browse files Browse the repository at this point in the history
  • Loading branch information
rraustad committed Dec 26, 2024
1 parent e2e3930 commit dad937a
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
6 changes: 3 additions & 3 deletions src/EnergyPlus/PlantPipingSystemsManager.cc
Original file line number Diff line number Diff line change
Expand Up @@ -431,22 +431,22 @@ namespace PlantPipingSystemsManager {
auto const &thisDomain = state.dataPlantPipingSysMgr->domains[DomainNum];

// validate pipe domain-circuit name-to-index references
for (auto const &thisCircuit : thisDomain.circuits) {
for (auto &thisCircuit : thisDomain.circuits) {
thisCircuit->ParentDomainIndex = DomainNum;
}

// correct segment locations for: INTERNAL DATA STRUCTURE Y VALUE MEASURED FROM BOTTOM OF DOMAIN,
// INPUT WAS MEASURED FROM GROUND SURFACE
for (auto const &thisCircuit : thisDomain.circuits) {
for (auto const &thisSegment : thisCircuit->pipeSegments) {
for (auto &thisSegment : thisCircuit->pipeSegments) {
thisSegment->PipeLocation.Y = thisDomain.Extents.yMax - thisSegment->PipeLocation.Y;
}
}

// correct segment locations for: BASEMENT X SHIFT
if (thisDomain.HasBasement && thisDomain.BasementZone.ShiftPipesByWidth) {
for (auto const &thisCircuit : thisDomain.circuits) {
for (auto const &thisSegment : thisCircuit->pipeSegments) {
for (auto &thisSegment : thisCircuit->pipeSegments) {
thisSegment->PipeLocation.X += thisDomain.BasementZone.Width;
}
}
Expand Down
2 changes: 1 addition & 1 deletion src/EnergyPlus/ReportCoilSelection.cc
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@

namespace EnergyPlus {

void createCoilSelectionReportObj(EnergyPlusData const &state)
void createCoilSelectionReportObj(EnergyPlusData &state)
{
state.dataRptCoilSelection->coilSelectionReportObj = std::make_unique<ReportCoilSelection>();
}
Expand Down
2 changes: 1 addition & 1 deletion src/EnergyPlus/ReportCoilSelection.hh
Original file line number Diff line number Diff line change
Expand Up @@ -449,7 +449,7 @@ public: // data

}; // end ReportCoilSelection class

void createCoilSelectionReportObj(EnergyPlusData const &state);
void createCoilSelectionReportObj(EnergyPlusData &state);

struct ReportCoilSelectionData : BaseGlobalStruct
{
Expand Down

3 comments on commit dad937a

@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.

CppCheck-consts (rraustad) - x86_64-Linux-Ubuntu-24.04-gcc-13.3: OK (2918 of 2918 tests passed, 0 test warnings)

Build Badge Test Badge

@nrel-bot-2
Copy link

Choose a reason for hiding this comment

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

CppCheck-consts (rraustad) - x86_64-Linux-Ubuntu-24.04-gcc-13.3-UnitTestsCoverage-RelWithDebInfo: OK (2064 of 2100 tests passed, 0 test warnings)

Failures:\n

API Test Summary

  • Failed: 11
  • notrun: 5

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.

CppCheck-consts (rraustad) - x86_64-Linux-Ubuntu-24.04-gcc-13.3-IntegrationCoverage-RelWithDebInfo: OK (801 of 801 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

Please sign in to comment.