fix: the precedence of + is higher than >> (#16312)

cuiweixie · web-flow · commit e855de52f5f8 · 2026-02-20T14:47:46.000-08:00
Motivation:
Code does not take operator precedence into account.

Modification:
Add parenthesis to force operator precedence to match the comments.

Result:
Array size increments works as intended.
diff --git a/transport-classes-kqueue/src/main/java/io/netty/channel/kqueue/KQueueEventArray.java b/transport-classes-kqueue/src/main/java/io/netty/channel/kqueue/KQueueEventArray.java
@@ -99,7 +99,7 @@ private void reallocIfNeeded() {
      */
     void realloc(boolean throwIfFail) {
         // Double the capacity while it is "sufficiently small", and otherwise increase by 50%.
-        int newLength = capacity <= 65536 ? capacity << 1 : capacity + capacity >> 1;
+        int newLength = capacity <= 65536 ? capacity << 1 : capacity + (capacity >> 1);
 
         try {
             int newCapacity = calculateBufferCapacity(newLength);
diff --git a/transport-classes-kqueue/src/main/java/io/netty/channel/kqueue/NativeLongArray.java b/transport-classes-kqueue/src/main/java/io/netty/channel/kqueue/NativeLongArray.java
@@ -88,7 +88,7 @@ private long memoryOffset(int index) {
     private void reallocIfNeeded() {
         if (size == capacity) {
             // Double the capacity while it is "sufficiently small", and otherwise increase by 50%.
-            int newLength = capacity <= 65536 ? capacity << 1 : capacity + capacity >> 1;
+            int newLength = capacity <= 65536 ? capacity << 1 : capacity + (capacity >> 1);
             int newCapacity = calculateBufferCapacity(newLength);
             CleanableDirectBuffer buffer = Buffer.allocateDirectBufferWithNativeOrder(newCapacity);
             // Copy over the old content of the memory and reset the position as we always act on the buffer as if
