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] Humanizing monkeys removes undergarments #4140

Merged
merged 1 commit into from
Jun 30, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#3395
Original PR: tgstation/tgstation#84439

About The Pull Request

Humanizing a monkey doesn't result in someone with trans thigh-highs and a bra

Why It's Good For The Game

It is kinda weird and bad for immersion. Humans can just get it back from dressers anyways.

Changelog

🆑 Melbert
add: Humanizing a monkey removes undergarments such as socks
/:cl:

* Humanizing monkeys removes undergarments  (#84439)

## About The Pull Request

Humanizing a monkey doesn't result in someone with trans thigh-highs and
a bra

## Why It's Good For The Game

It is kinda weird and bad for immersion. Humans can just get it back
from dressers anyways.

## Changelog

:cl: Melbert
add: Humanizing a monkey removes undergarments such as socks
/:cl:

* Humanizing monkeys removes undergarments

---------

Co-authored-by: MrMelbert <[email protected]>
Co-authored-by: NovaBot13 <[email protected]>
@Iajret Iajret merged commit 2ea4366 into master Jun 30, 2024
28 checks passed
@Iajret Iajret deleted the upstream-mirror-3395 branch June 30, 2024 13:27
AnywayFarus added a commit that referenced this pull request Jun 30, 2024
Iajret pushed a commit that referenced this pull request Jul 31, 2024
* Attempting to fix search_object hard dels (#85394)

## About The Pull Request


![image](https://github.com/user-attachments/assets/b4cd5369-06c4-4e01-98f4-343030e28199)

These were coming up often, had a hunch that the callback may have been
hanging their refs. It seems to be true from my testing, they should be
passing GLOBAL_PROC as the `object` arg, not `src`, since qdel is not a
`search_object` proc. (Thanks Melbert for noticing that detail!)

Also just tidies up the code a bit, adding checks to prevent redundant
qdel calls and preventing a potential race condition with qdeleted
atoms.

## Why It's Good For The Game

Hard dels here in particular have potential to cause significant lag and
issues.

## Changelog

Not player-facing really

* Attempting to fix search_object hard dels

---------

Co-authored-by: Bloop <[email protected]>
Co-authored-by: NovaBot13 <[email protected]>
FeenieRU pushed a commit that referenced this pull request Jul 31, 2024
* [MIRROR] Attempting to fix search_object hard dels [MDB IGNORE] (#4140)

* Attempting to fix search_object hard dels (#85394)

## About The Pull Request


![image](https://github.com/user-attachments/assets/b4cd5369-06c4-4e01-98f4-343030e28199)

These were coming up often, had a hunch that the callback may have been
hanging their refs. It seems to be true from my testing, they should be
passing GLOBAL_PROC as the `object` arg, not `src`, since qdel is not a
`search_object` proc. (Thanks Melbert for noticing that detail!)

Also just tidies up the code a bit, adding checks to prevent redundant
qdel calls and preventing a potential race condition with qdeleted
atoms.

## Why It's Good For The Game

Hard dels here in particular have potential to cause significant lag and
issues.

## Changelog

Not player-facing really

* Attempting to fix search_object hard dels

---------

Co-authored-by: Bloop <[email protected]>
Co-authored-by: NovaBot13 <[email protected]>

* [MIRROR] Attempting to fix search_object hard dels

---------

Co-authored-by: NovaBot <[email protected]>
Co-authored-by: Bloop <[email protected]>
Co-authored-by: NovaBot13 <[email protected]>
Co-authored-by: StealsThePRs <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants