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 documented return value #9505

Merged
merged 1 commit into from
Feb 28, 2025
Merged

Fix documented return value #9505

merged 1 commit into from
Feb 28, 2025

Conversation

adamwight
Copy link
Contributor

On success, this function will return the result of CreateProcessW which upstream docs say will be "nonzero":
https://learn.microsoft.com/en-us/windows/win32/api/processthreadsapi/nf-processthreadsapi-createprocessw#return-value

On success, this function will return the result of CreateProcessW
which upstream docs say will be "nonzero":
https://learn.microsoft.com/en-us/windows/win32/api/processthreadsapi/nf-processthreadsapi-createprocessw#return-value
Copy link
Contributor

github-actions bot commented Feb 26, 2025

CT Test Results

    3 files    141 suites   50m 41s ⏱️
1 607 tests 1 558 ✅ 49 💤 0 ❌
2 325 runs  2 256 ✅ 69 💤 0 ❌

Results for commit 43ef51f.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@bjorng bjorng self-assigned this Feb 27, 2025
@bjorng bjorng added team:VM Assigned to OTP team VM fix testing currently being tested, tag is used by OTP internal CI labels Feb 27, 2025
@bjorng bjorng merged commit 2980d4e into erlang:master Feb 28, 2025
27 checks passed
@bjorng
Copy link
Contributor

bjorng commented Feb 28, 2025

Thanks for your pull request.

@adamwight adamwight deleted the return-val branch March 3, 2025 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix team:VM Assigned to OTP team VM testing currently being tested, tag is used by OTP internal CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants