This is an automated email from the ASF dual-hosted git repository. acosentino pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/camel.git
commit 2b688c5435bf8eac62defead38cbe4d8ad72a647 Author: Andrea Cosentino <[email protected]> AuthorDate: Tue Feb 16 11:49:01 2021 +0100 CAMEL-16218 - Added NOSONAR comment in camel-core --- .../src/main/java/org/apache/camel/language/csimple/CSimpleHelper.java | 1 + .../java/org/apache/camel/language/simple/SimpleExpressionBuilder.java | 2 ++ .../java/org/apache/camel/processor/errorhandler/RedeliveryPolicy.java | 2 ++ 3 files changed, 5 insertions(+) diff --git a/core/camel-core-languages/src/main/java/org/apache/camel/language/csimple/CSimpleHelper.java b/core/camel-core-languages/src/main/java/org/apache/camel/language/csimple/CSimpleHelper.java index 68fa57a..7041e29 100644 --- a/core/camel-core-languages/src/main/java/org/apache/camel/language/csimple/CSimpleHelper.java +++ b/core/camel-core-languages/src/main/java/org/apache/camel/language/csimple/CSimpleHelper.java @@ -465,6 +465,7 @@ public final class CSimpleHelper { public static int random(Exchange exchange, Object min, Object max) { int num1 = exchange.getContext().getTypeConverter().tryConvertTo(int.class, exchange, min); int num2 = exchange.getContext().getTypeConverter().tryConvertTo(int.class, exchange, max); + // NOSONAR Random random = new Random(); return random.nextInt(num2 - num1) + num1; } diff --git a/core/camel-core-languages/src/main/java/org/apache/camel/language/simple/SimpleExpressionBuilder.java b/core/camel-core-languages/src/main/java/org/apache/camel/language/simple/SimpleExpressionBuilder.java index a081852..9844780 100644 --- a/core/camel-core-languages/src/main/java/org/apache/camel/language/simple/SimpleExpressionBuilder.java +++ b/core/camel-core-languages/src/main/java/org/apache/camel/language/simple/SimpleExpressionBuilder.java @@ -190,6 +190,7 @@ public final class SimpleExpressionBuilder { public Object evaluate(Exchange exchange) { int num1 = exp1.evaluate(exchange, Integer.class); int num2 = exp2.evaluate(exchange, Integer.class); + // NOSONAR Random random = new Random(); int randomNum = random.nextInt(num2 - num1) + num1; return randomNum; @@ -223,6 +224,7 @@ public final class SimpleExpressionBuilder { public static Expression randomExpression(final int min, final int max) { return new ExpressionAdapter() { public Object evaluate(Exchange exchange) { + // NOSONAR Random random = new Random(); int randomNum = random.nextInt(max - min) + min; return randomNum; diff --git a/core/camel-core-processor/src/main/java/org/apache/camel/processor/errorhandler/RedeliveryPolicy.java b/core/camel-core-processor/src/main/java/org/apache/camel/processor/errorhandler/RedeliveryPolicy.java index fd41bd0..27cabda 100644 --- a/core/camel-core-processor/src/main/java/org/apache/camel/processor/errorhandler/RedeliveryPolicy.java +++ b/core/camel-core-processor/src/main/java/org/apache/camel/processor/errorhandler/RedeliveryPolicy.java @@ -231,6 +231,7 @@ public class RedeliveryPolicy implements Cloneable, Serializable { * First random determines +/-, second random determines how far to * go in that direction. -cgs */ + // NOSONAR Random random = getRandomNumberGenerator(); double variance = (random.nextBoolean() ? collisionAvoidanceFactor : -collisionAvoidanceFactor) * random.nextDouble(); @@ -563,6 +564,7 @@ public class RedeliveryPolicy implements Cloneable, Serializable { protected static synchronized Random getRandomNumberGenerator() { if (randomNumberGenerator == null) { + // NOSONAR randomNumberGenerator = new Random(); } return randomNumberGenerator;
