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

refactor(internal/coroutine): use runtime.Goexit for aborting coroutines #495

Merged
merged 1 commit into from
Feb 13, 2025

Conversation

aofei
Copy link
Member

@aofei aofei commented Feb 12, 2025

Replaced panic/recover based abort mechanism with runtime.Goexit to preserve stack traces for user code panics, simplify error handling logic, and leverage standard Go runtime mechanism for goroutine abort.

…tines

Replaced panic/recover based abort mechanism with `runtime.Goexit` to
preserve stack traces for user code panics, simplify error handling
logic, and leverage standard Go runtime mechanism for goroutine abort.

Signed-off-by: Aofei Sheng <aofei@aofeisheng.com>
@aofei aofei marked this pull request as ready for review February 13, 2025 01:03
@xushiwei xushiwei merged commit c95a480 into goplus:main Feb 13, 2025
3 checks passed
@aofei aofei deleted the coro-abort branch February 13, 2025 03:19
aofei added a commit to aofei/.goplus.spx that referenced this pull request Feb 14, 2025
This follows up on goplus#495.

Signed-off-by: Aofei Sheng <aofei@aofeisheng.com>
nighca pushed a commit that referenced this pull request Feb 14, 2025
This follows up on #495.

Signed-off-by: Aofei Sheng <aofei@aofeisheng.com>
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