You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by pa...@apache.org on 2020/02/09 10:23:19 UTC
[camel] branch master updated (24303ed -> dfc0b71)
This is an automated email from the ASF dual-hosted git repository.
pascalschumacher pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/camel.git.
from 24303ed Upgrade JSoup to version 1.12.2
new 7d87106 JmsConfiguration#createInOutTemplate: Remove useless comparison test as reported by lgtm.com.
new 637bd26 PackageJaxbMojo#processClasses: Fix logging as reported by lgtm.com
new dfc0b71 AdviceWithTasks#createSelectorIterator: Remove comparison test which is always true (as reported by lgtm.com).
The 3 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
.../src/main/java/org/apache/camel/component/jms/JmsConfiguration.java | 2 +-
.../src/main/java/org/apache/camel/builder/AdviceWithTasks.java | 2 +-
.../src/main/java/org/apache/camel/maven/packaging/PackageJaxbMojo.java | 1 +
3 files changed, 3 insertions(+), 2 deletions(-)
[camel] 01/03: JmsConfiguration#createInOutTemplate: Remove useless
comparison test as reported by lgtm.com.
Posted by pa...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
pascalschumacher pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel.git
commit 7d8710685b517a81835701b4e4c0ceffd4fe1eec
Author: Pascal Schumacher <pa...@gmx.net>
AuthorDate: Sun Feb 9 11:18:43 2020 +0100
JmsConfiguration#createInOutTemplate: Remove useless comparison test as reported by lgtm.com.
---
.../src/main/java/org/apache/camel/component/jms/JmsConfiguration.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsConfiguration.java b/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsConfiguration.java
index e28b2a3..e802670 100644
--- a/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsConfiguration.java
+++ b/components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsConfiguration.java
@@ -653,7 +653,7 @@ public class JmsConfiguration implements Cloneable {
// prefer to use timeToLive over requestTimeout if both specified
long ttl = timeToLive > 0 ? timeToLive : requestTimeout;
- if (ttl > 0 && !isDisableTimeToLive()) {
+ if (!isDisableTimeToLive()) {
// only use TTL if not disabled
jmsTemplate.setTimeToLive(ttl);
}
[camel] 02/03: PackageJaxbMojo#processClasses: Fix logging as
reported by lgtm.com
Posted by pa...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
pascalschumacher pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel.git
commit 637bd2653aff33d5f34c57bed7035445c7545a2f
Author: Pascal Schumacher <pa...@gmx.net>
AuthorDate: Sun Feb 9 11:20:11 2020 +0100
PackageJaxbMojo#processClasses: Fix logging as reported by lgtm.com
---
.../src/main/java/org/apache/camel/maven/packaging/PackageJaxbMojo.java | 1 +
1 file changed, 1 insertion(+)
diff --git a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/PackageJaxbMojo.java b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/PackageJaxbMojo.java
index e497733..c9464d3 100644
--- a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/PackageJaxbMojo.java
+++ b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/PackageJaxbMojo.java
@@ -123,6 +123,7 @@ public class PackageJaxbMojo extends AbstractGeneratorMojo {
sb.append(NL);
}
updateResource(jaxbIndexDir, fn, sb.toString());
+ count++;
}
if (count > 0) {
[camel] 03/03: AdviceWithTasks#createSelectorIterator: Remove
comparison test which is always true (as reported by lgtm.com).
Posted by pa...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
pascalschumacher pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel.git
commit dfc0b7144bd3ec26d6f3a6c73a9a4b1d362a99e5
Author: Pascal Schumacher <pa...@gmx.net>
AuthorDate: Sun Feb 9 11:21:50 2020 +0100
AdviceWithTasks#createSelectorIterator: Remove comparison test which is always true (as reported by lgtm.com).
---
.../src/main/java/org/apache/camel/builder/AdviceWithTasks.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/core/camel-core-engine/src/main/java/org/apache/camel/builder/AdviceWithTasks.java b/core/camel-core-engine/src/main/java/org/apache/camel/builder/AdviceWithTasks.java
index 626e9ca..bd3bb22 100644
--- a/core/camel-core-engine/src/main/java/org/apache/camel/builder/AdviceWithTasks.java
+++ b/core/camel-core-engine/src/main/java/org/apache/camel/builder/AdviceWithTasks.java
@@ -542,7 +542,7 @@ public final class AdviceWithTasks {
// spool to beginning of range
current = selectFrom;
}
- return current >= selectFrom && current <= selectTo;
+ return current <= selectTo;
}
return current < list.size();