From 64bd3be978d0e4e1c1d0754fc059e622a59c25bf Mon Sep 17 00:00:00 2001 From: nkramer44 Date: Fri, 11 Aug 2023 17:42:18 -0400 Subject: [PATCH] add comment --- .../java/org/xrpl/xrpl4j/model/transactions/AccountSet.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/xrpl4j-core/src/main/java/org/xrpl/xrpl4j/model/transactions/AccountSet.java b/xrpl4j-core/src/main/java/org/xrpl/xrpl4j/model/transactions/AccountSet.java index 328bb2641..d558d1989 100644 --- a/xrpl4j-core/src/main/java/org/xrpl/xrpl4j/model/transactions/AccountSet.java +++ b/xrpl4j-core/src/main/java/org/xrpl/xrpl4j/model/transactions/AccountSet.java @@ -141,6 +141,8 @@ default AccountSet normalizeClearFlag() { // This can happen if: // 1. A developer sets clearFlagRawValue manually in the builder // 2. JSON has ClearFlag and jackson sets clearFlagRawValue. + // This value will never be negative due to XRPL representing this kind of flag as an unsigned number, + // so no lower bound check is required. if (clearFlagRawValue().get().longValue() <= 15) { // Set clearFlag to clearFlagRawValue if clearFlagRawValue matches a valid AccountSetFlag variant. return AccountSet.builder().from(this) @@ -232,7 +234,8 @@ default AccountSet normalizeSetFlag() { // This can happen if: // 1. A developer sets setFlagRawValue manually in the builder // 2. JSON has ClearFlag and jackson sets setFlagRawValue. - + // This value will never be negative due to XRPL representing this kind of flag as an unsigned number, + // so no lower bound check is required. if (setFlagRawValue().get().longValue() <= 15) { // Set setFlag to setFlagRawValue if setFlagRawValue matches a valid AccountSetFlag variant. return AccountSet.builder().from(this)