You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by ro...@apache.org on 2017/11/07 10:04:21 UTC
[sling-org-apache-sling-scripting-java] 05/16: Use string builder
instead of string buffer
This is an automated email from the ASF dual-hosted git repository.
rombert pushed a commit to annotated tag org.apache.sling.scripting.java-2.0.0
in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-scripting-java.git
commit 27ed977735b9cd5e399eebdd2b56efa3d0ffa66a
Author: Carsten Ziegeler <cz...@apache.org>
AuthorDate: Fri Jan 15 08:38:47 2010 +0000
Use string builder instead of string buffer
git-svn-id: https://svn.apache.org/repos/asf/sling/trunk/contrib/scripting/java@899563 13f79535-47bb-0310-9956-ffa450edef68
---
.../org/apache/sling/scripting/java/CompilationContext.java | 2 +-
.../java/org/apache/sling/scripting/java/CompilerUtil.java | 4 ++--
.../org/apache/sling/scripting/java/jdt/CompilationUnit.java | 12 ++++++------
3 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/src/main/java/org/apache/sling/scripting/java/CompilationContext.java b/src/main/java/org/apache/sling/scripting/java/CompilationContext.java
index 89af03d..ebf646c 100644
--- a/src/main/java/org/apache/sling/scripting/java/CompilationContext.java
+++ b/src/main/java/org/apache/sling/scripting/java/CompilationContext.java
@@ -211,7 +211,7 @@ public class CompilationContext {
protected final static class CompilerException extends ServletException {
public static CompilerException create(List<CompilerError> errors) {
- final StringBuffer buffer = new StringBuffer();
+ final StringBuilder buffer = new StringBuilder();
buffer.append("Compilation errors:\n");
for(final CompilerError e : errors) {
buffer.append(e.getFile());
diff --git a/src/main/java/org/apache/sling/scripting/java/CompilerUtil.java b/src/main/java/org/apache/sling/scripting/java/CompilerUtil.java
index d415ed7..f7d2805 100644
--- a/src/main/java/org/apache/sling/scripting/java/CompilerUtil.java
+++ b/src/main/java/org/apache/sling/scripting/java/CompilerUtil.java
@@ -59,8 +59,8 @@ public class CompilerUtil {
* @return Legal Java identifier corresponding to the given identifier
*/
public static final String makeJavaIdentifier(String identifier) {
- StringBuffer modifiedIdentifier =
- new StringBuffer(identifier.length());
+ StringBuilder modifiedIdentifier =
+ new StringBuilder(identifier.length());
if (!Character.isJavaIdentifierStart(identifier.charAt(0))) {
modifiedIdentifier.append('_');
}
diff --git a/src/main/java/org/apache/sling/scripting/java/jdt/CompilationUnit.java b/src/main/java/org/apache/sling/scripting/java/jdt/CompilationUnit.java
index c34a1c0..e491699 100644
--- a/src/main/java/org/apache/sling/scripting/java/jdt/CompilationUnit.java
+++ b/src/main/java/org/apache/sling/scripting/java/jdt/CompilationUnit.java
@@ -84,7 +84,7 @@ public class CompilationUnit
final Reader reader = new BufferedReader(new InputStreamReader(fr, this.options.getJavaEncoding()));
try {
char[] chars = new char[8192];
- StringBuffer buf = new StringBuffer();
+ StringBuilder buf = new StringBuilder();
int count;
while ((count = reader.read(chars, 0, chars.length)) > 0) {
buf.append(chars, 0, count);
@@ -128,7 +128,7 @@ public class CompilationUnit
* @see org.eclipse.jdt.internal.compiler.env.INameEnvironment#findType(char[][])
*/
public NameEnvironmentAnswer findType(char[][] compoundTypeName) {
- StringBuffer result = new StringBuffer();
+ StringBuilder result = new StringBuilder();
for (int i = 0; i < compoundTypeName.length; i++) {
if (i > 0) {
result.append(".");
@@ -142,7 +142,7 @@ public class CompilationUnit
* @see org.eclipse.jdt.internal.compiler.env.INameEnvironment#findType(char[], char[][])
*/
public NameEnvironmentAnswer findType(char[] typeName, char[][] packageName) {
- StringBuffer result = new StringBuffer();
+ StringBuilder result = new StringBuilder();
for (int i = 0; i < packageName.length; i++) {
if (i > 0) {
result.append(".");
@@ -213,7 +213,7 @@ public class CompilationUnit
* @see org.eclipse.jdt.internal.compiler.env.INameEnvironment#isPackage(char[][], char[])
*/
public boolean isPackage(char[][] parentPackageName, char[] packageName) {
- StringBuffer result = new StringBuffer();
+ StringBuilder result = new StringBuilder();
if (parentPackageName != null) {
for (int i = 0; i < parentPackageName.length; i++) {
if (i > 0) {
@@ -254,7 +254,7 @@ public class CompilationUnit
for (int i = 0; i < classFiles.length; i++) {
ClassFile classFile = classFiles[i];
char[][] compoundName = classFile.getCompoundName();
- StringBuffer className = new StringBuffer();
+ StringBuilder className = new StringBuilder();
for (int j = 0; j < compoundName.length; j++) {
if (j > 0) {
className.append(".");
@@ -262,7 +262,7 @@ public class CompilationUnit
className.append(compoundName[j]);
}
byte[] bytes = classFile.getBytes();
- final StringBuffer b = new StringBuffer(this.options.getDestinationPath());
+ final StringBuilder b = new StringBuilder(this.options.getDestinationPath());
b.append('/');
b.append(className.toString().replace('.', '/'));
b.append(".class");
--
To stop receiving notification emails like this one, please contact
"commits@sling.apache.org" <co...@sling.apache.org>.