This is an automated email from the ASF dual-hosted git repository. garydgregory pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-lang.git
commit 4b52ff89a68408dfe5e73786eceb23d0e75f967f Author: Gary Gregory <[email protected]> AuthorDate: Sat Jul 25 08:13:37 2026 -0400 Use final --- .../java/org/apache/commons/lang3/builder/HashCodeBuilder.java | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/main/java/org/apache/commons/lang3/builder/HashCodeBuilder.java b/src/main/java/org/apache/commons/lang3/builder/HashCodeBuilder.java index 2b3bb61e2..14f495a10 100644 --- a/src/main/java/org/apache/commons/lang3/builder/HashCodeBuilder.java +++ b/src/main/java/org/apache/commons/lang3/builder/HashCodeBuilder.java @@ -597,7 +597,7 @@ public HashCodeBuilder() { this(builder().setInitialOddNumber(17).setMultiplierOddNumber(37)); } - private HashCodeBuilder(Builder builder) { + private HashCodeBuilder(final Builder builder) { super(builder); Validate.isTrue(builder.initialOddNumber % 2 != 0, "HashCodeBuilder requires an odd initial value"); Validate.isTrue(builder.multiplierOddNumber % 2 != 0, "HashCodeBuilder requires an odd multiplier"); @@ -857,10 +857,7 @@ public HashCodeBuilder append(final long[] array) { * @return {@code this} instance. */ public HashCodeBuilder append(final Object object) { - if (object == null) { - total = total * constant; - } else if (isRegistered(object) || isAppendRegistered(object)) { - // Cycle detected: skip to avoid infinite recursion (mirrors reflectionAppend). + if (object == null || isRegistered(object) || isAppendRegistered(object)) { total = total * constant; } else if (ObjectUtils.isArray(object)) { try {
