You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by ac...@apache.org on 2021/02/16 10:49:40 UTC
[camel] 02/02: CAMEL-16218 - Added NOSONAR comment in camel-core
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 <an...@gmail.com>
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;