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

Fix gettid test #19

Open
rogersimmons opened this issue Jun 23, 2022 · 0 comments
Open

Fix gettid test #19

rogersimmons opened this issue Jun 23, 2022 · 0 comments
Assignees
Labels
enhancement New feature or request review

Comments

@rogersimmons
Copy link
Contributor

rogersimmons commented Jun 23, 2022

The gettid test relies on parallel streams to count the threads.
However, the number of threads actually used is arbitrary, and can legitimately be 1 in some cases (highlighted by the Windows build).
Refactor the test to use a known number of threads and then robustly check.
(Also apply the same change to getpid test - although there the variable number of actual threads used is less of an issue as the pid count should be 1 anyway)

@rogersimmons rogersimmons self-assigned this Jun 23, 2022
@tgd tgd added the enhancement New feature or request label Jan 25, 2024
@tgd tgd added the review label Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request review
Projects
None yet
Development

No branches or pull requests

2 participants