Quent42340
/
OpenMiner
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[MenuWidget] Now using std::deque instead of std::vector. 

Fixed a crash when triggering fullscreen mode while in Settings -> Graphics
I was keeping a TextButton pointer, but only std::deque ensures pointer validity

This bug only appeared with Visual Studio 2019
master
Quentin Bazin 2020-08-01 17:04:09 +02:00
parent e4007f8085
commit ca27a8823c
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
source/client/gui/MenuWidget.hpp
View File

@ -27,6 +27,8 @@
#ifndef MENUWIDGET_HPP_
#define MENUWIDGET_HPP_
#include <deque>
#include "SliderWidget.hpp"
#include "TextButton.hpp"
@ -62,8 +64,8 @@ class MenuWidget : public Widget {
u16 m_width = 1;
u16 m_height = 1;
std::vector<std::pair<TextButton, gk::Vector2i>> m_buttons;
std::vector<std::pair<SliderWidget, gk::Vector2i>> m_sliders;
std::deque<std::pair<TextButton, gk::Vector2i>> m_buttons;
std::deque<std::pair<SliderWidget, gk::Vector2i>> m_sliders;
};
#endif // MENUWIDGET_HPP_