Check that forceSmallChangeAsFee > 0

nervosnetwork · Jul 21, 2023 · f4dd99e · f4dd99e
1 parent 90e8775
commit f4dd99e
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/ckb/src/main/java/org/nervos/ckb/transaction/TransactionBuilderConfiguration.java b/ckb/src/main/java/org/nervos/ckb/transaction/TransactionBuilderConfiguration.java
@@ -73,11 +73,17 @@ public Long getForceSmallChangeAsFee() {
   }
 
   /**
-   * Set forceSmallChangeAsFee. When building transaction, a change output will not be required if it would be smaller than the specified amount.
+   * Set forceSmallChangeAsFee. When building transaction, a change output will not be required if its capacity would be
+   * smaller than the specified amount.
    *
-   * @param forceSmallChangeAsFee Unit shannons
+   * @param forceSmallChangeAsFee Should be positive. Unit is shannons.
    */
   public void setForceSmallChangeAsFee(@Nullable Long forceSmallChangeAsFee) {
+    if (forceSmallChangeAsFee != null) {
+      if (forceSmallChangeAsFee <= 0) {
+        throw new IllegalArgumentException("invalid forceSmallChangeAsFee: " + forceSmallChangeAsFee);
+      }
+    }
     this.forceSmallChangeAsFee = forceSmallChangeAsFee;
   }
 }