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

Dionae changes #11905

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

XeonMations
Copy link
Contributor

About The Pull Request

Some basic gameplay changes for dionae, requested by Ruko and Bacon.

Why It's Good For The Game

Having oversights is bad

Testing Photographs and Procedure

Screenshots&Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑 XeonMations
balance: Dionae now take 5 seconds to assimilate nymphs on the ground.
/:cl:

@XeonMations XeonMations marked this pull request as ready for review November 30, 2024 13:01
if(!LAZYLEN(limbs_to_heal))
return
playsound(H, 'sound/creatures/venus_trap_hit.ogg', 25, 1)
toggle_ai(AI_OFF)
if(!do_after(arrived_diona, 5 SECONDS, source, progress = TRUE))
Copy link
Member

Choose a reason for hiding this comment

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

This proc doesn't have the SIGNAL_HANDLER define in error. If you add it you will find that this line fails because signal handlers shouldn't be sleeping as they cause unpredictable behaviour. You will need to mark this proc as a signal handler and either call this behaviour asynchronously or set the proc to waitfor false.

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.

2 participants