Boom linetype 166 is S1 - Ceiling raise to highest ceiling. It is not a texture change type. ATM the affected sector is changing sector flat. See link for test wad. Note what happens to sector 2 when linedef 10 is activated.
http://www.speedyshare.com/files/25115089/test.zip
Problem with Boom linetype 166
Moderator: GZDoom Developers
Forum rules
Please don't bump threads here if you have a problem - it will often be forgotten about if you do. Instead, make a new thread here.
Please don't bump threads here if you have a problem - it will often be forgotten about if you do. Instead, make a new thread here.
- Graf Zahl
- Lead GZDoom+Raze Developer
- Posts: 49067
- Joined: Sat Jul 19, 2003 10:19 am
- Location: Germany
Re: Problem with Boom linetype 166
Oh god, what did the Boom guys do? That type is supposed to move both ceiling and floor but in Boom it only moves the floor if the ceiling is blocked. So in 99.9% of all cases it only moves the ceiling. The definition of this type clearly looks broken. Here's the relevant code from PrBoom:
This looks ok on first sight but if you think about it, the floor action only starts when the ceiling action fails...
ZDoom always moves both.
Code: Select all
case 166:
// Raise ceiling, Lower floor
// 166 S1 EV_DoCeiling(raiseToHighest), EV_DoFloor(lowerFloortoLowest)
if (EV_DoCeiling(line, raiseToHighest) ||
EV_DoFloor(line, lowerFloorToLowest))
P_ChangeSwitchTexture(line,0);
break;
ZDoom always moves both.
- Graf Zahl
- Lead GZDoom+Raze Developer
- Posts: 49067
- Joined: Sat Jul 19, 2003 10:19 am
- Location: Germany
Re: Problem with Boom linetype 166
Fixed - or if we want to be precise - un-fixed.
ZDoom did what Boom wanted to implement but obviously in this case compatibility comes first so it's now as broken as Boom.
ZDoom did what Boom wanted to implement but obviously in this case compatibility comes first so it's now as broken as Boom.
Re: Problem with Boom linetype 166
From Boomref.txt ...
---------------------------------
Section 3.3 Ceiling linedef types
---------------------------------
Regular and Extended Ceiling Types
-------------------------------------------------------------------
# Class Trig Dir Spd *Chg *Mdl Mon Crsh Target
166 Ext S1 Up Slow None -- No No Highest Neighbor Ceiling
As all can see, flat texture does not change ... maybe prboom introduced the bug ?
---------------------------------
Section 3.3 Ceiling linedef types
---------------------------------
Regular and Extended Ceiling Types
-------------------------------------------------------------------
# Class Trig Dir Spd *Chg *Mdl Mon Crsh Target
166 Ext S1 Up Slow None -- No No Highest Neighbor Ceiling
As all can see, flat texture does not change ... maybe prboom introduced the bug ?
- Graf Zahl
- Lead GZDoom+Raze Developer
- Posts: 49067
- Joined: Sat Jul 19, 2003 10:19 am
- Location: Germany
Re: Problem with Boom linetype 166
No, that code is precisely the same in Boom 2.02. And it doesn't really matter what Boomref.txt says. The important thing is what the code does.
And there was no texture change. The floor just moved below the surrounding sector so that it got covered by flat bleeding.
And there was no texture change. The floor just moved below the surrounding sector so that it got covered by flat bleeding.