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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MIRROR] Fixes oversight with DROP_BRAIN #2205

Merged
merged 2 commits into from
Mar 1, 2024
Merged

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#1215
Original PR: tgstation/tgstation#81735

About The Pull Request

Brains might not always be located in the head, e.g. HARS puts it in the chest, thus making the assumption that the brain will always be in the head in this conditional false.

image

Also fixes this compiler warning

cmd_nLK3wfCmVx

Why It's Good For The Game

Fixes a bug/oversight of a niche edge case.

Changelog

🆑 vinylspiders
fix: fixes an issue where being gibbed while under the HARS mutation can sometimes lead to the brain being deleted when it's not supposed to be
/:cl:

* Fixes oversight with DROP_BRAIN

* Update death.dm

---------

Co-authored-by: Bloop <[email protected]>
@Steals-The-PRs Steals-The-PRs added Mirroring conflict git cherry-pick во что-то уткнулся. Не ставить вручную, только для бота TG Mirror labels Feb 29, 2024
@Iajret Iajret merged commit a95888e into master Mar 1, 2024
23 checks passed
@Iajret Iajret deleted the upstream-mirror-1215 branch March 1, 2024 14:04
AnywayFarus added a commit that referenced this pull request Mar 1, 2024
Iajret pushed a commit that referenced this pull request Apr 27, 2024
* fixes jps mobs occasioanlly pathing forever (#82874)

## About The Pull Request
if MOVELOOP_NOT_READY gets passed once through the signal (which can be
often), the failure increments gets reset back to 0 and starts counting
again. this can cause the increments to never reach the max failure
threshold so itll keep checking forever. i saw some mobs get stuck
trying to reach an unreachable target because of this

## Why It's Good For The Game
fixes mobs getting stuck trying to reach something unreachable

## Changelog
:cl:
fix: fixes mobs getting stuck trying to reach something unreachable
/:cl:

* fixes jps mobs occasioanlly pathing forever

---------

Co-authored-by: Ben10Omintrix <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Mirroring conflict git cherry-pick во что-то уткнулся. Не ставить вручную, только для бота TG Mirror
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants