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
Windows: support getting the thread name #3345
Conversation
@bors r+ |
Windows: support getting the thread name Also organize the thread name tests a bit.
src/shims/windows/foreign_items.rs
Outdated
Some(Handle::Pseudo(PseudoHandle::CurrentThread)) => this.get_active_thread(), | ||
_ => this.invalid_handle("SetThreadDescription")?, | ||
}; | ||
// FIXME what is the right default thread name? Or should we raise an error? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The right default would be an empty string. Though in std that would be changed to <unnamed>
anyways.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So on actual Windows it succeeds and returns an empty string when no name was that?
That's easy enough to do. Is this documented anywhere?
Yeah I know this is not observable on std.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this documented anywhere?
No.
4fdabe4
to
bef7237
Compare
@bors r+ |
☀️ Test successful - checks-actions |
Also organize the thread name tests a bit.