From ffa693b3d560d5078f0cc4c7787bdba9f1232c50 Mon Sep 17 00:00:00 2001
From: Jerry Lee <oldratlee@gmail.com>
Date: Sat, 17 Aug 2024 18:21:55 +0800
Subject: [PATCH] docs: also add the javadoc comments of JDK bug info at
 internal `FutureCanceller` class

---
 .../java/io/foldright/cffu/CompletableFutureUtils.java     | 6 ++++--
 .../main/java/io/foldright/cffu/DelayExecutionHelpers.java | 7 +++++++
 2 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/cffu-core/src/main/java/io/foldright/cffu/CompletableFutureUtils.java b/cffu-core/src/main/java/io/foldright/cffu/CompletableFutureUtils.java
index 337e1505..b18a9892 100644
--- a/cffu-core/src/main/java/io/foldright/cffu/CompletableFutureUtils.java
+++ b/cffu-core/src/main/java/io/foldright/cffu/CompletableFutureUtils.java
@@ -3727,9 +3727,11 @@ public static <C extends CompletableFuture<?>> C cffuOrTimeout(
      * is one less thread switch of task execution when triggered by timeout.
      * <p>
      * Note: Before Java 21(Java 20-), {@link CompletableFuture#orTimeout(long, TimeUnit)}
-     * can leak if the future completes exceptionally, more info see
-     * <a href="https://bugs.openjdk.org/browse/JDK-8303742">JDK-8303742</a>
+     * leaks if the future completes exceptionally, more info see
+     * <a href="https://bugs.openjdk.org/browse/JDK-8303742">issue JDK-8303742</a>,
+     * <a href="https://github.com/openjdk/jdk/pull/13059">PR review openjdk/jdk/13059</a>
      * and <a href="https://github.com/openjdk/jdk/commit/ded6a8131970ac2f7ae59716769e6f6bae3b809a">JDK bugfix commit</a>.
+     * The cffu backport logic(for Java 8) has merged the fix of this JDK bug.
      *
      * @param timeout how long to wait before completing exceptionally with a TimeoutException, in units of {@code unit}
      * @param unit    a {@code TimeUnit} determining how to interpret the {@code timeout} parameter
diff --git a/cffu-core/src/main/java/io/foldright/cffu/DelayExecutionHelpers.java b/cffu-core/src/main/java/io/foldright/cffu/DelayExecutionHelpers.java
index 8f4325ff..76bf04bf 100644
--- a/cffu-core/src/main/java/io/foldright/cffu/DelayExecutionHelpers.java
+++ b/cffu-core/src/main/java/io/foldright/cffu/DelayExecutionHelpers.java
@@ -200,6 +200,13 @@ final class FutureCanceller implements BiConsumer<Object, Throwable> {
         this.f = requireNonNull(f);
     }
 
+    /**
+     * Note: Before Java 21(Java 20-), {@link CompletableFuture#orTimeout(long, TimeUnit)}
+     * leaks if the future completes exceptionally, more info see
+     * <a href="https://bugs.openjdk.org/browse/JDK-8303742">issue JDK-8303742</a>,
+     * <a href="https://github.com/openjdk/jdk/pull/13059">PR review openjdk/jdk/13059</a>
+     * and <a href="https://github.com/openjdk/jdk/commit/ded6a8131970ac2f7ae59716769e6f6bae3b809a">JDK bugfix commit</a>.
+     */
     @Override
     public void accept(Object ignore, @Nullable Throwable ex) {
         if (f != null && !f.isDone())