diff options
author | Googler <noreply@google.com> | 2016-08-30 14:53:27 +0000 |
---|---|---|
committer | Klaus Aehlig <aehlig@google.com> | 2016-08-31 08:10:54 +0000 |
commit | 92647f58764d021190b746eaedbe2c299783389b (patch) | |
tree | 97c2072db22ec1ad4350339516a84c82ffddbdb3 /src/main/java/com/google | |
parent | 3d65eb97d2244e8709b338beef3807edefa57ff5 (diff) |
Fix operator precedence error in LongArrayList.growCapacity
+ binds tighter than shift operators so this code was essentially doing (i+i)/2 when it was really trying to do i + (i/2).
--
MOS_MIGRATED_REVID=131709276
Diffstat (limited to 'src/main/java/com/google')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/util/LongArrayList.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/util/LongArrayList.java b/src/main/java/com/google/devtools/build/lib/util/LongArrayList.java index ff5a89155f..8adb027b23 100644 --- a/src/main/java/com/google/devtools/build/lib/util/LongArrayList.java +++ b/src/main/java/com/google/devtools/build/lib/util/LongArrayList.java @@ -278,7 +278,7 @@ public class LongArrayList { * Uses a growth factor of 1.5. */ private int growCapacity(int newSize) { - return newSize + newSize >> 1; + return newSize + (newSize >> 1); } } |