Merge pull request iluwatar#616 from mookkiah/issue_587_async-method-invocation

iluwatar · web-flow · commit f47dc1eb7cd2 · 2017-08-18T22:24:43.000+03:00
iluwatar#587 SonarQube reports bugs - async-method-invocation
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
@@ -139,7 +139,7 @@ public T getValue() throws ExecutionException {
     @Override
     public void await() throws InterruptedException {
       synchronized (lock) {
-        if (!isCompleted()) {
+        while (!isCompleted()) {
           lock.wait();
         }
       }

Original file line number	Diff line number	Diff line change
`@@ -139,7 +139,7 @@ public T getValue() throws ExecutionException {`
`139`	`139`	`@Override`
`140`	`140`	`public void await() throws InterruptedException {`
`141`	`141`	`synchronized (lock) {`
`142`		`- if (!isCompleted()) {`
	`142`	`+ while (!isCompleted()) {`
`143`	`143`	`lock.wait();`
`144`	`144`	`}`
`145`	`145`	`}`