mctaylors/suyu
1
0
Fork 0
forked from suyu/suyu
Code Pull requests Activity

gl_rasterizer_cache: Avoid iterator invalidation issues within InvalidateRegion()

Browse source 
A range-based for loop can't be used when the container being iterated
is also being erased from.
This commit is contained in:
Lioncash 2018-08-09 15:24:17 -04:00
parent 0bfe974281
commit 59ea37daa7
1 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/video_core/renderer_opengl/gl_rasterizer_cache.cpp
View file

@ -761,10 +761,12 @@ void RasterizerCacheOpenGL::FlushRegion(Tegra::GPUVAddr /*addr*/, size_t /*size*
} }
void RasterizerCacheOpenGL::InvalidateRegion(Tegra::GPUVAddr addr, size_t size) { void RasterizerCacheOpenGL::InvalidateRegion(Tegra::GPUVAddr addr, size_t size) {
for (const auto& pair : surface_cache) { for (auto iter = surface_cache.cbegin(); iter != surface_cache.cend();) {
const auto& surface{pair.second}; const auto& surface{iter->second};
const auto& params{surface->GetSurfaceParams()}; const auto& params{surface->GetSurfaceParams()};
++iter;
if (params.IsOverlappingRegion(addr, size)) { if (params.IsOverlappingRegion(addr, size)) {
UnregisterSurface(surface); UnregisterSurface(surface);
} }