fix stack overflow of tests #4

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.

Sign up for GitHub

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

Jump to bottom

Open

zhiqiangxu wants to merge 1 commit into spacesprotocol:main from zhiqiangxu:fix_stackoverflow

Contributor

zhiqiangxu commented Apr 9, 2024

Currently only the first test will pass, others will report stack overflow.

It can be worked around by setting larger RUST_MIN_STACK but it's inconvenient.

This pr fixes this issue by making WriteBuffer allocate on heap instead of stack.


          fix stack overflow

32f81d3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet