Why not use std::atomic<bool> instead of bool?

[Igamegum]

The member variable stop

[wilx]

You do not need atomic access there because you still have to cover its use with mutex when you are using the conditional variable.

[jellynian]

oh ， I see your answer here.