Inspectopedia
 
2024.3

Unconditional 'wait()' call

Warning
New
Last modified: 03 December 2024

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: