| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,2 @@ | ||
| On FreeBSD, call ``pthread_attr_get_np()`` to get the stack size (and guard | ||
| size) of the current thread. Patch by Victor Stinner. |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
| Back | FazBrowse Home | New Git URL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low QualityIt is already called here; is it safe to call here again?
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Qualitypthread_attr_init() should be called every time you allocate a new pthread_attr_t attr on the stack memory: https://www.man7.org/linux/man-pages/man3/pthread_attr_init.3.html. Here the strange part is that the pthread_getattr_np() code path doesn't call pthread_attr_init(), but it calls pthread_attr_destroy().
At least, I can say that calling pthread_getattr_np() on a fresh pthread_attr_t attr variable without calling pthread_attr_init() fails on FreeBSD. So I had to add pthread_attr_init() call.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low QualityI wrote a follow-up PR #159857 to also call pthread_attr_init() on Linux.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.