Prefer std::lock_guard when std::unique_lock features are not required

[swebb2066]

This PR applies the guidelines in this highly rated answer on StackOverflow.

Also, the call site of std::condition_variable::notify_all does not need to hold the lock on the same mutex as the one held by the waiting thread(s)