From a54a48ebfebbcc815a93b767b8fd49944d85f3dc Mon Sep 17 00:00:00 2001 From: Jerry Lee <oldratlee@gmail.com> Date: Mon, 27 Jan 2025 21:13:08 +0800 Subject: [PATCH] =?UTF-8?q?refactor:=20use=20final=20var=20instead=20of=20?= =?UTF-8?q?reassigning=20var=20=F0=9F=A5=A7?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../main/java/io/foldright/cffu/CffuFactoryBuilder.java | 8 ++++---- 1 file changed, 4 insertions(+), 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 index 2140fdf0..09b27e6b 100644 --- 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); }