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 2021/01/18 17:50:55 UTC
[commons-rng] branch master updated: Minor Improvement:
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
The following commit(s) were added to refs/heads/master by this push:
new 881650c Minor Improvement:
new 456f053 Merge branch 'arturobernalg-feature/feature/minor_improvement'
881650c is described below
commit 881650c63cbeefa8ba7d2f9404cf8be5de1c6e25
Author: Arturo Bernal <ar...@gmail.com>
AuthorDate: Mon Jan 18 09:57:44 2021 +0100
Minor Improvement:
* Add final on private variable
* fix javadoc
* Use Array Filling
---
.../java/org/apache/commons/rng/core/source32/ISAACRandom.java | 4 +---
.../org/apache/commons/rng/core/source32/MersenneTwister.java | 2 +-
.../apache/commons/rng/core/source64/MersenneTwister64.java | 2 +-
.../commons/rng/sampling/distribution/PoissonSamplerCache.java | 2 +-
.../org/apache/commons/rng/simple/ThreadLocalRandomSource.java | 10 +++++-----
5 files changed, 9 insertions(+), 11 deletions(-)
diff --git a/commons-rng-core/src/main/java/org/apache/commons/rng/core/source32/ISAACRandom.java b/commons-rng-core/src/main/java/org/apache/commons/rng/core/source32/ISAACRandom.java
index 7e448d3..80e1c1e 100644
--- a/commons-rng-core/src/main/java/org/apache/commons/rng/core/source32/ISAACRandom.java
+++ b/commons-rng-core/src/main/java/org/apache/commons/rng/core/source32/ISAACRandom.java
@@ -189,9 +189,7 @@ public class ISAACRandom extends IntProvider {
isaacA = 0;
isaacB = 0;
isaacC = 0;
- for (int j = 0; j < arr.length; j++) {
- arr[j] = GLD_RATIO;
- }
+ Arrays.fill(arr, GLD_RATIO);
for (int j = 0; j < 4; j++) {
shuffle();
}
diff --git a/commons-rng-core/src/main/java/org/apache/commons/rng/core/source32/MersenneTwister.java b/commons-rng-core/src/main/java/org/apache/commons/rng/core/source32/MersenneTwister.java
index 7d30440..2bcaa29 100644
--- a/commons-rng-core/src/main/java/org/apache/commons/rng/core/source32/MersenneTwister.java
+++ b/commons-rng-core/src/main/java/org/apache/commons/rng/core/source32/MersenneTwister.java
@@ -104,7 +104,7 @@ public class MersenneTwister extends IntProvider {
/** X * MATRIX_A for X = {0, 1}. */
private static final int[] MAG01 = {0x0, 0x9908b0df};
/** Bytes pool. */
- private int[] mt = new int[N];
+ private final int[] mt = new int[N];
/** Current index in the bytes pool. */
private int mti;
diff --git a/commons-rng-core/src/main/java/org/apache/commons/rng/core/source64/MersenneTwister64.java b/commons-rng-core/src/main/java/org/apache/commons/rng/core/source64/MersenneTwister64.java
index 03d68e9..4d7bb8e 100644
--- a/commons-rng-core/src/main/java/org/apache/commons/rng/core/source64/MersenneTwister64.java
+++ b/commons-rng-core/src/main/java/org/apache/commons/rng/core/source64/MersenneTwister64.java
@@ -84,7 +84,7 @@ public class MersenneTwister64 extends LongProvider {
/** Least significant 31 bits. */
private static final long LM = 0x7fffffffL;
/** Bytes pool. */
- private long[] mt = new long[NN];
+ private final long[] mt = new long[NN];
/** Current index in the bytes pool. */
private int mti;
diff --git a/commons-rng-sampling/src/main/java/org/apache/commons/rng/sampling/distribution/PoissonSamplerCache.java b/commons-rng-sampling/src/main/java/org/apache/commons/rng/sampling/distribution/PoissonSamplerCache.java
index 1ee8f0f..e935d2b 100644
--- a/commons-rng-sampling/src/main/java/org/apache/commons/rng/sampling/distribution/PoissonSamplerCache.java
+++ b/commons-rng-sampling/src/main/java/org/apache/commons/rng/sampling/distribution/PoissonSamplerCache.java
@@ -75,7 +75,7 @@ public class PoissonSamplerCache {
* {@code N = (int)Math.floor(mean)}.
*/
private final int maxN;
- /** The cache of states between {@link minN} and {@link maxN}. */
+ /** The cache of states between {@link #minN} and {@link #maxN}. */
private final LargeMeanPoissonSamplerState[] values;
/**
diff --git a/commons-rng-simple/src/main/java/org/apache/commons/rng/simple/ThreadLocalRandomSource.java b/commons-rng-simple/src/main/java/org/apache/commons/rng/simple/ThreadLocalRandomSource.java
index 2d0af50..604b01b 100644
--- a/commons-rng-simple/src/main/java/org/apache/commons/rng/simple/ThreadLocalRandomSource.java
+++ b/commons-rng-simple/src/main/java/org/apache/commons/rng/simple/ThreadLocalRandomSource.java
@@ -69,7 +69,7 @@ public final class ThreadLocalRandomSource {
*
* <p>This should only be modified to create new instances in a synchronized block.
*/
- private static EnumMap<RandomSource, ThreadLocal<UniformRandomProvider>> sources =
+ private static final EnumMap<RandomSource, ThreadLocal<UniformRandomProvider>> SOURCES =
new EnumMap<RandomSource,
ThreadLocal<UniformRandomProvider>>(RandomSource.class);
@@ -112,7 +112,7 @@ public final class ThreadLocalRandomSource {
* @throws IllegalArgumentException if the source is null or the source requires arguments
*/
public static UniformRandomProvider current(RandomSource source) {
- ThreadLocal<UniformRandomProvider> rng = sources.get(source);
+ ThreadLocal<UniformRandomProvider> rng = SOURCES.get(source);
// Implement double-checked locking:
// https://en.wikipedia.org/wiki/Double-checked_locking#Usage_in_Java
if (rng == null) {
@@ -122,11 +122,11 @@ public final class ThreadLocalRandomSource {
throw new IllegalArgumentException("Random source is null");
}
- synchronized (sources) {
- rng = sources.get(source);
+ synchronized (SOURCES) {
+ rng = SOURCES.get(source);
if (rng == null) {
rng = new ThreadLocalRng(source);
- sources.put(source, rng);
+ SOURCES.put(source, rng);
}
}
}