FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-159603: Call pthread_attr_init() in hardware_stack_limits() by vstinner · Pull Request #159857 · python/cpython · GitHub

Repository navigation

gh-159603: Call pthread_attr_init() in hardware_stack_limits() - #159857

Merged
vstinner merged 2 commits into
python:mainfrom
vstinner:attr_init
Oct 20, 2026
Merged

vstinner merged 2 commits into
python:mainfrom
vstinner:attr_init

Conversation

vstinner commented Oct 20, 2026 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Member

Always call pthread_attr_init() in hardware_stack_limits(), not only on FreeBSD (if pthread_attr_get_np() is available).

Always call pthread_attr_init() in hardware_stack_limits(), not only
on FreeBSD (if pthread_attr_get_np() is available).

Copy link
Copy Markdown
Member Author

Emscripten fails to build:

../../../../Python/ceval.c:162:33: error: incompatible pointer types passing 'workaround_pthread_attr_t *' to parameter of type 'pthread_attr_t *' [-Wincompatible-pointer-types]
  162 |     int err = pthread_attr_init(&attr);
      |                                 ^~~~~
/home/runner/work/_temp/emsdk-cache/6.0.10/emsdk/upstream/emscripten/cache/sysroot/include/pthread.h:149:39: note: passing argument to parameter here
  149 | int pthread_attr_init(pthread_attr_t *);
      |                                       ^

Copy link
Copy Markdown
Member Author

Good: "Tests / Emscripten / build and test (pull_request)" passed with my fix.

vstinner merged commit 14b43d4 into python:main Oct 20, 2026
57 of 58 checks passed
vstinner deleted the attr_init branch October 20, 2026 18:16
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL