Page 1 of 1

Rail Puff Support for Floors and Ceilings

Posted: Thu Jan 14, 2016 11:24 am
by Major Cooke
Pull Request

A simple change, this adds the ability for puffs to spawn on floors and ceilings if the puffs have ALWAYSPUFF specified. In a way, this is technically a bugfix and a feature.

I went this route and added support for the flag because it was either this, or expanding bullet attack functionalities with the abilities to provide offsets from the shooter which no one can really see without a beam. This I feel was much simpler.

Re: Rail Puff Support for Floors and Ceilings

Posted: Sat Jan 16, 2016 2:05 pm
by Nash
This broke my line traces (spawned with LineAttack) that rely on the puffs automatically disappearing when it hits sky ceilings and floors (and also required +ALWAYSPUFF so that they collide with live actors). Please revert or add an option for your new behavior.

Re: Rail Puff Support for Floors and Ceilings

Posted: Sat Jan 16, 2016 2:16 pm
by Graf Zahl
How can this affect your traces spawned with P_LineAttack, when the change is in P_RailAttack?

That said, it should definitely have a sky check added.

Re: Rail Puff Support for Floors and Ceilings

Posted: Sat Jan 16, 2016 2:31 pm
by Major Cooke
Better yet, a flag to allow the +ALWAYSPUFF flag. Not just skies.

Re: Rail Puff Support for Floors and Ceilings

Posted: Sat Jan 16, 2016 2:43 pm
by Graf Zahl
Skies should be off by default, unless the SKYEXPLODE flag is specified. No more needless options.

Re: Rail Puff Support for Floors and Ceilings

Posted: Sat Jan 16, 2016 4:59 pm
by Major Cooke