refactor: use final var instead of reassigning var 🥧

foldright · Jan 27, 2025 · a54a48e · a54a48e
1 parent 8d1cecf
commit a54a48e
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/cffu-core/src/main/java/io/foldright/cffu/CffuFactoryBuilder.java b/cffu-core/src/main/java/io/foldright/cffu/CffuFactoryBuilder.java
@@ -87,15 +87,15 @@ static CffuFactory withDefaultExecutor(CffuFactory fac, Executor defaultExecutor
         return new CffuFactory(makeExecutor(defaultExecutor), fac.forbidObtrudeMethods());
     }
 
-    private static Executor makeExecutor(Executor defaultExecutor) {
+    private static Executor makeExecutor(final Executor defaultExecutor) {
         // check CffuMadeExecutor interface to avoid re-wrapping.
         if (defaultExecutor instanceof CffuMadeExecutor) return defaultExecutor;
-
         requireNonNull(defaultExecutor, "defaultExecutor is null");
+
         // because wraps the input executor below, MUST call `screenExecutor` translation beforehand;
         // otherwise the sequent operations can NOT recognize the input executor.
-        defaultExecutor = LLCF.screenExecutor(defaultExecutor);
-        Executor wrapByProviders = wrapExecutorByProviders(defaultExecutor);
+        final Executor screenExecutor = LLCF.screenExecutor(defaultExecutor);
+        final Executor wrapByProviders = wrapExecutorByProviders(screenExecutor);
         return wrapMadeInterface(wrapByProviders);
     }