MAPINFO - checkswitchrange no longer accounts for 3D floors

Bugs that have been investigated and resolved somehow.

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.
Post Reply
User avatar
GooberMan
Posts: 1336
Joined: Fri Aug 08, 2003 12:57 am
Location: Helsinki, Finland

MAPINFO - checkswitchrange no longer accounts for 3D floors

Post by GooberMan »

Worked at the time of Prime Directive's release in June 2015.

Fairly early recognisable break with this behavior:
  • Console - map ge5map02
  • Press J to skip the cutscene
  • Run forward to the landing pad. Notice door on left of brick wall ahead of you. Run to it.
  • Attempt to use the door
The door is a 3D floor controlled by a script, which when used makes it look just like a real door. Notice that you can run on top of the brick wall where the door would be. checkswitchrange was enabled in MAPINFO to stop people pressing use and activating the door when they're obviously not next to it in a three-dimensional sense.

Bump behavior works just fine, but this is a Doom map not a Quake map. The bump constantly retriggers if you keep walking in to it though, so you get spammed use sounds and console message about the door needing a yellow key.
_mental_
 
 
Posts: 3812
Joined: Sun Aug 07, 2011 4:32 am

Re: MAPINFO - checkswitchrange no longer accounts for 3D flo

Post by _mental_ »

It was introduced with this pull request because of the change in this line. It seems to be a fix for similar issue which broke your map as a side effect :?
User avatar
Graf Zahl
Lead GZDoom+Raze Developer
Lead GZDoom+Raze Developer
Posts: 49056
Joined: Sat Jul 19, 2003 10:19 am
Location: Germany

Re: MAPINFO - checkswitchrange no longer accounts for 3D flo

Post by Graf Zahl »

To summarize, this appeared to have worked solely by happenstance, not by design - to be precise, it "worked" because the function was bugged. If you look at the code, it never performs any checks whatsoever on 3D floors.

The only possible solution here looks like adding some compatibility handler
User avatar
GooberMan
Posts: 1336
Joined: Fri Aug 08, 2003 12:57 am
Location: Helsinki, Finland

Re: MAPINFO - checkswitchrange no longer accounts for 3D flo

Post by GooberMan »

Compatibility + new MAPINFO flag would be preferred. I was testing PD because a colleague wants to play it, I'm inclined to do some more bugfixes while I'm there and upload a new version. If I recall, the compat system worked off file name and/or file hash? I'd rather preserve the behavior in any update I make.

EDIT: Checked the code, yeah, I see how that managed to just work since the player's Z equaled the sector Z. So tl;dr is that the MAPINFO flag would want to be the correct version that checks 3D floors in the back sector.
_mental_
 
 
Posts: 3812
Joined: Sun Aug 07, 2011 4:32 am

Re: MAPINFO - checkswitchrange no longer accounts for 3D flo

Post by _mental_ »

Fixed in eaf0ac1.
Post Reply

Return to “Closed Bugs [GZDoom]”