Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Long actions (like resisting out of handcuffs) will no longer count space drifting as moving. #14187

Merged
merged 3 commits into from Jan 1, 2016

Conversation

MrStonedOne
Copy link
Member

So I was looking into atom/movable/Move() being high up on the profile when I stumbled upon inertia_dir and had ideas.

Everything is SUPPOSE to set this to 0 when they stop space moving. I'm just gonna trust that to be the case (it is in base atom/movable/newtonian_move.

I don't want to keep calling Process_Spacemove in the loop because that feels dirty.
fixes #10235
馃啈
tweak: Long actions (like resisting out of handcuffs) will no longer count space/nograv drifting as the user moving.
tweak: This does not apply to the target of a long action if that target isn't you. Pulling something while space drifting and working on it intentionally won't work.
/:cl:

So I was looking into atom/movable/Move() being high up on the profile when I stumbled upon inertia_dir and had ideas.

Everything is SUPPOSE to set this to 0 when they stop space moving. I'm just gonna trust that to be the case (it is in base atom/movable/newtonian_move.

I don't want to keep calling Process_Spacemove in the loop because that feels dirty.
drifting = 0
Uloc = user.loc

if(!user || user.stat || user.weakened || user.stunned || (!drifting && user.loc != Uloc)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

forgot a )

@ChangelingRain
Copy link
Contributor

Changeling buff/10

@Bawhoppen
Copy link
Contributor

its all relative brah

@WJohn WJohn added the Fix Rewrites a bug so it appears in different circumstances label Dec 31, 2015
tkdrg added a commit that referenced this pull request Jan 1, 2016
Long actions (like resisting out of handcuffs) will no longer count space drifting as moving.
@tkdrg tkdrg merged commit 083a98d into tgstation:master Jan 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Rewrites a bug so it appears in different circumstances
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot complete long actions in space
5 participants