Mirrorlandia_minetest/minetest
1
0
Fork 1
mirror of https://github.com/minetest/minetest.git synced 2025-04-02 08:22:56 +02:00
Code Issues Packages Projects Releases Wiki Activity

Remove ugly hack in static_text.h

Browse Source 
Just use the root element, like GUIButton:add().
This commit is contained in:
Desour
2023-08-09 01:55:14 +02:00
committed by sfan5
parent f7f3aaf43c
commit d75c956dbc
1 changed files with 1 additions and 21 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
src/irrlicht_changes/static_text.h
View File

@ -49,27 +49,7 @@ namespace gui
s32 id = -1, s32 id = -1,
bool fillBackground = false) bool fillBackground = false)
{ {
if (parent == NULL) { parent = parent ? parent : guienv->getRootGUIElement();
// parent is NULL, so we must find one, or we need not to drop
// result, but then there will be a memory leak.
//
// What Irrlicht does is to use guienv as a parent, but the problem
// is that guienv is here only an IGUIEnvironment, while it is a
// CGUIEnvironment in Irrlicht, which inherits from both IGUIElement
// and IGUIEnvironment.
//
// A solution would be to dynamic_cast guienv to a
// IGUIElement*, but Irrlicht is shipped without rtti support
// in some distributions, causing the dymanic_cast to segfault.
//
// Thus, to find the parent, we create a dummy StaticText and ask
// for its parent, and then remove it.
irr::gui::IGUIStaticText *dummy_text =
guienv->addStaticText(L"", rectangle, border, wordWrap,
parent, id, fillBackground);
parent = dummy_text->getParent();
dummy_text->remove();
}
irr::gui::IGUIStaticText *result = new irr::gui::StaticText( irr::gui::IGUIStaticText *result = new irr::gui::StaticText(
text, border, guienv, parent, text, border, guienv, parent,
id, rectangle, fillBackground); id, rectangle, fillBackground);