Skip to content

Window Mutex: Document that we properly initialize the SRWLock#53311

Merged
bors merged 2 commits intorust-lang:masterfrom
RalfJung:windows-mutex
Aug 24, 2018
Merged

Window Mutex: Document that we properly initialize the SRWLock#53311
bors merged 2 commits intorust-lang:masterfrom
RalfJung:windows-mutex

Conversation

@RalfJung
Copy link
Member

@RalfJung RalfJung commented Aug 13, 2018

See #35836

@rust-highfive
Copy link
Contributor

rust-highfive commented Aug 13, 2018

The job x86_64-gnu-llvm-5.0 of your PR failed on Travis (raw log). Through arcane magic we have determined that the following fragments from the build log may contain information about the problem.

Click to expand the log. 
[00:49:21] ....................................................................................................
[00:49:24] ....................................................................................................
[00:49:26] ....................................................................................................
[00:49:29] ....................................................................................................
[00:49:32] ................iiiiiiiii...........................................................................
[00:49:39] ....................................................................................................
[00:49:43] ......................i.............................................................................
[00:49:46] .................................i..................................................................
[00:49:49] ....................................................................................................

I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact @TimNN. (Feature Requests)

@RalfJung
Copy link
Member Author

RalfJung commented Aug 13, 2018

The system is going down for power off NOW!

travis-ci/travis-ci#4924

@RalfJung RalfJung changed the title Window Mutex: make sure we properly initialize the SRWLock Window Mutex: Document that we properly initialize the SRWLock Aug 13, 2018
retep998
retep998 reviewed Aug 14, 2018
View reviewed changes
src/libstd/sys/windows/mutex.rs Outdated
impl Mutex {
pub const fn new() -> Mutex {
Mutex {
// This works because SRWLOCK_INIT is a NULL pointer, so we are also properly
Copy link
Contributor

@retep998 retep998 Aug 14, 2018 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Technically SRWLOCK is a struct wrapping a pointer which isn't actually used as a pointer, so SRWLOCK_INIT is zero initializing a struct.

Copy link
Member Author

@RalfJung RalfJung Aug 14, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh wow. I guess Windows API doesn't really care about meaningful types?

Copy link
Member Author

@RalfJung RalfJung Aug 14, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed, I hope.

@TimNN TimNN added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 14, 2018
@RalfJung
Copy link
Member Author

RalfJung commented Aug 22, 2018

r? @retep998

@retep998
Copy link
Contributor

retep998 commented Aug 22, 2018

As much as I'd love to r+ this PR, I don't have such permissions, so someone else will have to actually r+ it.

@Mark-Simulacrum
Copy link
Member

Mark-Simulacrum commented Aug 22, 2018

@bors r=retep998

@bors
Copy link
Collaborator

bors commented Aug 22, 2018

📌 Commit b7a49e7 has been approved by retep998

@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 Aug 22, 2018
@bors
Copy link
Collaborator

bors commented Aug 23, 2018

⌛ Testing commit b7a49e7 with merge 8a1ba374f7ae73fd0752f4fc26b9bec5a9268981...

@bors
Copy link
Collaborator

bors commented Aug 23, 2018

💔 Test failed - status-travis

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Aug 23, 2018
@rust-highfive

This comment has been minimized.

Sign in to view
@kennytm
Copy link
Member

kennytm commented Aug 23, 2018

@bors retry rollup

Travis hiccups.

@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 Aug 23, 2018
kennytm added a commit to kennytm/rust that referenced this pull request Aug 24, 2018
@kennytm
Rollup merge of rust-lang#53311 - RalfJung:windows-mutex, r=retep998
ab5a71b 
Window Mutex: Document that we properly initialize the SRWLock

See rust-lang#35836
@kennytm kennytm mentioned this pull request Aug 24, 2018
Merged
bors added a commit that referenced this pull request Aug 24, 2018
@bors
Auto merge of #53662 - kennytm:rollup, r=kennytm
727eabd 
Rollup of 16 pull requests

Successful merges:

 - #53311 (Window Mutex: Document that we properly initialize the SRWLock)
 - #53503 (Discourage overuse of mem::forget)
 - #53545 (Fix #50865: ICE on impl-trait returning functions reaching private items)
 - #53559 (add macro check for lint)
 - #53562 (Lament the invincibility of the Turbofish)
 - #53563 (use String::new() instead of String::from(""), "".to_string(), "".to_owned() or "".into())
 - #53592 (docs: minor stylistic changes to str/string docs)
 - #53594 (Update RELEASES.md to include clippy-preview)
 - #53600 (Fix a grammatical mistake in "expected generic arguments" errors)
 - #53614 (update nomicon and book)
 - #53617 (tidy: Stop requiring a license header)
 - #53618 (Add missing fmt examples)
 - #53636 (Prefer `.nth(n)` over `.skip(n).next()`.)
 - #53644 (Use SmallVec for SmallCStr)
 - #53664 (Remove unnecessary closure in rustc_mir/build/mod.rs)
 - #53666 (Added rustc_codegen_llvm to compiler documentation.)
@bors bors merged commit b7a49e7 into rust-lang:master Aug 24, 2018
@RalfJung RalfJung deleted the windows-mutex branch September 1, 2018 09:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@retep998 retep998 retep998 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@retep998 retep998

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@RalfJung @rust-highfive @retep998 @Mark-Simulacrum @bors @kennytm @TimNN