Skip to content

Conversation

ChrisDenton
Copy link
Member

@ChrisDenton ChrisDenton commented May 31, 2025

This isn't necessary as cleanup will happen when the process exits regardless.

fixes #141799

@rustbot
Copy link
Collaborator

rustbot commented May 31, 2025

r? @thomcc

rustbot has assigned @thomcc.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels May 31, 2025
@tgross35
Copy link
Contributor

tgross35 commented Jul 1, 2025

@bors2 try jobs=x86_64-msvc-*

@rust-bors
Copy link

rust-bors bot commented Jul 1, 2025

Unknown value for argument "jobs".

@tgross35
Copy link
Contributor

tgross35 commented Jul 1, 2025

@bors2 try jobs=x86_64-msvc-*

@rust-bors
Copy link

rust-bors bot commented Jul 1, 2025

⌛ Trying commit b110d45 with merge 77862f6

To cancel the try build, run the command @bors2 try cancel.

rust-bors bot added a commit that referenced this pull request Jul 1, 2025
Don't call WSACleanup on process exit

This isn't necessary as cleanup will happen when the process exits regardless.

fixes #141799
try-job: x86_64-msvc-*
@rust-bors
Copy link

rust-bors bot commented Jul 1, 2025

☀️ Try build successful (CI)
Build commit: 77862f6 (77862f64491b3eaa745c29822460416af3b41265, parent: f46ce66fcc3d6058f90ac5bf0930f940f1e7b0ca)

@ChrisDenton
Copy link
Member Author

r? libs

@rustbot rustbot assigned jhpratt and unassigned thomcc Jul 9, 2025
@jhpratt
Copy link
Member

jhpratt commented Jul 14, 2025

@ChrisDenton I presume you have sufficient familiarity with Windows that you are certain this is correct? I do not, but at the same time don't see anything obviously wrong. My only suggestion would be to add a comment about why cleanup is empty. Other than that, r=me if you're satisfied with correctness.

@rustbot

This comment has been minimized.

@ChrisDenton
Copy link
Member Author

Added a comment (and then rebased to make rustbot happy)

@bors r=jhpratt

@bors
Copy link
Collaborator

bors commented Jul 14, 2025

📌 Commit c8fb37a has been approved by jhpratt

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 14, 2025
bors added a commit that referenced this pull request Jul 14, 2025
Rollup of 8 pull requests

Successful merges:

 - #141809 (Don't call WSACleanup on process exit)
 - #143710 (Updates to random number generation APIs)
 - #143848 (Rename `stable_mir` and `rustc_smir`)
 - #143855 (Port `#[omit_gdb_pretty_printer_section]` to the new attribute parsing)
 - #143868 (warn on align on fields to avoid breaking changes)
 - #143870 ([COMPILETEST-UNTANGLE 6/N] Use `TestSuite` enum instead of stringly-typed test suites)
 - #143901 (Region constraint nits)
 - #143903 (Fix typos in documentation files)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 74b5ac5 into rust-lang:master Jul 14, 2025
11 checks passed
@rustbot rustbot added this to the 1.90.0 milestone Jul 14, 2025
rust-timer added a commit that referenced this pull request Jul 14, 2025
Rollup merge of #141809 - ChrisDenton:no-cleaup, r=jhpratt

Don't call WSACleanup on process exit

This isn't necessary as cleanup will happen when the process exits regardless.

fixes #141799
github-actions bot pushed a commit to devnexen/miri that referenced this pull request Jul 15, 2025
Rollup of 8 pull requests

Successful merges:

 - rust-lang/rust#141809 (Don't call WSACleanup on process exit)
 - rust-lang/rust#143710 (Updates to random number generation APIs)
 - rust-lang/rust#143848 (Rename `stable_mir` and `rustc_smir`)
 - rust-lang/rust#143855 (Port `#[omit_gdb_pretty_printer_section]` to the new attribute parsing)
 - rust-lang/rust#143868 (warn on align on fields to avoid breaking changes)
 - rust-lang/rust#143870 ([COMPILETEST-UNTANGLE 6/N] Use `TestSuite` enum instead of stringly-typed test suites)
 - rust-lang/rust#143901 (Region constraint nits)
 - rust-lang/rust#143903 (Fix typos in documentation files)

r? `@ghost`
`@rustbot` modify labels: rollup
github-actions bot pushed a commit to model-checking/verify-rust-std that referenced this pull request Jul 29, 2025
Don't call WSACleanup on process exit

This isn't necessary as cleanup will happen when the process exits regardless.

fixes rust-lang#141799
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-libs Relevant to the library team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

windows: consider skipping WSACleanup on exit
6 participants