bemanproject / timed_lock_alg / 27275299565

100%

push

github

Adjust the try_lock_until algorithm to match the wording in P3832R2

Since a timed lockable's try_lock_until will not return failure in case
the time given has already expired, but instead return this->try_lock(),
there was a theoretical possibility of never getting out of the loop.

It was essentially possible to get into this situation:
  m1.try_lock_until(expired_tp); // succeeds
  m2.try_lock();                 // fails
  m2.try_lock_until(expired_tp); // succeeds
  m1.try_lock();                 // fails
  // back to the original situation and it could keep going

The remedy was to add wording about "the above steps are repeated until
the time point abs_time has been reached" which made it necessary to do
a Clock::now() call before trying a new sequence (sorry Bosse).

Signed-off-by: Ted Lyngmo <ted@lyncon.se>

2 of 2 new or added lines in 1 file covered. (100.0%)

100 of 100 relevant lines covered (100.0%)

13.5 hits per line