stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8298090: Use String.join() instead of manual loop in DescriptorSupport.toString

Browse Source 
Reviewed-by: stsypanov, sspitsyn, lmesnik
This commit is contained in:
Andrey Turbanov 2022-12-12 07:36:34 +00:00
parent a37de62d9d
commit d646e32b7a
1 changed files with 2 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/java.management/share/classes/javax/management/modelmbean/DescriptorSupport.java
View File

@ -1234,14 +1234,13 @@ public class DescriptorSupport
MODELMBEAN_LOGGER.log(Level.TRACE, "Entry");
}
String respStr = "";
String[] fields = getFields();
if ((fields == null) || (fields.length == 0)) {
if (MODELMBEAN_LOGGER.isLoggable(Level.TRACE)) {
MODELMBEAN_LOGGER.log(Level.TRACE, "Empty Descriptor");
}
return respStr;
return "";
}
if (MODELMBEAN_LOGGER.isLoggable(Level.TRACE)) {
@ -1249,13 +1248,7 @@ public class DescriptorSupport
"Printing " + fields.length + " fields");
}
for (int i=0; i < fields.length; i++) {
if (i == (fields.length - 1)) {
respStr = respStr.concat(fields[i]);
} else {
respStr = respStr.concat(fields[i] + ", ");
}
}
String respStr = String.join(", ", fields);
if (MODELMBEAN_LOGGER.isLoggable(Level.TRACE)) {
MODELMBEAN_LOGGER.log(Level.TRACE, "Exit returning " + respStr);