You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by or...@apache.org on 2023/07/08 19:12:41 UTC

[camel] branch main updated: (chores) camel-core: removed unnecessary boxing/unboxing

This is an automated email from the ASF dual-hosted git repository.

orpiske pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git


The following commit(s) were added to refs/heads/main by this push:
     new cb11fda2270 (chores) camel-core: removed unnecessary boxing/unboxing
cb11fda2270 is described below

commit cb11fda2270cd9481bf249ce6c5791f9379b5ec7
Author: Otavio R. Piske <an...@gmail.com>
AuthorDate: Sat Jul 8 20:42:34 2023 +0200

    (chores) camel-core: removed unnecessary boxing/unboxing
    
    Signed-off-by: Otavio R. Piske <an...@gmail.com>
---
 .../java/org/apache/camel/impl/engine/AbstractCamelContext.java   | 2 +-
 .../org/apache/camel/impl/cloud/DefaultServiceDefinition.java     | 2 +-
 .../java/org/apache/camel/catalog/impl/TimePatternConverter.java  | 8 ++++----
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/core/camel-base-engine/src/main/java/org/apache/camel/impl/engine/AbstractCamelContext.java b/core/camel-base-engine/src/main/java/org/apache/camel/impl/engine/AbstractCamelContext.java
index a282225893c..c92234628a6 100644
--- a/core/camel-base-engine/src/main/java/org/apache/camel/impl/engine/AbstractCamelContext.java
+++ b/core/camel-base-engine/src/main/java/org/apache/camel/impl/engine/AbstractCamelContext.java
@@ -3369,7 +3369,7 @@ public abstract class AbstractCamelContext extends BaseService
     public boolean isJMXDisabled() {
         String override = System.getProperty(JmxSystemPropertyKeys.DISABLED);
         if (override != null) {
-            return Boolean.valueOf(override);
+            return Boolean.parseBoolean(override);
         }
 
         return disableJMX;
diff --git a/core/camel-cloud/src/main/java/org/apache/camel/impl/cloud/DefaultServiceDefinition.java b/core/camel-cloud/src/main/java/org/apache/camel/impl/cloud/DefaultServiceDefinition.java
index 8c7eca303dd..f4f36744f8c 100644
--- a/core/camel-cloud/src/main/java/org/apache/camel/impl/cloud/DefaultServiceDefinition.java
+++ b/core/camel-cloud/src/main/java/org/apache/camel/impl/cloud/DefaultServiceDefinition.java
@@ -157,7 +157,7 @@ public class DefaultServiceDefinition implements ServiceDefinition {
                     String servicePort = StringHelper.after(part, ":");
 
                     if (ObjectHelper.isNotEmpty(serviceHost) && ObjectHelper.isNotEmpty(servicePort)) {
-                        return new DefaultServiceDefinition(serviceId, serviceName, serviceHost, Integer.valueOf(servicePort));
+                        return new DefaultServiceDefinition(serviceId, serviceName, serviceHost, Integer.parseInt(servicePort));
                     }
 
                     return null;
diff --git a/core/camel-core-catalog/src/main/java/org/apache/camel/catalog/impl/TimePatternConverter.java b/core/camel-core-catalog/src/main/java/org/apache/camel/catalog/impl/TimePatternConverter.java
index a9df650386c..99cb4adbcf9 100644
--- a/core/camel-core-catalog/src/main/java/org/apache/camel/catalog/impl/TimePatternConverter.java
+++ b/core/camel-core-catalog/src/main/java/org/apache/camel/catalog/impl/TimePatternConverter.java
@@ -51,13 +51,13 @@ public final class TimePatternConverter {
         } else {
             matcher = createMatcher(HOUR_REGEX_PATTERN, source);
             if (matcher.find()) {
-                milliseconds = milliseconds + (3600000 * Long.valueOf(matcher.group(1)));
+                milliseconds = milliseconds + (3600000 * Long.parseLong(matcher.group(1)));
                 foundFlag = true;
             }
 
             matcher = createMatcher(MINUTES_REGEX_PATTERN, source);
             if (matcher.find()) {
-                long minutes = Long.valueOf(matcher.group(1));
+                long minutes = Long.parseLong(matcher.group(1));
                 if (minutes > 59 && foundFlag) {
                     throw new IllegalArgumentException("Minutes should contain a valid value between 0 and 59: " + source);
                 }
@@ -67,7 +67,7 @@ public final class TimePatternConverter {
 
             matcher = createMatcher(SECONDS_REGEX_PATTERN, source);
             if (matcher.find()) {
-                long seconds = Long.valueOf(matcher.group(1));
+                long seconds = Long.parseLong(matcher.group(1));
                 if (seconds > 59 && foundFlag) {
                     throw new IllegalArgumentException("Seconds should contain a valid value between 0 and 59: " + source);
                 }
@@ -78,7 +78,7 @@ public final class TimePatternConverter {
             // No pattern matched... initiating fallback check and conversion (if required).
             // The source at this point may contain illegal values or special characters
             if (!foundFlag) {
-                milliseconds = Long.valueOf(source);
+                milliseconds = Long.parseLong(source);
             }
         }