You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by mg...@apache.org on 2020/07/13 11:49:48 UTC
[tomcat] branch 9.0.x updated: Simplify some #toString() methods
This is an automated email from the ASF dual-hosted git repository.
mgrigorov pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/9.0.x by this push:
new b81ed7b Simplify some #toString() methods
b81ed7b is described below
commit b81ed7b03d7fcb027b112fd04ddca9353ee80532
Author: Martin Tzvetanov Grigorov <mg...@apache.org>
AuthorDate: Fri Jul 10 17:33:39 2020 +0300
Simplify some #toString() methods
Do not use StringBuilder when String concatenation is just fine.
Append char instead of String when possible.
Fix the toString of ExampleFilter
(cherry picked from commit 768524f02643c58f412c0ecea67bc41ebf4b7085)
---
java/org/apache/catalina/connector/CoyotePrincipal.java | 5 +----
java/org/apache/catalina/core/StandardThreadExecutor.java | 4 +---
java/org/apache/catalina/tribes/UniqueId.java | 6 ++----
java/org/apache/juli/VerbatimFormatter.java | 9 ++-------
.../tomcat/dbcp/dbcp2/datasources/CPDSConnectionFactory.java | 2 +-
.../tomcat/dbcp/dbcp2/datasources/InstanceKeyDataSource.java | 4 ++--
.../apache/tomcat/util/descriptor/web/ContextTransaction.java | 4 +---
java/org/apache/tomcat/util/digester/SetPropertiesRule.java | 4 +---
java/org/apache/tomcat/util/http/fileupload/FileUploadBase.java | 2 +-
webapps/examples/WEB-INF/classes/async/Stockticker.java | 6 +++---
webapps/examples/WEB-INF/classes/filters/ExampleFilter.java | 5 +----
11 files changed, 16 insertions(+), 35 deletions(-)
diff --git a/java/org/apache/catalina/connector/CoyotePrincipal.java b/java/org/apache/catalina/connector/CoyotePrincipal.java
index 93d7c02..c20db11 100644
--- a/java/org/apache/catalina/connector/CoyotePrincipal.java
+++ b/java/org/apache/catalina/connector/CoyotePrincipal.java
@@ -62,10 +62,7 @@ public class CoyotePrincipal implements Principal, Serializable {
*/
@Override
public String toString() {
- StringBuilder sb = new StringBuilder("CoyotePrincipal[");
- sb.append(this.name);
- sb.append(']');
- return sb.toString();
+ return "CoyotePrincipal[" + this.name + "]";
}
diff --git a/java/org/apache/catalina/core/StandardThreadExecutor.java b/java/org/apache/catalina/core/StandardThreadExecutor.java
index caed2d3..9ab3e7d 100644
--- a/java/org/apache/catalina/core/StandardThreadExecutor.java
+++ b/java/org/apache/catalina/core/StandardThreadExecutor.java
@@ -338,8 +338,6 @@ public class StandardThreadExecutor extends LifecycleMBeanBase
@Override
protected String getObjectNameKeyProperties() {
- StringBuilder name = new StringBuilder("type=Executor,name=");
- name.append(getName());
- return name.toString();
+ return "type=Executor,name=" + getName();
}
}
diff --git a/java/org/apache/catalina/tribes/UniqueId.java b/java/org/apache/catalina/tribes/UniqueId.java
index 40e5fba..cfb4387 100644
--- a/java/org/apache/catalina/tribes/UniqueId.java
+++ b/java/org/apache/catalina/tribes/UniqueId.java
@@ -70,9 +70,7 @@ public final class UniqueId implements Serializable{
@Override
public String toString() {
- StringBuilder buf = new StringBuilder("UniqueId");
- buf.append(Arrays.toString(id));
- return buf.toString();
+ return "UniqueId" + Arrays.toString(id);
}
-}
\ No newline at end of file
+}
diff --git a/java/org/apache/juli/VerbatimFormatter.java b/java/org/apache/juli/VerbatimFormatter.java
index a7debe7..c08fce7 100644
--- a/java/org/apache/juli/VerbatimFormatter.java
+++ b/java/org/apache/juli/VerbatimFormatter.java
@@ -31,13 +31,8 @@ public class VerbatimFormatter extends Formatter {
@Override
public String format(LogRecord record) {
- // Timestamp
- StringBuilder sb = new StringBuilder(record.getMessage());
-
- // New line for next record
- sb.append(System.lineSeparator());
-
- return sb.toString();
+ // Timestamp + New line for next record
+ return record.getMessage() + System.lineSeparator();
}
}
diff --git a/java/org/apache/tomcat/dbcp/dbcp2/datasources/CPDSConnectionFactory.java b/java/org/apache/tomcat/dbcp/dbcp2/datasources/CPDSConnectionFactory.java
index 02929e9..915e4cd 100644
--- a/java/org/apache/tomcat/dbcp/dbcp2/datasources/CPDSConnectionFactory.java
+++ b/java/org/apache/tomcat/dbcp/dbcp2/datasources/CPDSConnectionFactory.java
@@ -420,7 +420,7 @@ class CPDSConnectionFactory
builder.append(validatingSet);
builder.append(", pcMap=");
builder.append(pcMap);
- builder.append("]");
+ builder.append(']');
return builder.toString();
}
}
diff --git a/java/org/apache/tomcat/dbcp/dbcp2/datasources/InstanceKeyDataSource.java b/java/org/apache/tomcat/dbcp/dbcp2/datasources/InstanceKeyDataSource.java
index e50e34d..69d6eff 100644
--- a/java/org/apache/tomcat/dbcp/dbcp2/datasources/InstanceKeyDataSource.java
+++ b/java/org/apache/tomcat/dbcp/dbcp2/datasources/InstanceKeyDataSource.java
@@ -1062,9 +1062,9 @@ public abstract class InstanceKeyDataSource implements DataSource, Referenceable
@Override
public synchronized String toString() {
final StringBuilder builder = new StringBuilder(super.toString());
- builder.append("[");
+ builder.append('[');
toStringFields(builder);
- builder.append("]");
+ builder.append(']');
return builder.toString();
}
diff --git a/java/org/apache/tomcat/util/descriptor/web/ContextTransaction.java b/java/org/apache/tomcat/util/descriptor/web/ContextTransaction.java
index 3707d74..54ca6a2 100644
--- a/java/org/apache/tomcat/util/descriptor/web/ContextTransaction.java
+++ b/java/org/apache/tomcat/util/descriptor/web/ContextTransaction.java
@@ -84,8 +84,6 @@ public class ContextTransaction implements Serializable {
*/
@Override
public String toString() {
- StringBuilder sb = new StringBuilder("Transaction[");
- sb.append(']');
- return sb.toString();
+ return "Transaction[]";
}
}
diff --git a/java/org/apache/tomcat/util/digester/SetPropertiesRule.java b/java/org/apache/tomcat/util/digester/SetPropertiesRule.java
index 929af7e..66d82f9 100644
--- a/java/org/apache/tomcat/util/digester/SetPropertiesRule.java
+++ b/java/org/apache/tomcat/util/digester/SetPropertiesRule.java
@@ -131,8 +131,6 @@ public class SetPropertiesRule extends Rule {
*/
@Override
public String toString() {
- StringBuilder sb = new StringBuilder("SetPropertiesRule[");
- sb.append(']');
- return sb.toString();
+ return "SetPropertiesRule[]";
}
}
diff --git a/java/org/apache/tomcat/util/http/fileupload/FileUploadBase.java b/java/org/apache/tomcat/util/http/fileupload/FileUploadBase.java
index 96b2508..9af3772 100644
--- a/java/org/apache/tomcat/util/http/fileupload/FileUploadBase.java
+++ b/java/org/apache/tomcat/util/http/fileupload/FileUploadBase.java
@@ -491,7 +491,7 @@ public abstract class FileUploadBase {
}
// Continuation line found
end = parseEndOfLine(headerPart, nonWs);
- header.append(' ').append(headerPart.substring(nonWs, end));
+ header.append(' ').append(headerPart, nonWs, end);
start = end + 2;
}
parseHeaderLine(headers, header.toString());
diff --git a/webapps/examples/WEB-INF/classes/async/Stockticker.java b/webapps/examples/WEB-INF/classes/async/Stockticker.java
index e87744e..49744b8 100644
--- a/webapps/examples/WEB-INF/classes/async/Stockticker.java
+++ b/webapps/examples/WEB-INF/classes/async/Stockticker.java
@@ -186,11 +186,11 @@ public class Stockticker implements Runnable {
public String toString() {
StringBuilder buf = new StringBuilder("STOCK#");
buf.append(getSymbol());
- buf.append("#");
+ buf.append('#');
buf.append(getValueAsString());
- buf.append("#");
+ buf.append('#');
buf.append(getLastChangeAsString());
- buf.append("#");
+ buf.append('#');
buf.append(String.valueOf(getCnt()));
return buf.toString();
diff --git a/webapps/examples/WEB-INF/classes/filters/ExampleFilter.java b/webapps/examples/WEB-INF/classes/filters/ExampleFilter.java
index 3e0123b..a56c4e5 100644
--- a/webapps/examples/WEB-INF/classes/filters/ExampleFilter.java
+++ b/webapps/examples/WEB-INF/classes/filters/ExampleFilter.java
@@ -93,10 +93,7 @@ public final class ExampleFilter extends GenericFilter {
*/
@Override
public String toString() {
- StringBuilder sb = new StringBuilder("TimingFilter(");
- sb.append(getFilterConfig());
- sb.append(")");
- return sb.toString();
+ return "ExampleFilter(" + getFilterConfig() + ")";
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org