zdoom/raze-gles
0
0
Fork 0
mirror of https://github.com/ZDoom/raze-gles.git synced 2024-12-28 12:30:46 +00:00
Code Issues Projects Releases Packages Wiki Activity

- delete lines in proper order when eliminating a pair.

Browse source 
Deleting the lower one first will delete the wrong second one.
This commit is contained in:
Christoph Oelckers 2021-05-23 16:36:06 +02:00
parent b2545f8042
commit 572f45ca53
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
source/core/sectorgeometry.cpp
View file

@ -385,8 +385,8 @@ bool SectorGeometry::MakeVertices2(unsigned int secnum, int plane, const FVector
{ {
// handle the simple case first, i.e. line j is the inverse of line i. // handle the simple case first, i.e. line j is the inverse of line i.
// in this case both lines need to be deleted. // in this case both lines need to be deleted.
lines.Delete(i);
lines.Delete(j); lines.Delete(j);
lines.Delete(i);
i--; i--;
goto nexti; goto nexti;
} }
@ -433,7 +433,7 @@ bool SectorGeometry::MakeVertices2(unsigned int secnum, int plane, const FVector
double pp1x = vert ? pp1.X : pp1.Y; double pp1x = vert ? pp1.X : pp1.Y;
double pp2x = vert ? pp2.X : pp2.Y; double pp2x = vert ? pp2.X : pp2.Y;
if (pp1x > min(p1x, p2x) && pp2x < max(p1x, p2x)) if (pp1x > min(p1x, p2x) && pp1x < max(p1x, p2x))
{ {
// pp1 is on line i. // pp1 is on line i.
lines[i].v1 = lines[j].v1; lines[i].v1 = lines[j].v1;