luanti-org/luanti
1
0
Fork 0
mirror of https://github.com/luanti-org/luanti.git synced 2025-10-31 07:25:22 +01:00
Code Activity

guiScalingFilter: Fix most memory leaks (#12256)

Browse Source 
Calls to the cache function ended up creating a new texture regardless whether
the texture is already cached.
This commit is contained in:
SmallJoker
2022-05-04 20:55:13 +02:00
committed by sfan5
parent 8b010c5a9f
commit b405985b80
2 changed files with 11 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/client/client.cpp
View File

@@ -334,6 +334,8 @@ Client::~Client()
// cleanup 3d model meshes on client shutdown // cleanup 3d model meshes on client shutdown
m_rendering_engine->cleanupMeshCache(); m_rendering_engine->cleanupMeshCache();
guiScalingCacheClear();
delete m_minimap; delete m_minimap;
m_minimap = nullptr; m_minimap = nullptr;

12
src/client/guiscalingfilter.cpp
View File

@@ -43,6 +43,10 @@ void guiScalingCache(const io::path &key, video::IVideoDriver *driver, video::II
{ {
if (!g_settings->getBool("gui_scaling_filter")) if (!g_settings->getBool("gui_scaling_filter"))
return; return;
if (g_imgCache.find(key) != g_imgCache.end())
return; // Already cached.
video::IImage *copied = driver->createImage(value->getColorFormat(), video::IImage *copied = driver->createImage(value->getColorFormat(),
value->getDimension()); value->getDimension());
value->copyTo(copied); value->copyTo(copied);
@@ -90,14 +94,16 @@ video::ITexture *guiScalingResizeCached(video::IVideoDriver *driver,
io::path scalename = origname + "@guiScalingFilter:" + rectstr; io::path scalename = origname + "@guiScalingFilter:" + rectstr;
// Search for existing scaled texture. // Search for existing scaled texture.
video::ITexture *scaled = g_txrCache[scalename]; auto it_txr = g_txrCache.find(scalename);
video::ITexture *scaled = (it_txr != g_txrCache.end()) ? it_txr->second : nullptr;
if (scaled) if (scaled)
return scaled; return scaled;
// Try to find the texture converted to an image in the cache. // Try to find the texture converted to an image in the cache.
// If the image was not found, try to extract it from the texture. // If the image was not found, try to extract it from the texture.
video::IImage* srcimg = g_imgCache[origname]; auto it_img = g_imgCache.find(origname);
if (srcimg == NULL) { video::IImage *srcimg = (it_img != g_imgCache.end()) ? it_img->second : nullptr;
if (!srcimg) {
if (!g_settings->getBool("gui_scaling_filter_txr2img")) if (!g_settings->getBool("gui_scaling_filter_txr2img"))
return src; return src;
srcimg = driver->createImageFromData(src->getColorFormat(), srcimg = driver->createImageFromData(src->getColorFormat(),