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;