From 2a7feba427873136bb96c6dbdeb3ee11d6374718 Mon Sep 17 00:00:00 2001 From: "mahendran.mookkiah" Date: Fri, 18 Aug 2017 14:18:43 -0400 Subject: [PATCH] #587 SonarQube reports bugs As recommended in https://sonarcloud.io/organizations/default/rules#rule_key=squid%3AS2274 Used while insteadof if - for waiting upon a condition. --- .../iluwatar/async/method/invocation/ThreadAsyncExecutor.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/async-method-invocation/src/main/java/com/iluwatar/async/method/invocation/ThreadAsyncExecutor.java b/async-method-invocation/src/main/java/com/iluwatar/async/method/invocation/ThreadAsyncExecutor.java index f92792c39..d12ebbe19 100644 --- a/async-method-invocation/src/main/java/com/iluwatar/async/method/invocation/ThreadAsyncExecutor.java +++ b/async-method-invocation/src/main/java/com/iluwatar/async/method/invocation/ThreadAsyncExecutor.java @@ -139,7 +139,7 @@ public class ThreadAsyncExecutor implements AsyncExecutor { @Override public void await() throws InterruptedException { synchronized (lock) { - if (!isCompleted()) { + while (!isCompleted()) { lock.wait(); } }