diff --git a/appserver/orb/orb-iiop/src/main/java/org/glassfish/enterprise/iiop/impl/GlassFishORBManager.java b/appserver/orb/orb-iiop/src/main/java/org/glassfish/enterprise/iiop/impl/GlassFishORBManager.java index d07f8fccf25..68b3580d80e 100644 --- a/appserver/orb/orb-iiop/src/main/java/org/glassfish/enterprise/iiop/impl/GlassFishORBManager.java +++ b/appserver/orb/orb-iiop/src/main/java/org/glassfish/enterprise/iiop/impl/GlassFishORBManager.java @@ -37,7 +37,7 @@ * only if the new code is made subject to such option by the copyright * holder. */ -// Portions Copyright [2016-2022] [Payara Foundation and/or its affiliates] +// Portions Copyright [2016-2024] [Payara Foundation and/or its affiliates] package org.glassfish.enterprise.iiop.impl; @@ -952,19 +952,19 @@ private String[] getORBInitRef(String endpoints) { private String getCorbalocURL(Object[] list) { - String corbalocURL = ""; + StringBuilder corbalocURL = new StringBuilder(); //convert list into corbaloc url for (int i = 0; i < list.length; i++) { logger.log(Level.INFO, "list[i] ==> {0}", list[i]); - if (corbalocURL.equals("")) { - corbalocURL = IIOP_URL + ((String) list[i]).trim(); + if (corbalocURL.length()==0) { + corbalocURL.append(IIOP_URL).append(((String) list[i]).trim()); } else { - corbalocURL = corbalocURL + "," + - IIOP_URL + ((String) list[i]).trim(); + corbalocURL.append(",").append(IIOP_URL).append(((String) list[i]).trim()); } } - logger.log(Level.INFO, "corbaloc url ==> {0}", corbalocURL); - return corbalocURL; + String corbalocURLString = corbalocURL.toString(); + logger.info(() -> "corbaloc url ==> " + corbalocURLString); + return corbalocURLString; } String getIIOPEndpoints() { diff --git a/appserver/payara-appserver-modules/microprofile/metrics/src/main/java/fish/payara/microprofile/metrics/impl/MetricsServiceImpl.java b/appserver/payara-appserver-modules/microprofile/metrics/src/main/java/fish/payara/microprofile/metrics/impl/MetricsServiceImpl.java index bfca197d204..bfa17f40dea 100644 --- a/appserver/payara-appserver-modules/microprofile/metrics/src/main/java/fish/payara/microprofile/metrics/impl/MetricsServiceImpl.java +++ b/appserver/payara-appserver-modules/microprofile/metrics/src/main/java/fish/payara/microprofile/metrics/impl/MetricsServiceImpl.java @@ -1,7 +1,7 @@ /* * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER. * - * Copyright (c) [2018-2023] Payara Foundation and/or its affiliates. All rights reserved. + * Copyright (c) [2018-2024] Payara Foundation and/or its affiliates. All rights reserved. * * The contents of this file are subject to the terms of either the GNU * General Public License Version 2 only ("GPL") or the Common Development @@ -366,10 +366,11 @@ private static String toName(MetricID metric, String suffix) { } if (name.indexOf('.') > 0) { String[] words = name.split("\\."); - name = ""; + StringBuilder nameBuilder = new StringBuilder(); for (String word : words) { - name += toFirstLetterUpperCase(word); + nameBuilder.append(toFirstLetterUpperCase(word)); } + name += nameBuilder.toString(); } } name = toFirstLetterUpperCase(name); diff --git a/appserver/payara-appserver-modules/payara-micro-service/src/main/java/fish/payara/appserver/micro/services/asadmin/SendAsadminCommand.java b/appserver/payara-appserver-modules/payara-micro-service/src/main/java/fish/payara/appserver/micro/services/asadmin/SendAsadminCommand.java index 81f5895cc68..f5a863ce92f 100644 --- a/appserver/payara-appserver-modules/payara-micro-service/src/main/java/fish/payara/appserver/micro/services/asadmin/SendAsadminCommand.java +++ b/appserver/payara-appserver-modules/payara-micro-service/src/main/java/fish/payara/appserver/micro/services/asadmin/SendAsadminCommand.java @@ -1,7 +1,7 @@ /* * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER. * - * Copyright (c) [2016-2021] Payara Foundation and/or its affiliates. All rights reserved. + * Copyright (c) [2016-2024] Payara Foundation and/or its affiliates. All rights reserved. * * The contents of this file are subject to the terms of either the GNU * General Public License Version 2 only ("GPL") or the Common Development @@ -202,21 +202,23 @@ public void execute(AdminCommandContext context) // Skip if neither verbose or logOutput were selected if (verbose || logOutput) { - String output = ""; + StringBuilder output = new StringBuilder(); // Combine the success messages into one String for (String successMessage : successMessages) { - output += "\n" + successMessage; + output.append("\n").append(successMessage); } + + String outputString = output.toString(); // Only print out the messages if verbose was chosen if (verbose) { - actionReport.setMessage(output); + actionReport.setMessage(outputString); } // Only log the messages if logOutput was chosen if (logOutput) { - Logger.getLogger(SendAsadminCommand.class.getName()).log(Level.INFO, output); + Logger.getLogger(SendAsadminCommand.class.getName()).info(()->outputString); } }