You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2022/03/20 17:26:05 UTC
[commons-lang] 01/03: In-line only used once local variable.
This is an automated email from the ASF dual-hosted git repository.
ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-lang.git
commit 9658994c9a825ef369e6419173d5e665dcb7938f
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Sun Mar 20 13:15:04 2022 -0400
In-line only used once local variable.
---
src/main/java/org/apache/commons/lang3/concurrent/Memoizer.java | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/src/main/java/org/apache/commons/lang3/concurrent/Memoizer.java b/src/main/java/org/apache/commons/lang3/concurrent/Memoizer.java
index 8d009e5..416f19f 100644
--- a/src/main/java/org/apache/commons/lang3/concurrent/Memoizer.java
+++ b/src/main/java/org/apache/commons/lang3/concurrent/Memoizer.java
@@ -16,7 +16,6 @@
*/
package org.apache.commons.lang3.concurrent;
-import java.util.concurrent.Callable;
import java.util.concurrent.CancellationException;
import java.util.concurrent.ConcurrentHashMap;
import java.util.concurrent.ConcurrentMap;
@@ -115,8 +114,7 @@ public class Memoizer<I, O> implements Computable<I, O> {
while (true) {
Future<O> future = cache.get(arg);
if (future == null) {
- final Callable<O> eval = () -> computable.compute(arg);
- final FutureTask<O> futureTask = new FutureTask<>(eval);
+ final FutureTask<O> futureTask = new FutureTask<>(() -> computable.compute(arg));
future = cache.putIfAbsent(arg, futureTask);
if (future == null) {
future = futureTask;