Skip to content

std thread join handle#18

Open
JakkuSakura wants to merge 1 commit into
najamelan:devfrom
JakkuSakura:std_thread_join_handle
Open

std thread join handle#18
JakkuSakura wants to merge 1 commit into
najamelan:devfrom
JakkuSakura:std_thread_join_handle

Conversation

@JakkuSakura
Copy link
Copy Markdown
Contributor

No description provided.

@najamelan
Copy link
Copy Markdown
Owner

If I understand well you want to be able to wrap std::thread::Handle in JoinHandle so it can be used the same way. I am in favor of the feature.

I see there is still a FIXME in the PR. I suppose this is not ready for merging?

@JakkuSakura
Copy link
Copy Markdown
Contributor Author

I see there is still a FIXME in the PR. I suppose this is not ready for merging?

I can't figure out a proper waking machenism with std::thread::Handle. Currently it wakes every time.

Do you have any ideas?

If not, it has to be merged with the FIXME unfunately.

@najamelan najamelan force-pushed the dev branch 2 times, most recently from 11c005d to c3e0ead Compare September 17, 2022 11:59
@najamelan najamelan force-pushed the dev branch 5 times, most recently from 5afa6ef to b7cf3e6 Compare July 22, 2023 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants