You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by ah...@apache.org on 2019/09/16 18:58:57 UTC

[commons-rng] 01/13: Extract assignment out of expression.

This is an automated email from the ASF dual-hosted git repository.

aherbert pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-rng.git

commit 188deb4ddb2f911750ecdfe9d9926ddd0c0f744e
Author: Alex Herbert <ah...@apache.org>
AuthorDate: Sat Sep 14 12:11:46 2019 +0100

    Extract assignment out of expression.
---
 .../java/org/apache/commons/rng/core/source64/XorShift1024Star.java    | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/commons-rng-core/src/main/java/org/apache/commons/rng/core/source64/XorShift1024Star.java b/commons-rng-core/src/main/java/org/apache/commons/rng/core/source64/XorShift1024Star.java
index 0b732c2..7525129 100644
--- a/commons-rng-core/src/main/java/org/apache/commons/rng/core/source64/XorShift1024Star.java
+++ b/commons-rng-core/src/main/java/org/apache/commons/rng/core/source64/XorShift1024Star.java
@@ -126,7 +126,8 @@ public class XorShift1024Star extends LongProvider implements JumpableUniformRan
     @Override
     public long next() {
         final long s0 = state[index];
-        long s1 = state[index = (index + 1) & 15];
+        index = (index + 1) & 15;
+        long s1 = state[index];
         s1 ^= s1 << 31; // a
         state[index] = s1 ^ s0 ^ (s1 >>> 11) ^ (s0 >>> 30); // b,c
         return state[index] * multiplier;