Skip to content

Bump sys.setrecursionlimit within test_nest_limit_1024#682

Merged
methane merged 1 commit into
msgpack:mainfrom
clin1234:stackerror
Jun 3, 2026
Merged

Bump sys.setrecursionlimit within test_nest_limit_1024#682
methane merged 1 commit into
msgpack:mainfrom
clin1234:stackerror

Conversation

@clin1234
Copy link
Copy Markdown
Contributor

@clin1234 clin1234 commented Jun 2, 2026

Fixes test failures with no-GIL interpreters

@clin1234 clin1234 marked this pull request as ready for review June 2, 2026 16:28
@methane methane merged commit d627d30 into msgpack:main Jun 3, 2026
34 checks passed
Comment thread test/test_limits.py
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why was 10000 not enough? The nest is only 1024....

@methane
Copy link
Copy Markdown
Member

methane commented Jun 3, 2026

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