Unconditional 'wait()' call

Reports wait() being called unconditionally within a synchronized context.

Normally, wait() is used to block a thread until some condition is true. If wait() is called unconditionally, it often indicates that the condition was checked before a lock was acquired. In that case a data race may occur, with the condition becoming true between the time it was checked and the time the lock was acquired.

While constructs found by this inspection are not necessarily incorrect, they are certainly worth examining.

Example: