Skip to content

Conversation

@Aqua-Peak
Copy link
Contributor

my_sigactionhandler parament:sig is belong to host.

my_sigactionhandler parament:sig is belong to host.
@xiangzhai
Copy link
Contributor

Already fixed? #3292

@Aqua-Peak
Copy link
Contributor Author

#3292 for function named "my_box64signalhandler". Now for function named "my_sigactionhandler".
Reason that "#define MY_SIGHANDLER ((signum==X64_SIGSEGV || signum==X64_SIGBUS || signum==X64_SIGILL || signum==X64_SIGABRT)?my_box64signalhandler:my_sigactionhandler)", "my_sigactionhandler" should also need conversion.

@ptitSeb
Copy link
Owner

ptitSeb commented Jan 6, 2026

LGTM

@ksco ksco marked this pull request as ready for review January 8, 2026 07:24
@ksco
Copy link
Collaborator

ksco commented Jan 8, 2026

LGTM

Okay, I made it ready for review. Let's merge it in.

@ksco ksco requested a review from ptitSeb January 8, 2026 07:32
@ptitSeb ptitSeb merged commit fd4a9ea into ptitSeb:main Jan 8, 2026
27 checks 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.

4 participants