OrdinaryMagician/vkdoom_m
OrdinaryMagician/vkdoom_m
1
Fork 0
Code Activity

- remove SetLineSideRefs - this actually cannot be exported right now, due to the fact that both sides and lines have backreferences to themselves and their ancestry, and that linedefs have forward references to the sectors on their respective sides.

Browse source 
- made FlipLineSideRefs native, due to the SetLineSideRefs removal
- fixed a bug with FlipLineSideRefs that rendered upper and lower textures incorrectly due to incorrect sector references
- FlipLineSideRefs now should only work on single-sided lines
This commit is contained in:
Rachael Alexanderson 2019-11-14 14:23:53 -05:00
commit 4d66e9a8bb
2 changed files with 14 additions and 18 deletions
Download patch file Download diff file Expand all files Collapse all files

23
src/maploader/postprocessor.cpp
View file

@ -459,23 +459,26 @@ DEFINE_ACTION_FUNCTION(DLevelPostProcessor, SetLineVertexes)
return 0;
}
DEFINE_ACTION_FUNCTION(DLevelPostProcessor, SetLineSideRefs)
DEFINE_ACTION_FUNCTION(DLevelPostProcessor, FlipLineSideRefs)
{
PARAM_SELF_PROLOGUE(DLevelPostProcessor);
PARAM_UINT(lineidx);
PARAM_UINT(sideidx1);
PARAM_UINT(sideidx2);
if (lineidx < self->Level->lines.Size() &&
sideidx1 < self->Level->sides.Size() &&
sideidx2 < self->Level->sides.Size())
if (lineidx < self->Level->lines.Size())
{
line_t *line = &self->Level->lines[lineidx];
side_t *side1 = &self->Level->sides[sideidx1];
side_t *side2 = &self->Level->sides[sideidx2];
side_t *side1 = line->sidedef[1];
side_t *side2 = line->sidedef[0];
line->sidedef[0] = side1;
line->sidedef[1] = side2;
if (!!side1 && !!side2) // don't flip single-sided lines
{
sector_t *frontsector = line->sidedef[1]->sector;
sector_t *backsector = line->sidedef[0]->sector;
line->sidedef[0] = side1;
line->sidedef[1] = side2;
line->frontsector = frontsector;
line->backsector = backsector;
}
}
self->loader->ForceNodeBuild = true;
return 0;