Skip to content

Commit

Permalink
Warning fix, CMake formatting fix
Browse files Browse the repository at this point in the history
  • Loading branch information
lefticus committed Sep 2, 2020
1 parent 8b26e0d commit 335d201
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions src/EnergyPlus/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -384,8 +384,8 @@ SET( SRC
OutAirNodeManager.hh
OutdoorAirUnit.cc
OutdoorAirUnit.hh
IOFiles.cc
IOFiles.hh
IOFiles.cc
IOFiles.hh
OutputProcessor.cc
OutputProcessor.hh
OutputReportPredefined.cc
Expand Down
4 changes: 2 additions & 2 deletions src/EnergyPlus/StringUtilities.hh
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ template <typename... Param> bool readList(std::string input, Param &&... param)
// to do make this a C++17 fold expression when possible

auto reader = stringReader(std::move(input));
std::initializer_list<bool>{readListItem(reader, std::forward<Param>(param))...};
(void)std::initializer_list<bool>{readListItem(reader, std::forward<Param>(param))...};
return !reader.fail();
}

Expand All @@ -105,4 +105,4 @@ template <typename T> std::istream &operator>>(std::istream &stream, Array1S<T>
}
} // namespace ObjexxFCL

#endif
#endif

7 comments on commit 335d201

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

finish_removing_gio (lefticus) - x86_64-MacOS-10.15-clang-11.0.0: OK (2924 of 2926 tests passed, 0 test warnings)

Failures:\n

integration Test Summary

  • Passed: 716
  • Failed: 2

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.

finish_removing_gio (lefticus) - 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.

finish_removing_gio (lefticus) - x86_64-Linux-Ubuntu-18.04-custom_check: OK (13 of 13 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.

finish_removing_gio (lefticus) - x86_64-Linux-Ubuntu-18.04-gcc-7.5: OK (2966 of 2966 tests passed, 0 test warnings)

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.

finish_removing_gio (lefticus) - x86_64-Linux-Ubuntu-18.04-gcc-7.5-UnitTestsCoverage-Debug: OK (1492 of 1492 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.

finish_removing_gio (lefticus) - Win64-Windows-10-VisualStudio-16: OK (2185 of 2185 tests passed, 0 test warnings)

Build Badge Test 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.

finish_removing_gio (lefticus) - x86_64-Linux-Ubuntu-18.04-gcc-7.5-IntegrationCoverage-Debug: OK (721 of 721 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

Please sign in to comment.