Permalink
Browse files

#130 - Bug in boost::condition_variable on Windows.

  • Loading branch information...
viboes committed Aug 24, 2017
1 parent f27a292 commit fdc0cbcd8ce8f34a19c7cdaf6f1fa679f0a49657
Showing with 2 additions and 2 deletions.
  1. +2 −2 include/boost/thread/win32/condition_variable.hpp
@@ -211,7 +211,7 @@ namespace boost
{}
#endif
void remove_waiter()
void remove_waiter_and_reset()
{
if (entry) {
boost::lock_guard<boost::mutex> internal_lock(internal_mutex);
@@ -250,7 +250,7 @@ namespace boost
woken=entry->woken();
}
// do it here to avoid throwing on the destructor
entry->remove_waiter();
entry.remove_waiter_and_reset();
locker.lock();
return woken;
}

3 comments on commit fdc0cbc

@kivadiu

This comment has been minimized.

Show comment
Hide comment
@kivadiu

kivadiu Sep 1, 2017

Contributor

Is it normal that you replace "->" by "."?

Contributor

kivadiu replied Sep 1, 2017

Is it normal that you replace "->" by "."?

@kivadiu

This comment has been minimized.

Show comment
Hide comment
@kivadiu

kivadiu Sep 1, 2017

Contributor

OK, I just saw #130, sorry for this comment.

Contributor

kivadiu replied Sep 1, 2017

OK, I just saw #130, sorry for this comment.

@viboes

This comment has been minimized.

Show comment
Hide comment
@viboes

viboes Sep 1, 2017

Collaborator

no problem.

Collaborator

viboes replied Sep 1, 2017

no problem.

Please sign in to comment.