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

std: adjust requested stack size for thread-local storage. #11284

Merged
merged 1 commit into from
Jan 4, 2014

Conversation

huonw
Copy link
Member

@huonw huonw commented Jan 3, 2014

If there is a lot of data in thread-local storage some implementations
of pthreads (e.g. glibc) fail if you don't request a stack large enough
-- by adjusting for the minimum size we guarantee that our stacks are
always large enough. Issue #6233.

@huonw
Copy link
Member Author

huonw commented Jan 3, 2014

This function appears to be glibc specific, so we're making it harder to support other libc's by doing this.

If there is a lot of data in thread-local storage some implementations
of pthreads (e.g. glibc) fail if you don't request a stack large enough
-- by adjusting for the minimum size we guarantee that our stacks are
always large enough. Issue rust-lang#6233.
bors added a commit that referenced this pull request Jan 4, 2014
If there is a lot of data in thread-local storage some implementations
of pthreads (e.g. glibc) fail if you don't request a stack large enough
-- by adjusting for the minimum size we guarantee that our stacks are
always large enough. Issue #6233.
@bors bors closed this Jan 4, 2014
@bors bors merged commit f1b5f59 into rust-lang:master Jan 4, 2014
@huonw huonw deleted the issue-6233 branch January 5, 2014 12:16
flip1995 pushed a commit to flip1995/rust that referenced this pull request Aug 11, 2023
Alphabetically order arms in `methods/mod.rs` match

changelog: none

just order the arms in the match block
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.

3 participants