optimizations: working with Abraham on these: this one avoids trailing ':' which will create a CP hit more frequently.

1 files changed, 8 insertions, 4 deletions

diff --git a/weaver/src/org/aspectj/weaver/bcel/LazyClassGen.java b/weaver/src/org/aspectj/weaver/bcel/LazyClassGen.java
index 018b45309..79ae0c6fc 100644
--- a/weaver/src/org/aspectj/weaver/bcel/LazyClassGen.java
+++ b/weaver/src/org/aspectj/weaver/bcel/LazyClassGen.java
@@ -1389,10 +1389,12 @@ public final class LazyClassGen {
 		if (types == null) {
 			return "";
 		}
-		StringBuffer buf = new StringBuffer();
+		StringBuilder buf = new StringBuilder();
 		for (int i = 0, len = types.length; i < len; i++) {
+			if (i > 0) {
+				buf.append(':');
+			}
 			buf.append(makeString(types[i]));
-			buf.append(':');
 		}
 		return buf.toString();
 	}
@@ -1401,10 +1403,12 @@ public final class LazyClassGen {
 		if (names == null) {
 			return "";
 		}
-		StringBuffer buf = new StringBuffer();
+		StringBuilder buf = new StringBuilder();
 		for (int i = 0, len = names.length; i < len; i++) {
+			if (i > 0) {
+				buf.append(':');
+			}
 			buf.append(names[i]);
-			buf.append(':');
 		}
 		return buf.toString();
 	}