zdoom
/
raze-gles
mirror of https://github.com/ZDoom/raze-gles.git
0
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

- fixed ordering issues in drawer 

* do not check the clipper in the collection pass to reduce number of bunches. Clipping here brings no performance gain.
* fixed the loop reset in FindClosestBunch to actually process the array's first element.
This commit is contained in:
Christoph Oelckers 2021-03-23 19:20:06 +01:00
parent 0c7f05a416
commit 46e0b16370
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
source/core/rendering/scene/hw_bunchdrawer.cpp
View File

@ -40,7 +40,6 @@
#include "gamefuncs.h" #include "gamefuncs.h"
//========================================================================== //==========================================================================
// //
// //
@ -238,7 +237,6 @@ void BunchDrawer::ProcessBunch(int bnch)
SetupWall.Clock(); SetupWall.Clock();
HWWall hwwall; HWWall hwwall;
//Printf("Rendering wall %d\n", i);
hwwall.Process(di, &wall[i], &sector[bunch->sectnum], wall[i].nextsector < 0 ? nullptr : &sector[wall[i].nextsector]); hwwall.Process(di, &wall[i], &sector[bunch->sectnum], wall[i].nextsector < 0 ? nullptr : &sector[wall[i].nextsector]);
rendered_lines++; rendered_lines++;
@ -401,7 +399,7 @@ int BunchDrawer::FindClosestBunch()
closest = CompareData[i]; closest = CompareData[i];
CompareData[i] = CompareData.Last(); CompareData[i] = CompareData.Last();
CompareData.Pop(); CompareData.Pop();
i = 0; // we need to recheck everything that's still marked. i = -1; // we need to recheck everything that's still marked. -1 because this will get incremented before being used.
continue; continue;
case 1: // is behind case 1: // is behind
@ -459,11 +457,13 @@ void BunchDrawer::ProcessSector(int sectnum)
// Backside // Backside
inbunch = false; inbunch = false;
} }
/* disabled because it only fragments the bunches without any performance gain.
else if (!clipper->SafeCheckRange(ang1, ang2)) else if (!clipper->SafeCheckRange(ang1, ang2))
{ {
// is it visible? // is it visible?
inbunch = false; inbunch = false;
} }
*/
else if (!inbunch || ang2.asbam() - startangle.asbam() >= ANGLE_180) else if (!inbunch || ang2.asbam() - startangle.asbam() >= ANGLE_180)
{ {
// don't let a bunch span more than 180° to avoid problems. // don't let a bunch span more than 180° to avoid problems.