mirrors/OpenRCT2
1
0
Fork 0
mirror of https://github.com/OpenRCT2/OpenRCT2 synced 2026-01-24 15:24:30 +01:00
Code Issues Releases Wiki Activity

Fix sprite culling glitching by culling sprites at full column width

Browse Source
This commit is contained in:
mix
2025-04-27 02:16:58 +01:00
parent 3d1c2f4e29
commit b5c925da36
1 changed files with 16 additions and 33 deletions
Download Patch File Download Diff File Expand all files Collapse all files

49
src/openrct2/paint/Paint.cpp
View File

@@ -103,40 +103,23 @@ static void PaintSessionAddPSToQuadrant(PaintSession& session, PaintStruct* ps)
session.QuadrantFrontIndex = std::max(session.QuadrantFrontIndex, paintQuadrantIndex);
}
static constexpr bool ImageWithinDPI(const ScreenCoordsXY& imagePos, const G1Element& g1, const DrawPixelInfo& dpi)
static constexpr bool imageWithinColumn(const ScreenCoordsXY& imagePos, const G1Element& g1, const DrawPixelInfo& dpi)
{
int32_t left = imagePos.x + g1.x_offset;
int32_t bottom = imagePos.y + g1.y_offset;
const int32_t left = imagePos.x + g1.x_offset;
const int32_t right = left + g1.width;
int32_t right = left + g1.width;
int32_t top = bottom + g1.height;
const int32_t columnWidth = dpi.zoom_level.ApplyInversedTo(kCoordsXYStep);
const int32_t alignedX = floor2(dpi.x, columnWidth);
// check if a sprite is within the full unclipped column width
// culling sprites outside the clipped column causes sorting differences between invalidation blocks
// not culling sprites outside the full column width also causes a different kind of glitching
if (dpi.zoom_level.ApplyInversedTo(right) <= alignedX)
return false;
if (dpi.zoom_level.ApplyInversedTo(left) >= alignedX + columnWidth)
return false;
// mber: It is possible to use only the bottom else block here if you change <= and >= to simply < and >.
// However, since this is used to cull paint structs, I'd prefer to keep the condition strict and calculate
// the culling differently for minifying and magnifying.
auto zoom = dpi.zoom_level;
if (zoom > ZoomLevel{ 0 })
{
if (right <= dpi.WorldX())
return false;
if (top <= dpi.WorldY())
return false;
if (left >= dpi.WorldX() + dpi.WorldWidth())
return false;
if (bottom >= dpi.WorldY() + dpi.WorldHeight())
return false;
}
else
{
if (zoom.ApplyInversedTo(right) <= dpi.x)
return false;
if (zoom.ApplyInversedTo(top) <= dpi.y)
return false;
if (zoom.ApplyInversedTo(left) >= dpi.x + dpi.width)
return false;
if (zoom.ApplyInversedTo(bottom) >= dpi.y + dpi.height)
return false;
}
return true;
}
@@ -184,7 +167,7 @@ static PaintStruct* CreateNormalPaintStruct(
const auto imagePos = Translate3DTo2DWithZ(session.CurrentRotation, swappedRotCoord);
if (!ImageWithinDPI(imagePos, *g1, session.DPI))
if (!imageWithinColumn(imagePos, *g1, session.DPI))
{
return nullptr;
}
@@ -232,7 +215,7 @@ static PaintStruct* CreateNormalPaintStructHeight(
const auto imagePos = Translate3DTo2DWithZ(session.CurrentRotation, swappedRotCoord);
if (!ImageWithinDPI(imagePos, *g1, session.DPI))
if (!imageWithinColumn(imagePos, *g1, session.DPI))
{
return nullptr;
}