Split ACTLIKEBRIDGE behaviors into two flags?

Post a reply

Smilies
:D :) :( :o :shock: :? 8-) :lol: :x :P :oops: :cry: :evil: :twisted: :roll: :wink: :geek: :ugeek: :!: :?: :idea: :arrow: :| :mrgreen: :3: :wub: >:( :blergh:
View more smilies

BBCode is ON
[img] is OFF
[url] is ON
Smilies are ON

Topic review
   

Expand view Topic review: Split ACTLIKEBRIDGE behaviors into two flags?

Split ACTLIKEBRIDGE behaviors into two flags?

by Caligari87 » Thu Aug 04, 2022 1:03 pm

Doing some code-diving to help Fishy with the 3D Platform Actor, it seems like the ACTLIKEBRIDGE flag controls two behaviors: A) Letting actors walk on and/or pass the actor, and B) not shifting in relation to floor/ceiling changes.

It would be useful to split these two behaviors up into separate flags, such as DONTMOVEWITHSECTOR and CANWALKON (or something, names can be bikeshedded over of course).

From what I can tell, all the behavior of this flag is controlled in p_map.cpp and would be pretty straightforward to split. Would a PR be accepted if I did so? The main drawback I can see would be adding yet more actor flags, but I'm not sure there's any way to replicate either desired behavior individually in ZScript.

8-)

Top