Instances of std::shared_ptr<Menu> can be replaced with std::optional<Menu>
opened this issue · comments
As debug state is clear owner of these, there is no need for std::shared_ptr.
Furthermore (with available now c++17) it's possible to avoid dynamic memory allocation and reduce overhead.