Studio: Make _cyclicCheck a const member function

This doesn't actually modify the internal state of the studio instance,
so we can mark it as a const member function.
This commit is contained in:
Lioncash 2019-09-10 20:20:17 -04:00
parent 6c07ec907a
commit 04fbc328e9
2 changed files with 8 additions and 8 deletions

View File

@ -18,7 +18,7 @@ class Studio {
std::list<StudioSend> m_studiosOut; std::list<StudioSend> m_studiosOut;
#ifndef NDEBUG #ifndef NDEBUG
bool _cyclicCheck(Studio* leaf); bool _cyclicCheck(const Studio* leaf) const;
#endif #endif
public: public:

View File

@ -1,16 +1,16 @@
#include "amuse/Studio.hpp" #include "amuse/Studio.hpp"
#include <algorithm>
#include "amuse/Engine.hpp" #include "amuse/Engine.hpp"
namespace amuse { namespace amuse {
#ifndef NDEBUG #ifndef NDEBUG
bool Studio::_cyclicCheck(Studio* leaf) { bool Studio::_cyclicCheck(const Studio* leaf) const {
for (auto it = m_studiosOut.begin(); it != m_studiosOut.end();) { return std::any_of(m_studiosOut.cbegin(), m_studiosOut.cend(), [leaf](const auto& studio) {
if (leaf == it->m_targetStudio.get() || it->m_targetStudio->_cyclicCheck(leaf)) return leaf == studio.m_targetStudio.get() || studio.m_targetStudio->_cyclicCheck(leaf);
return true; });
++it;
}
return false;
} }
#endif #endif