Skip to content

sirens: Improve Hook Compatibility#227

Merged
user-grinch merged 3 commits intouser-grinch:mainfrom
AmeerDotEXE:ameer/sirens-hooks
Mar 31, 2026
Merged

sirens: Improve Hook Compatibility#227
user-grinch merged 3 commits intouser-grinch:mainfrom
AmeerDotEXE:ameer/sirens-hooks

Conversation

@AmeerDotEXE
Copy link
Copy Markdown
Contributor

removes the assembly code in hkRegisterCorona
and uses original call address for better mod compatiblility

@AmeerDotEXE AmeerDotEXE marked this pull request as ready for review March 29, 2026 16:17
@AmeerDotEXE AmeerDotEXE changed the title sirens: Improve hkRegisterCorona Hook sirens: Improve Hook Compatibility Mar 29, 2026
@AmeerDotEXE
Copy link
Copy Markdown
Contributor Author

removed hkVehiclePreRender in favor for ThiscallEvent
and changed hook for hkUsesSiren to thiscall instead of fastcall
i haven't tested it yet since i couldn't build it, i'll try again once p-sdk workflow starts working

@user-grinch user-grinch force-pushed the main branch 2 times, most recently from c9bed73 to da8bd07 Compare March 29, 2026 21:56
@user-grinch
Copy link
Copy Markdown
Owner

Try with the latest changes. It should resolve the build issue.

@AmeerDotEXE
Copy link
Copy Markdown
Contributor Author

very nice :D
i've kept the git timeline linear 👌

@user-grinch
Copy link
Copy Markdown
Owner

Sorry but is it tested? (haven't been able to try it out yet)

@AmeerDotEXE
Copy link
Copy Markdown
Contributor Author

i haven't either, i don't have models to test with
told simborris to try it out, if the hook fails it would show duplicate coronas and sirens wouldn't work
haven't heard from him yet, lets wait a bit more

@AmeerDotEXE
Copy link
Copy Markdown
Contributor Author

@user-grinch ready to merge, build succeeded and test passed 🫡

@user-grinch user-grinch merged commit ffbb318 into user-grinch:main Mar 31, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants