<Swing Dev> Replace concat String to append in StringBuilder parameters
Otávio Gonçalves de Santana
otaviojava at java.net
Mon Aug 11 03:29:39 UTC 2014
Done.
https://dl.dropboxusercontent.com/u/16109193/open_jdk/string_builder_concat_2.zip
obs: stay the 2 chars to better performance.
On Sun, Aug 10, 2014 at 8:03 PM, Claes Redestad <claes.redestad at oracle.com>
wrote:
> +1
>
> Some suggestions (mostly nits):
>
> - in places like src/share/classes/java/util/regex/Pattern.java you
> introducesingle-char
> strings which might use a char instead:
>
> - result.append("|"+next);
> + result.append('|').append(next);
>
> - in places like src/share/classes/sun/security/provider/PolicyFile.java
> you end up with a sequence of String literal appends which could be
> merged into one:
>
> - sb.append(principalInfo[i][0] + " " +
> - "\"" + principalInfo[i][1] + "\"");
> + sb.append(principalInfo[i][0]).append(" \"")
> + .append(principalInfo[i][1]).append('"');
>
> - in some places like src/share/classes/java/text/ChoiceFormat.java
> you end up doing append(""). I guess the empty string concatenation was
> used as a form
> of primitive-to-string coercion and was probably always redundant
> already, but care needs
> to be taken that doing the append directly behave as intended.
>
> I think it should be safe to just remove the empty string append in most
> cases:
>
> - result.append(""+choiceLimits[i]);
> + result.append(choiceLimits[i]);
>
> Thanks!
>
> /Claes
>
> On 2014-08-10 23:33, Otávio Gonçalves de Santana wrote:
>
>> *Motivation:* Make another append instead of concat String inside of
>> append
>>
>> parameter in StringBuilder class. To avoid an extra StringBuilder created
>> for the purpose of concatenating. So it will save memory and will faster
>> than concat String.
>> Doing a code to benchMark[1], the result is:
>>
>> Benchmark Mode Samples
>> Mean Mean error Units
>> m.StringBuilderConcatBenchMark.stringBuilder thrpt 10
>> 6317444.705 108673.584 ops/s
>> m.StringBuilderConcatBenchMark.stringBuilderWithConcat thrpt 10
>> 3354554.435 68353.924 ops/s
>>
>> The webrev of all code is:
>> https://dl.dropboxusercontent.com/u/16109193/open_jdk/
>> string_builder_concat.zip
>> <https://dl.dropboxusercontent.com/u/16109193/open_jdk/string_
>> builder_concat.zip>
>>
>> [1]
>>
>> @State(Scope.Thread)
>> @OutputTimeUnit(TimeUnit.SECONDS)
>> public class StringBuilderConcatBenchMark {
>>
>>
>> private static final String F = "!!!!";
>> private static final String E = " running in Java ";
>> private static final String D = " in the code ";
>> private static final String C = " to try impact ";
>> private static final String B = " with some text ";
>> private static final String A = "Doing a test";
>>
>> @GenerateMicroBenchmark
>> public void stringBuilder(BlackHole bh) {
>> bh.consume(createBuilder(A, B, C, D, E, F));
>> }
>>
>> @GenerateMicroBenchmark
>> public void stringBuilderWithConcat(BlackHole bh) {
>> bh.consume(createBuilderWithConcat(A, B, C, D, E, F));
>> }
>>
>> private StringBuilder createBuilder(String... values) {
>> StringBuilder text = new StringBuilder();
>> text.append(values[0]).append(values[1])
>> .append(values[2]).append(values[3])
>> .append(values[4]).append(values[5]);
>> return text;
>> }
>> private StringBuilder createBuilderWithConcat(String... values) {
>> StringBuilder text = new StringBuilder();
>> text.append(values[0] + values[1])
>> .append(values[2] + values[3])
>> .append(values[4]+ values[5]);
>> return text;
>> }
>> }
>>
>>
>
--
Otávio Gonçalves de Santana
blog: http://otaviosantana.blogspot.com.br/
twitter: http://twitter.com/otaviojava
site: *http://about.me/otaviojava <http://about.me/otaviojava>*
55 (11) 98255-3513
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20140811/2295b01e/attachment.html>
-------------- next part --------------
diff -r dde9f5cfde5f src/share/classes/sun/tools/jconsole/ThreadTab.java
--- a/src/share/classes/sun/tools/jconsole/ThreadTab.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/tools/jconsole/ThreadTab.java Sun Aug 10 22:07:00 2014 -0300
@@ -368,7 +368,7 @@
sb.append(Messages.STACK_TRACE);
int index = 0;
for (StackTraceElement e : ti.getStackTrace()) {
- sb.append(e.toString()+"\n");
+ sb.append(e.toString()).append('\n');
if (monitors != null) {
for (MonitorInfo mi : monitors) {
if (mi.getLockedStackDepth() == index) {
diff -r dde9f5cfde5f src/share/classes/sun/tools/jconsole/inspector/XArrayDataViewer.java
--- a/src/share/classes/sun/tools/jconsole/inspector/XArrayDataViewer.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/tools/jconsole/inspector/XArrayDataViewer.java Sun Aug 10 22:07:00 2014 -0300
@@ -79,25 +79,24 @@
String textColor = String.format("%06x",
foreground.getRGB() & 0xFFFFFF);
StringBuilder sb = new StringBuilder();
- sb.append("<html><body text=#"+textColor+"><table width=\"100%\">");
+ sb.append("<html><body text=#").append(textColor).append("><table width=\"100%\">");
for (int i = 0; i < arr.length; i++) {
if (i % 2 == 0) {
- sb.append("<tr style=\"background-color: " +
- evenRowColorStr + "\"><td><pre>" +
- (arr[i] == null ?
- arr[i] : htmlize(arr[i].toString())) +
- "</pre></td></tr>");
+ sb.append("<tr style=\"background-color: ")
+ .append(evenRowColorStr).append("\"><td><pre>")
+ .append(arr[i] == null ?
+ arr[i] : htmlize(arr[i].toString()))
+ .append("</pre></td></tr>");
} else {
- sb.append("<tr style=\"background-color: " +
- oddRowColorStr + "\"><td><pre>" +
- (arr[i] == null ?
- arr[i] : htmlize(arr[i].toString())) +
- "</pre></td></tr>");
+ sb.append("<tr style=\"background-color: ")
+ .append(oddRowColorStr).append("\"><td><pre>")
+ .append(arr[i] == null ?
+ arr[i] : htmlize(arr[i].toString()))
+ .append("</pre></td></tr>");
}
}
if (arr.length == 0) {
- sb.append("<tr style=\"background-color: " +
- evenRowColorStr + "\"><td></td></tr>");
+ sb.append("<tr style=\"background-color: ").append(evenRowColorStr).append("\"><td></td></tr>");
}
sb.append("</table></body></html>");
arrayEditor.setText(sb.toString());
diff -r dde9f5cfde5f src/share/classes/sun/tools/jconsole/inspector/XTree.java
--- a/src/share/classes/sun/tools/jconsole/inspector/XTree.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/tools/jconsole/inspector/XTree.java Sun Aug 10 22:07:00 2014 -0300
@@ -507,13 +507,13 @@
// key order defined by the "orderedKeyPropertyList"
for (String key : orderedKeyPropertyList) {
if (map.containsKey(key)) {
- sb.append(key + "=" + map.get(key) + ",");
+ sb.append(key).append('=').append(map.get(key)).append(',');
map.remove(key);
}
}
// Add the remaining key/value pairs to the buffer
for (Map.Entry<String, String> entry : map.entrySet()) {
- sb.append(entry.getKey() + "=" + entry.getValue() + ",");
+ sb.append(entry.getKey()).append('=').append(entry.getValue()).append(',');
}
String orderedKeyPropertyListString = sb.toString();
orderedKeyPropertyListString = orderedKeyPropertyListString.substring(
@@ -622,7 +622,7 @@
//
StringBuilder sb = new StringBuilder();
for (MBeanParameterInfo mbpi : mboi.getSignature()) {
- sb.append(mbpi.getType() + ",");
+ sb.append(mbpi.getType()).append(',');
}
String signature = sb.toString();
if (signature.length() > 0) {
diff -r dde9f5cfde5f src/share/classes/sun/tools/jps/Jps.java
--- a/src/share/classes/sun/tools/jps/Jps.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/tools/jps/Jps.java Sun Aug 10 22:07:00 2014 -0300
@@ -92,28 +92,28 @@
vm = monitoredHost.getMonitoredVm(id, 0);
errorString = " -- main class information unavailable";
- output.append(" " + MonitoredVmUtil.mainClass(vm,
+ output.append(' ').append(MonitoredVmUtil.mainClass(vm,
arguments.showLongPaths()));
if (arguments.showMainArgs()) {
errorString = " -- main args information unavailable";
String mainArgs = MonitoredVmUtil.mainArgs(vm);
if (mainArgs != null && mainArgs.length() > 0) {
- output.append(" " + mainArgs);
+ output.append(' ').append(mainArgs);
}
}
if (arguments.showVmArgs()) {
errorString = " -- jvm args information unavailable";
String jvmArgs = MonitoredVmUtil.jvmArgs(vm);
if (jvmArgs != null && jvmArgs.length() > 0) {
- output.append(" " + jvmArgs);
+ output.append(' ').append(jvmArgs);
}
}
if (arguments.showVmFlags()) {
errorString = " -- jvm flags information unavailable";
String jvmFlags = MonitoredVmUtil.jvmFlags(vm);
if (jvmFlags != null && jvmFlags.length() > 0) {
- output.append(" " + jvmFlags);
+ output.append(' ').append(jvmFlags);
}
}
diff -r dde9f5cfde5f src/share/classes/sun/tools/jstat/RawOutputFormatter.java
--- a/src/share/classes/sun/tools/jstat/RawOutputFormatter.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/tools/jstat/RawOutputFormatter.java Sun Aug 10 22:07:00 2014 -0300
@@ -50,7 +50,7 @@
StringBuilder headerBuilder = new StringBuilder();
for (Iterator<Monitor> i = logged.iterator(); i.hasNext(); /* empty */ ) {
Monitor m = i.next();
- headerBuilder.append(m.getName() + " ");
+ headerBuilder.append(m.getName()).append(' ');
}
header = headerBuilder.toString();
}
diff -r dde9f5cfde5f src/share/classes/sun/tools/jstat/SyntaxException.java
--- a/src/share/classes/sun/tools/jstat/SyntaxException.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/tools/jstat/SyntaxException.java Sun Aug 10 22:07:00 2014 -0300
@@ -65,7 +65,7 @@
public SyntaxException(int lineno, Set<String> expected, Token found) {
StringBuilder msg = new StringBuilder();
- msg.append("Syntax error at line " + lineno + ": Expected one of \'");
+ msg.append("Syntax error at line ").append(lineno).append(": Expected one of \'");
boolean first = true;
for (Iterator<String> i = expected.iterator(); i.hasNext(); /* empty */) {
@@ -74,11 +74,11 @@
msg.append(keyWord);
first = false;
} else {
- msg.append("|" + keyWord);
+ msg.append('|').append(keyWord);
}
}
- msg.append("\', Found " + found.toMessage());
+ msg.append("\', Found ").append(found.toMessage());
message = msg.toString();
}
diff -r dde9f5cfde5f src/share/classes/sun/tools/jstat/Token.java
--- a/src/share/classes/sun/tools/jstat/Token.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/tools/jstat/Token.java Sun Aug 10 22:07:00 2014 -0300
@@ -89,18 +89,18 @@
sb.append("ttype=TT_EOF");
break;
case StreamTokenizer.TT_NUMBER:
- sb.append("ttype=TT_NUM,").append("nval="+nval);
+ sb.append("ttype=TT_NUM,").append("nval=").append(nval);
break;
case StreamTokenizer.TT_WORD:
if (sval == null) {
sb.append("ttype=TT_WORD:IDENTIFIER");
} else {
- sb.append("ttype=TT_WORD:").append("sval="+sval);
+ sb.append("ttype=TT_WORD:").append("sval=").append(sval);
}
break;
default:
if (ttype == (int)'"') {
- sb.append("ttype=TT_STRING:").append("sval="+sval);
+ sb.append("ttype=TT_STRING:").append("sval=").append(sval);
} else {
sb.append("ttype=TT_CHAR:").append((char)ttype);
}
diff -r dde9f5cfde5f src/share/classes/sun/tools/jstatd/RemoteHostImpl.java
--- a/src/share/classes/sun/tools/jstatd/RemoteHostImpl.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/tools/jstatd/RemoteHostImpl.java Sun Aug 10 22:07:00 2014 -0300
@@ -68,7 +68,7 @@
sb.append("local://").append(lvmid).append("@localhost");
if (mode != null) {
- sb.append("?mode=" + mode);
+ sb.append("?mode=").append(mode);
}
String vmidStr = sb.toString();
-------------- next part --------------
diff -r dde9f5cfde5f src/share/classes/sun/security/acl/AclEntryImpl.java
--- a/src/share/classes/sun/security/acl/AclEntryImpl.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/acl/AclEntryImpl.java Sun Aug 10 22:07:00 2014 -0300
@@ -147,13 +147,13 @@
s.append("Group.");
else
s.append("User.");
- s.append(user + "=");
+ s.append(user).append('=');
Enumeration<Permission> e = permissions();
while(e.hasMoreElements()) {
Permission p = e.nextElement();
s.append(p);
if (e.hasMoreElements())
- s.append(",");
+ s.append(',');
}
return new String(s);
}
diff -r dde9f5cfde5f src/share/classes/sun/security/krb5/internal/crypto/dk/DkCrypto.java
--- a/src/share/classes/sun/security/krb5/internal/crypto/dk/DkCrypto.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/krb5/internal/crypto/dk/DkCrypto.java Sun Aug 10 22:07:00 2014 -0300
@@ -640,8 +640,7 @@
for (int i = 0; i < digest.length; i++) {
if ((digest[i] & 0x000000ff) < 0x10) {
- digestString.append("0" +
- Integer.toHexString(digest[i] & 0x000000ff));
+ digestString.append('0').append(Integer.toHexString(digest[i] & 0x000000ff));
} else {
digestString.append(
Integer.toHexString(digest[i] & 0x000000ff));
diff -r dde9f5cfde5f src/share/classes/sun/security/pkcs/SigningCertificateInfo.java
--- a/src/share/classes/sun/security/pkcs/SigningCertificateInfo.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/pkcs/SigningCertificateInfo.java Sun Aug 10 22:07:00 2014 -0300
@@ -158,8 +158,8 @@
}
sb.append(hexDumper.encode(certHash));
if (issuer != null && serialNumber != null) {
- sb.append("\n\tIssuer: " + issuer + "\n");
- sb.append("\t" + serialNumber);
+ sb.append("\n\tIssuer: ").append(issuer).append('\n');
+ sb.append('\t').append(serialNumber);
}
sb.append("\n]");
return sb.toString();
diff -r dde9f5cfde5f src/share/classes/sun/security/pkcs11/P11KeyStore.java
--- a/src/share/classes/sun/security/pkcs11/P11KeyStore.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/pkcs11/P11KeyStore.java Sun Aug 10 22:07:00 2014 -0300
@@ -196,24 +196,24 @@
} else if (type == ATTR_CLASS_CERT) {
sb.append("\ttype=[trusted cert]\n");
}
- sb.append("\tlabel=[" + label + "]\n");
+ sb.append("\tlabel=[").append(label).append("]\n");
if (id == null) {
sb.append("\tid=[null]\n");
} else {
- sb.append("\tid=" + P11KeyStore.getID(id) + "\n");
+ sb.append("\tid=").append(P11KeyStore.getID(id)).append('\n');
}
- sb.append("\ttrusted=[" + trusted + "]\n");
- sb.append("\tmatched=[" + matched + "]\n");
+ sb.append("\ttrusted=[").append(trusted).append("]\n");
+ sb.append("\tmatched=[").append(matched).append("]\n");
if (cert == null) {
sb.append("\tcert=[null]\n");
} else {
- sb.append("\tcert=[\tsubject: " +
- cert.getSubjectX500Principal() +
- "\n\t\tissuer: " +
- cert.getIssuerX500Principal() +
- "\n\t\tserialNum: " +
- cert.getSerialNumber().toString() +
- "]");
+ sb.append("\tcert=[\tsubject: ")
+ .append(cert.getSubjectX500Principal())
+ .append("\n\t\tissuer: ")
+ .append(cert.getIssuerX500Principal())
+ .append("\n\t\tserialNum: ")
+ .append(cert.getSerialNumber().toString())
+ .append(']');
}
return sb.toString();
}
diff -r dde9f5cfde5f src/share/classes/sun/security/provider/PolicyFile.java
--- a/src/share/classes/sun/security/provider/PolicyFile.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/provider/PolicyFile.java Sun Aug 10 22:07:00 2014 -0300
@@ -1500,8 +1500,7 @@
if (i != 0) {
sb.append(", ");
}
- sb.append(principalInfo[i][0] + " " +
- "\"" + principalInfo[i][1] + "\"");
+ sb.append(principalInfo[i][0]).append(' ').append('\"').append(principalInfo[i][1]).append('\"');
}
if (pli.hasNext()) {
sb.append(", ");
@@ -1776,13 +1775,13 @@
if (principals != null && principals.length > 0) {
StringBuilder palBuf = new StringBuilder("(principals ");
for (int i = 0; i < principals.length; i++) {
- palBuf.append(principals[i].getClass().getName() +
- " \"" + principals[i].getName() +
- "\"");
+ palBuf.append(principals[i].getClass().getName())
+ .append(" \"").append(principals[i].getName())
+ .append('\"');
if (i < principals.length-1)
palBuf.append(", ");
else
- palBuf.append(")");
+ palBuf.append(')');
}
pals = palBuf.toString();
}
@@ -1884,7 +1883,7 @@
throw new Exception(form.format(source));
}
- sb.append(X500PRINCIPAL + " \"" + suffix + "\"");
+ sb.append(X500PRINCIPAL).append(" \"").append(suffix).append('\"');
startIndex = e+2;
} else {
MessageFormat form = new MessageFormat
diff -r dde9f5cfde5f src/share/classes/sun/security/provider/certpath/CertId.java
--- a/src/share/classes/sun/security/provider/certpath/CertId.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/provider/certpath/CertId.java Sun Aug 10 22:07:00 2014 -0300
@@ -223,13 +223,13 @@
@Override public String toString() {
StringBuilder sb = new StringBuilder();
sb.append("CertId \n");
- sb.append("Algorithm: " + hashAlgId.toString() +"\n");
+ sb.append("Algorithm: ").append(hashAlgId.toString()).append('\n');
sb.append("issuerNameHash \n");
HexDumpEncoder encoder = new HexDumpEncoder();
sb.append(encoder.encode(issuerNameHash));
sb.append("\nissuerKeyHash: \n");
sb.append(encoder.encode(issuerKeyHash));
- sb.append("\n" + certSerialNumber.toString());
+ sb.append('\n').append(certSerialNumber.toString());
return sb.toString();
}
}
diff -r dde9f5cfde5f src/share/classes/sun/security/provider/certpath/OCSPResponse.java
--- a/src/share/classes/sun/security/provider/certpath/OCSPResponse.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/provider/certpath/OCSPResponse.java Sun Aug 10 22:07:00 2014 -0300
@@ -812,14 +812,14 @@
StringBuilder sb = new StringBuilder();
sb.append("SingleResponse: \n");
sb.append(certId);
- sb.append("\nCertStatus: "+ certStatus + "\n");
+ sb.append("\nCertStatus: ").append(certStatus).append('\n');
if (certStatus == CertStatus.REVOKED) {
- sb.append("revocationTime is " + revocationTime + "\n");
- sb.append("revocationReason is " + revocationReason + "\n");
+ sb.append("revocationTime is ").append(revocationTime).append('\n');
+ sb.append("revocationReason is ").append(revocationReason).append('\n');
}
- sb.append("thisUpdate is " + thisUpdate + "\n");
+ sb.append("thisUpdate is ").append(thisUpdate).append('\n');
if (nextUpdate != null) {
- sb.append("nextUpdate is " + nextUpdate + "\n");
+ sb.append("nextUpdate is ").append(nextUpdate).append('\n');
}
return sb.toString();
}
diff -r dde9f5cfde5f src/share/classes/sun/security/provider/certpath/SunCertPathBuilderParameters.java
--- a/src/share/classes/sun/security/provider/certpath/SunCertPathBuilderParameters.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/provider/certpath/SunCertPathBuilderParameters.java Sun Aug 10 22:07:00 2014 -0300
@@ -124,7 +124,7 @@
StringBuilder sb = new StringBuilder();
sb.append("[\n");
sb.append(super.toString());
- sb.append(" Build Forward Flag: " + String.valueOf(buildForward) + "\n");
+ sb.append(" Build Forward Flag: ").append(String.valueOf(buildForward)).append('\n');
sb.append("]\n");
return sb.toString();
}
diff -r dde9f5cfde5f src/share/classes/sun/security/ssl/HandshakeMessage.java
--- a/src/share/classes/sun/security/ssl/HandshakeMessage.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/ssl/HandshakeMessage.java Sun Aug 10 22:07:00 2014 -0300
@@ -1478,7 +1478,7 @@
boolean opened = false;
for (SignatureAndHashAlgorithm signAlg : algorithms) {
if (opened) {
- sb.append(", " + signAlg.getAlgorithmName());
+ sb.append(", ").append(signAlg.getAlgorithmName());
} else {
sb.append(signAlg.getAlgorithmName());
opened = true;
diff -r dde9f5cfde5f src/share/classes/sun/security/ssl/ServerNameExtension.java
--- a/src/share/classes/sun/security/ssl/ServerNameExtension.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/ssl/ServerNameExtension.java Sun Aug 10 22:07:00 2014 -0300
@@ -268,7 +268,7 @@
public String toString() {
StringBuilder sb = new StringBuilder();
for (SNIServerName sniName : sniMap.values()) {
- sb.append("[" + sniName + "]");
+ sb.append('[').append(sniName).append(']');
}
return "Extension " + type + ", server_name: " + sb;
diff -r dde9f5cfde5f src/share/classes/sun/security/ssl/SignatureAlgorithmsExtension.java
--- a/src/share/classes/sun/security/ssl/SignatureAlgorithmsExtension.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/ssl/SignatureAlgorithmsExtension.java Sun Aug 10 22:07:00 2014 -0300
@@ -122,7 +122,7 @@
boolean opened = false;
for (SignatureAndHashAlgorithm signAlg : algorithms) {
if (opened) {
- sb.append(", " + signAlg.getAlgorithmName());
+ sb.append(", ").append(signAlg.getAlgorithmName());
} else {
sb.append(signAlg.getAlgorithmName());
opened = true;
diff -r dde9f5cfde5f src/share/classes/sun/security/ssl/SupportedEllipticCurvesExtension.java
--- a/src/share/classes/sun/security/ssl/SupportedEllipticCurvesExtension.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/ssl/SupportedEllipticCurvesExtension.java Sun Aug 10 22:07:00 2014 -0300
@@ -113,7 +113,7 @@
@Override
public String toString() {
StringBuilder sb = new StringBuilder();
- sb.append("Extension " + type + ", curve names: {");
+ sb.append("Extension ").append(type).append(", curve names: {");
boolean first = true;
for (int curveId : curveIds) {
if (first) {
@@ -138,7 +138,7 @@
} else if (curveId == ARBITRARY_CHAR2) {
sb.append("arbitrary_explicit_char2_curves");
} else {
- sb.append("unknown curve " + curveId);
+ sb.append("unknown curve ").append(curveId);
}
}
sb.append("}");
diff -r dde9f5cfde5f src/share/classes/sun/security/tools/jarsigner/Main.java
--- a/src/share/classes/sun/security/tools/jarsigner/Main.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/tools/jarsigner/Main.java Sun Aug 10 22:07:00 2014 -0300
@@ -676,14 +676,13 @@
((man.getAttributes(name) != null) ||
(man.getAttributes("./"+name) != null) ||
(man.getAttributes("/"+name) != null));
- sb.append(
- (isSigned ? rb.getString("s") : rb.getString("SPACE")) +
- (inManifest ? rb.getString("m") : rb.getString("SPACE")) +
- (inStore ? rb.getString("k") : rb.getString("SPACE")) +
- (inScope ? rb.getString("i") : rb.getString("SPACE")) +
- ((inStoreOrScope & NOT_ALIAS) != 0 ?"X":" ") +
- rb.getString("SPACE"));
- sb.append("|");
+ sb.append(isSigned ? rb.getString("s") : rb.getString("SPACE"))
+ .append(inManifest ? rb.getString("m") : rb.getString("SPACE"))
+ .append(inStore ? rb.getString("k") : rb.getString("SPACE"))
+ .append(inScope ? rb.getString("i") : rb.getString("SPACE"))
+ .append((inStoreOrScope & NOT_ALIAS) != 0 ? 'X' : ' ')
+ .append(rb.getString("SPACE"));
+ sb.append('|');
}
// When -certs provided, display info has extra empty
@@ -704,11 +703,11 @@
// Print no info for unsigned entries when -verbose:all,
// to be consistent with old behavior.
if (signatureRelated(name)) {
- sb.append("\n" + tab + rb.getString(
- ".Signature.related.entries.") + "\n\n");
+ sb.append('\n').append(tab).append(rb.getString(
+ ".Signature.related.entries.")).append("\n\n");
} else {
- sb.append("\n" + tab + rb.getString(
- ".Unsigned.entries.") + "\n\n");
+ sb.append('\n').append(tab).append(rb.getString(
+ ".Unsigned.entries.")).append("\n\n");
}
}
@@ -1605,8 +1604,8 @@
// No more warning, we alreay have hasExpiredCert or notYetValidCert
} else {
chainNotValidated = true;
- sb.append(tab + rb.getString(".CertPath.not.validated.") +
- e.getLocalizedMessage() + "]\n"); // TODO
+ sb.append(tab).append(rb.getString(".CertPath.not.validated."))
+ .append(e.getLocalizedMessage()).append("]\n"); // TODO
}
}
String result = sb.toString();
diff -r dde9f5cfde5f src/share/classes/sun/security/tools/policytool/PolicyTool.java
--- a/src/share/classes/sun/security/tools/policytool/PolicyTool.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/tools/policytool/PolicyTool.java Sun Aug 10 22:07:00 2014 -0300
@@ -980,8 +980,9 @@
grantEntry.principals.listIterator();
while (list.hasNext()) {
PolicyParser.PrincipalEntry pppe = list.next();
- sb.append(" Principal " + pppe.getDisplayClass() + " " +
- pppe.getDisplayName(true));
+ sb.append(" Principal ").append(pppe.getDisplayClass())
+ .append(' ')
+ .append(pppe.getDisplayName(true));
if (list.hasNext()) sb.append(", ");
}
result = sb.toString();
diff -r dde9f5cfde5f src/share/classes/sun/security/x509/AVA.java
--- a/src/share/classes/sun/security/x509/AVA.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/x509/AVA.java Sun Aug 10 22:07:00 2014 -0300
@@ -1147,7 +1147,7 @@
// Emit the string ... quote it if needed
// if string is already quoted, don't re-quote
if (!alreadyQuoted && quoteNeeded) {
- retval.append("\"" + sbuffer.toString() + "\"");
+ retval.append('\"').append(sbuffer.toString()).append('\"');
} else {
retval.append(sbuffer.toString());
}
diff -r dde9f5cfde5f src/share/classes/sun/security/x509/DistributionPoint.java
--- a/src/share/classes/sun/security/x509/DistributionPoint.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/x509/DistributionPoint.java Sun Aug 10 22:07:00 2014 -0300
@@ -381,22 +381,22 @@
public String toString() {
StringBuilder sb = new StringBuilder();
if (fullName != null) {
- sb.append("DistributionPoint:\n " + fullName + "\n");
+ sb.append("DistributionPoint:\n ").append(fullName).append('\n');
}
if (relativeName != null) {
- sb.append("DistributionPoint:\n " + relativeName + "\n");
+ sb.append("DistributionPoint:\n ").append(relativeName).append('\n');
}
if (reasonFlags != null) {
sb.append(" ReasonFlags:\n");
for (int i = 0; i < reasonFlags.length; i++) {
if (reasonFlags[i]) {
- sb.append(" " + reasonToString(i) + "\n");
+ sb.append(" ").append(reasonToString(i)).append('\n');
}
}
}
if (crlIssuer != null) {
- sb.append(" CRLIssuer:" + crlIssuer + "\n");
+ sb.append(" CRLIssuer:").append(crlIssuer).append('\n');
}
return sb.toString();
}
diff -r dde9f5cfde5f src/share/classes/sun/security/x509/DistributionPointName.java
--- a/src/share/classes/sun/security/x509/DistributionPointName.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/x509/DistributionPointName.java Sun Aug 10 22:07:00 2014 -0300
@@ -231,10 +231,10 @@
public String toString() {
StringBuilder sb = new StringBuilder();
if (fullName != null) {
- sb.append("DistributionPointName:\n " + fullName + "\n");
+ sb.append("DistributionPointName:\n ").append(fullName).append('\n');
} else {
- sb.append("DistributionPointName:\n " + relativeName + "\n");
+ sb.append("DistributionPointName:\n ").append(relativeName).append('\n');
}
return sb.toString();
diff -r dde9f5cfde5f src/share/classes/sun/security/x509/PolicyInformation.java
--- a/src/share/classes/sun/security/x509/PolicyInformation.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/x509/PolicyInformation.java Sun Aug 10 22:07:00 2014 -0300
@@ -259,7 +259,7 @@
*/
public String toString() {
StringBuilder s = new StringBuilder(" [" + policyIdentifier.toString());
- s.append(policyQualifiers + " ]\n");
+ s.append(policyQualifiers).append(" ]\n");
return s.toString();
}
diff -r dde9f5cfde5f src/share/classes/sun/security/x509/X509CRLEntryImpl.java
--- a/src/share/classes/sun/security/x509/X509CRLEntryImpl.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/x509/X509CRLEntryImpl.java Sun Aug 10 22:07:00 2014 -0300
@@ -292,17 +292,17 @@
StringBuilder sb = new StringBuilder();
sb.append(serialNumber.toString());
- sb.append(" On: " + revocationDate.toString());
+ sb.append(" On: ").append(revocationDate.toString());
if (certIssuer != null) {
- sb.append("\n Certificate issuer: " + certIssuer);
+ sb.append("\n Certificate issuer: ").append(certIssuer);
}
if (extensions != null) {
Collection<Extension> allEntryExts = extensions.getAllExtensions();
Extension[] exts = allEntryExts.toArray(new Extension[0]);
- sb.append("\n CRL Entry Extensions: " + exts.length);
+ sb.append("\n CRL Entry Extensions: ").append(exts.length);
for (int i = 0; i < exts.length; i++) {
- sb.append("\n [" + (i+1) + "]: ");
+ sb.append("\n [").append(i + 1).append("]: ");
Extension ext = exts[i];
try {
if (OIDMap.getClass(ext.getExtensionId()) == null) {
@@ -313,9 +313,8 @@
out.putOctetString(extValue);
extValue = out.toByteArray();
HexDumpEncoder enc = new HexDumpEncoder();
- sb.append("Extension unknown: "
- + "DER encoded OCTET string =\n"
- + enc.encodeBuffer(extValue) + "\n");
+ sb.append("Extension unknown: " + "DER encoded OCTET string =\n")
+ .append(enc.encodeBuffer(extValue)).append('\n');
}
} else
sb.append(ext.toString()); //sub-class exists
diff -r dde9f5cfde5f src/share/classes/sun/security/x509/X509CRLImpl.java
--- a/src/share/classes/sun/security/x509/X509CRLImpl.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/x509/X509CRLImpl.java Sun Aug 10 22:07:00 2014 -0300
@@ -537,31 +537,32 @@
*/
public String toString() {
StringBuilder sb = new StringBuilder();
- sb.append("X.509 CRL v" + (version+1) + "\n");
+ sb.append("X.509 CRL v").append(version + 1).append("\n");
if (sigAlgId != null)
- sb.append("Signature Algorithm: " + sigAlgId.toString() +
- ", OID=" + (sigAlgId.getOID()).toString() + "\n");
+ sb.append("Signature Algorithm: ").append(sigAlgId.toString())
+ .append(", OID=")
+ .append((sigAlgId.getOID()).toString()).append('\n');
if (issuer != null)
- sb.append("Issuer: " + issuer.toString() + "\n");
+ sb.append("Issuer: ").append(issuer.toString()).append('\n');
if (thisUpdate != null)
- sb.append("\nThis Update: " + thisUpdate.toString() + "\n");
+ sb.append("\nThis Update: ").append(thisUpdate.toString()).append('\n');
if (nextUpdate != null)
- sb.append("Next Update: " + nextUpdate.toString() + "\n");
+ sb.append("Next Update: ").append(nextUpdate.toString()).append('\n');
if (revokedList.isEmpty())
sb.append("\nNO certificates have been revoked\n");
else {
- sb.append("\nRevoked Certificates: " + revokedList.size());
+ sb.append("\nRevoked Certificates: ").append(revokedList.size());
int i = 1;
for (X509CRLEntry entry: revokedList) {
- sb.append("\n[" + i++ + "] " + entry.toString());
+ sb.append("\n[").append(i++).append("] ").append(entry.toString());
}
}
if (extensions != null) {
Collection<Extension> allExts = extensions.getAllExtensions();
Object[] objs = allExts.toArray();
- sb.append("\nCRL Extensions: " + objs.length);
+ sb.append("\nCRL Extensions: ").append(objs.length);
for (int i = 0; i < objs.length; i++) {
- sb.append("\n[" + (i+1) + "]: ");
+ sb.append("\n[").append(i + 1).append("]: ");
Extension ext = (Extension)objs[i];
try {
if (OIDMap.getClass(ext.getExtensionId()) == null) {
@@ -572,9 +573,8 @@
out.putOctetString(extValue);
extValue = out.toByteArray();
HexDumpEncoder enc = new HexDumpEncoder();
- sb.append("Extension unknown: "
- + "DER encoded OCTET string =\n"
- + enc.encodeBuffer(extValue) + "\n");
+ sb.append("Extension unknown: " + "DER encoded OCTET string =\n")
+ .append(enc.encodeBuffer(extValue)).append('\n');
}
} else
sb.append(ext.toString()); // sub-class exists
@@ -585,8 +585,7 @@
}
if (signature != null) {
HexDumpEncoder encoder = new HexDumpEncoder();
- sb.append("\nSignature:\n" + encoder.encodeBuffer(signature)
- + "\n");
+ sb.append("\nSignature:\n").append(encoder.encodeBuffer(signature)).append('\n');
} else
sb.append("NOT signed yet\n");
return sb.toString();
diff -r dde9f5cfde5f src/share/classes/sun/security/x509/X509CertImpl.java
--- a/src/share/classes/sun/security/x509/X509CertImpl.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/x509/X509CertImpl.java Sun Aug 10 22:07:00 2014 -0300
@@ -802,11 +802,11 @@
StringBuilder sb = new StringBuilder();
sb.append("[\n");
- sb.append(info.toString() + "\n");
- sb.append(" Algorithm: [" + algId.toString() + "]\n");
+ sb.append(info.toString()).append('\n');
+ sb.append(" Algorithm: [").append(algId.toString()).append("]\n");
HexDumpEncoder encoder = new HexDumpEncoder();
- sb.append(" Signature:\n" + encoder.encodeBuffer(signature));
+ sb.append(" Signature:\n").append(encoder.encodeBuffer(signature));
sb.append("\n]");
return sb.toString();
diff -r dde9f5cfde5f src/share/classes/sun/security/x509/X509CertInfo.java
--- a/src/share/classes/sun/security/x509/X509CertInfo.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/security/x509/X509CertInfo.java Sun Aug 10 22:07:00 2014 -0300
@@ -298,27 +298,27 @@
StringBuilder sb = new StringBuilder();
sb.append("[\n");
- sb.append(" " + version.toString() + "\n");
- sb.append(" Subject: " + subject.toString() + "\n");
- sb.append(" Signature Algorithm: " + algId.toString() + "\n");
- sb.append(" Key: " + pubKey.toString() + "\n");
- sb.append(" " + interval.toString() + "\n");
- sb.append(" Issuer: " + issuer.toString() + "\n");
- sb.append(" " + serialNum.toString() + "\n");
+ sb.append(" ").append(version.toString()).append('\n');
+ sb.append(" Subject: ").append(subject.toString()).append('\n');
+ sb.append(" Signature Algorithm: ").append(algId.toString()).append('\n');
+ sb.append(" Key: ").append(pubKey.toString()).append('\n');
+ sb.append(" ").append(interval.toString()).append('\n');
+ sb.append(" Issuer: ").append(issuer.toString()).append('\n');
+ sb.append(" ").append(serialNum.toString()).append('\n');
// optional v2, v3 extras
if (issuerUniqueId != null) {
- sb.append(" Issuer Id:\n" + issuerUniqueId.toString() + "\n");
+ sb.append(" Issuer Id:\n").append(issuerUniqueId.toString()).append('\n');
}
if (subjectUniqueId != null) {
- sb.append(" Subject Id:\n" + subjectUniqueId.toString() + "\n");
+ sb.append(" Subject Id:\n").append(subjectUniqueId.toString()).append('\n');
}
if (extensions != null) {
Collection<Extension> allExts = extensions.getAllExtensions();
Extension[] exts = allExts.toArray(new Extension[0]);
- sb.append("\nCertificate Extensions: " + exts.length);
+ sb.append("\nCertificate Extensions: ").append(exts.length);
for (int i = 0; i < exts.length; i++) {
- sb.append("\n[" + (i+1) + "]: ");
+ sb.append("\n[").append(i + 1).append("]: ");
Extension ext = exts[i];
try {
if (OIDMap.getClass(ext.getExtensionId()) == null) {
@@ -329,9 +329,8 @@
out.putOctetString(extValue);
extValue = out.toByteArray();
HexDumpEncoder enc = new HexDumpEncoder();
- sb.append("Extension unknown: "
- + "DER encoded OCTET string =\n"
- + enc.encodeBuffer(extValue) + "\n");
+ sb.append("Extension unknown: " + "DER encoded OCTET string =\n")
+ .append(enc.encodeBuffer(extValue)).append('\n');
}
} else
sb.append(ext.toString()); //sub-class exists
@@ -341,10 +340,10 @@
}
Map<String,Extension> invalid = extensions.getUnparseableExtensions();
if (invalid.isEmpty() == false) {
- sb.append("\nUnparseable certificate extensions: " + invalid.size());
+ sb.append("\nUnparseable certificate extensions: ").append(invalid.size());
int i = 1;
for (Extension ext : invalid.values()) {
- sb.append("\n[" + (i++) + "]: ");
+ sb.append("\n[").append(i++).append("]: ");
sb.append(ext);
}
}
-------------- next part --------------
diff -r dde9f5cfde5f src/share/classes/sun/net/spi/nameservice/dns/DNSNameService.java
--- a/src/share/classes/sun/net/spi/nameservice/dns/DNSNameService.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/net/spi/nameservice/dns/DNSNameService.java Sun Aug 10 22:07:00 2014 -0300
@@ -465,10 +465,10 @@
private static void appendIfLiteralAddress(String addr, StringBuffer sb) {
if (IPAddressUtil.isIPv4LiteralAddress(addr)) {
- sb.append("dns://" + addr + " ");
+ sb.append("dns://").append(addr).append(' ');
} else {
if (IPAddressUtil.isIPv6LiteralAddress(addr)) {
- sb.append("dns://[" + addr + "] ");
+ sb.append("dns://[").append(addr).append("] ");
}
}
}
diff -r dde9f5cfde5f src/share/classes/sun/net/www/HeaderParser.java
--- a/src/share/classes/sun/net/www/HeaderParser.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/net/www/HeaderParser.java Sun Aug 10 22:07:00 2014 -0300
@@ -221,16 +221,16 @@
public String toString () {
Iterator<String> k = keys();
StringBuffer sbuf = new StringBuffer();
- sbuf.append ("{size="+asize+" nkeys="+nkeys+" ");
+ sbuf.append("{size=").append(asize).append(" nkeys=").append(nkeys).append(' ');
for (int i=0; k.hasNext(); i++) {
String key = k.next();
String val = findValue (i);
if (val != null && "".equals (val)) {
val = null;
}
- sbuf.append (" {"+key+(val==null?"":","+val)+"}");
+ sbuf.append(" {").append(key).append(val == null ? "" : "," + val).append('}');
if (k.hasNext()) {
- sbuf.append (",");
+ sbuf.append (',');
}
}
sbuf.append (" }");
diff -r dde9f5cfde5f src/share/classes/sun/net/www/MimeEntry.java
--- a/src/share/classes/sun/net/www/MimeEntry.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/net/www/MimeEntry.java Sun Aug 10 22:07:00 2014 -0300
@@ -288,7 +288,7 @@
int action = getAction();
if (action != MimeEntry.UNKNOWN) {
- sb.append("action=" + actionKeywords[action]);
+ sb.append("action=").append(actionKeywords[action]);
needSeparator = true;
}
@@ -297,7 +297,7 @@
if (needSeparator) {
sb.append(separator);
}
- sb.append("application=" + command);
+ sb.append("application=").append(command);
needSeparator = true;
}
@@ -305,7 +305,7 @@
if (needSeparator) {
sb.append(separator);
}
- sb.append("icon=" + getImageFileName());
+ sb.append("icon=").append(getImageFileName());
needSeparator = true;
}
@@ -314,7 +314,7 @@
if (needSeparator) {
sb.append(separator);
}
- sb.append("file_extensions=" + extensions);
+ sb.append("file_extensions=").append(extensions);
needSeparator = true;
}
@@ -323,7 +323,7 @@
if (needSeparator) {
sb.append(separator);
}
- sb.append("description=" + description);
+ sb.append("description=").append(description);
}
return sb.toString();
-------------- next part --------------
diff -r dde9f5cfde5f src/share/classes/sun/management/Agent.java
--- a/src/share/classes/sun/management/Agent.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/management/Agent.java Sun Aug 10 22:07:00 2014 -0300
@@ -502,7 +502,7 @@
} else {
StringBuilder message = new StringBuilder(params[0]);
for (int i = 1; i < params.length; i++) {
- message.append(" " + params[i]);
+ message.append(' ').append(params[i]);
}
error(key, message.toString());
}
diff -r dde9f5cfde5f src/share/classes/sun/management/MappedMXBeanType.java
--- a/src/share/classes/sun/management/MappedMXBeanType.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/management/MappedMXBeanType.java Sun Aug 10 22:07:00 2014 -0300
@@ -289,7 +289,7 @@
if (et.isPrimitive()) {
className = new StringBuilder(c.getName());
} else {
- className.append("L" + baseElementType.getTypeName() + ";");
+ className.append('L').append(baseElementType.getTypeName()).append(';');
}
try {
mappedTypeClass = Class.forName(className.toString());
@@ -385,7 +385,7 @@
if (elementType instanceof Class && ((Class) elementType).isPrimitive()) {
className = new StringBuilder(gat.toString());
} else {
- className.append("L" + baseElementType.getTypeName() + ";");
+ className.append('L').append(baseElementType.getTypeName()).append(';');
}
try {
mappedTypeClass = Class.forName(className.toString());
-------------- next part --------------
diff -r dde9f5cfde5f src/share/classes/sun/launcher/LauncherHelper.java
--- a/src/share/classes/sun/launcher/LauncherHelper.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/launcher/LauncherHelper.java Sun Aug 10 22:07:00 2014 -0300
@@ -370,8 +370,7 @@
outBuf = outBuf.append(getLocalizedMessage("java.launcher.ergo.message1",
vm));
outBuf = (isServerClass)
- ? outBuf.append(",\n" +
- getLocalizedMessage("java.launcher.ergo.message2") + "\n\n")
+ ? outBuf.append(",\n").append(getLocalizedMessage("java.launcher.ergo.message2")).append("\n\n")
: outBuf.append(".\n\n");
}
-------------- next part --------------
diff -r dde9f5cfde5f src/share/classes/sun/font/AttributeValues.java
--- a/src/share/classes/sun/font/AttributeValues.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/font/AttributeValues.java Sun Aug 10 22:06:59 2014 -0300
@@ -542,7 +542,7 @@
}
}
}
- b.append("[btx=" + baselineTransform + ", ctx=" + charTransform + "]");
+ b.append("[btx=").append(baselineTransform).append(", ctx=").append(charTransform).append(']');
b.append('}');
return b.toString();
}
diff -r dde9f5cfde5f src/share/classes/sun/font/Decoration.java
--- a/src/share/classes/sun/font/Decoration.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/font/Decoration.java Sun Aug 10 22:06:59 2014 -0300
@@ -431,12 +431,12 @@
StringBuilder sb = new StringBuilder();
sb.append(super.toString());
sb.append("[");
- if (fgPaint != null) sb.append("fgPaint: " + fgPaint);
- if (bgPaint != null) sb.append(" bgPaint: " + bgPaint);
+ if (fgPaint != null) sb.append("fgPaint: ").append(fgPaint);
+ if (bgPaint != null) sb.append(" bgPaint: ").append(bgPaint);
if (swapColors) sb.append(" swapColors: true");
if (strikethrough) sb.append(" strikethrough: true");
- if (stdUnderline != null) sb.append(" stdUnderline: " + stdUnderline);
- if (imUnderline != null) sb.append(" imUnderline: " + imUnderline);
+ if (stdUnderline != null) sb.append(" stdUnderline: ").append(stdUnderline);
+ if (imUnderline != null) sb.append(" imUnderline: ").append(imUnderline);
sb.append("]");
return sb.toString();
}
diff -r dde9f5cfde5f src/share/classes/sun/font/StandardGlyphVector.java
--- a/src/share/classes/sun/font/StandardGlyphVector.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/sun/font/StandardGlyphVector.java Sun Aug 10 22:06:59 2014 -0300
@@ -1894,9 +1894,9 @@
}
}
catch(Exception e) {
- buf.append(" " + e.getMessage());
+ buf.append(' ').append(e.getMessage());
}
- buf.append("}");
+ buf.append('}');
return buf;
}
-------------- next part --------------
diff -r dde9f5cfde5f src/share/classes/javax/swing/GroupLayout.java
--- a/src/share/classes/javax/swing/GroupLayout.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/swing/GroupLayout.java Sun Aug 10 22:06:59 2014 -0300
@@ -1239,15 +1239,12 @@
padding = ", userCreated=" + paddingSpring.getUserCreated() +
", matches=" + paddingSpring.getMatchDescription();
}
- buffer.append(indent + spring.getClass().getName() + " " +
- Integer.toHexString(spring.hashCode()) + " " +
- origin +
- ", size=" + spring.getSize() +
- ", alignment=" + spring.getAlignment() +
- " prefs=[" + spring.getMinimumSize(axis) +
- " " + spring.getPreferredSize(axis) +
- " " + spring.getMaximumSize(axis) +
- padding + "]\n");
+ buffer.append(indent).append(spring.getClass().getName())
+ .append(' ').append(Integer.toHexString(spring.hashCode())).append(' ')
+ .append(origin).append(", size=").append(spring.getSize()).append(", alignment=")
+ .append(spring.getAlignment()).append(" prefs=[").append(spring.getMinimumSize(axis)).append(' ')
+ .append(spring.getPreferredSize(axis)).append(' ').append(spring.getMaximumSize(axis))
+ .append(padding).append("]\n");
if (spring instanceof Group) {
List<Spring> springs = ((Group)spring).springs;
indent += " ";
diff -r dde9f5cfde5f src/share/classes/javax/swing/JColorChooser.java
--- a/src/share/classes/javax/swing/JColorChooser.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/swing/JColorChooser.java Sun Aug 10 22:06:59 2014 -0300
@@ -545,8 +545,7 @@
protected String paramString() {
StringBuilder chooserPanelsString = new StringBuilder("");
for (int i=0; i<chooserPanels.length; i++) {
- chooserPanelsString.append("[" + chooserPanels[i].toString()
- + "]");
+ chooserPanelsString.append('[').append(chooserPanels[i].toString()).append(']');
}
String previewPanelString = (previewPanel != null ?
previewPanel.toString() : "");
diff -r dde9f5cfde5f src/share/classes/javax/swing/MultiUIDefaults.java
--- a/src/share/classes/javax/swing/MultiUIDefaults.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/swing/MultiUIDefaults.java Sun Aug 10 22:06:59 2014 -0300
@@ -195,13 +195,13 @@
Enumeration<?> keys = keys();
while (keys.hasMoreElements()) {
Object key = keys.nextElement();
- sb.append(key + "=" + get(key) + ", ");
+ sb.append(key).append('=').append(get(key)).append(", ");
}
int length = sb.length();
if (length > 1) {
sb.delete(length-2, length);
}
- sb.append("}");
+ sb.append('}');
return sb.toString();
}
}
diff -r dde9f5cfde5f src/share/classes/javax/swing/RepaintManager.java
--- a/src/share/classes/javax/swing/RepaintManager.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/swing/RepaintManager.java Sun Aug 10 22:06:59 2014 -0300
@@ -992,7 +992,7 @@
public synchronized String toString() {
StringBuilder sb = new StringBuilder();
if(dirtyComponents != null)
- sb.append("" + dirtyComponents);
+ sb.append(dirtyComponents);
return sb.toString();
}
diff -r dde9f5cfde5f src/share/classes/javax/swing/event/TreeModelEvent.java
--- a/src/share/classes/javax/swing/event/TreeModelEvent.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/swing/event/TreeModelEvent.java Sun Aug 10 22:06:59 2014 -0300
@@ -294,21 +294,20 @@
public String toString() {
StringBuilder sb = new StringBuilder();
- sb.append(getClass().getName() + " " +
- Integer.toString(hashCode()));
+ sb.append(getClass().getName()).append(" ").append(Integer.toString(hashCode()));
if(path != null)
- sb.append(" path " + path);
+ sb.append(" path ").append(path);
if(childIndices != null) {
sb.append(" indices [ ");
for(int counter = 0; counter < childIndices.length; counter++)
- sb.append(Integer.toString(childIndices[counter])+ " ");
- sb.append("]");
+ sb.append(Integer.toString(childIndices[counter])).append(' ');
+ sb.append(']');
}
if(children != null) {
sb.append(" children [ ");
for(int counter = 0; counter < children.length; counter++)
- sb.append(children[counter] + " ");
- sb.append("]");
+ sb.append(children[counter]).append(' ');
+ sb.append(']');
}
return sb.toString();
}
diff -r dde9f5cfde5f src/share/classes/javax/swing/plaf/basic/BasicFileChooserUI.java
--- a/src/share/classes/javax/swing/plaf/basic/BasicFileChooserUI.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/swing/plaf/basic/BasicFileChooserUI.java Sun Aug 10 22:06:59 2014 -0300
@@ -1330,8 +1330,8 @@
for (Object obj : values) {
String val = ((obj == null) ? "" : obj.toString());
- plainBuf.append(val + "\n");
- htmlBuf.append(" <li>" + val + "\n");
+ plainBuf.append(val).append('\n');
+ htmlBuf.append(" <li>").append(val).append('\n');
}
// remove the last newline
diff -r dde9f5cfde5f src/share/classes/javax/swing/plaf/basic/BasicListUI.java
--- a/src/share/classes/javax/swing/plaf/basic/BasicListUI.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/swing/plaf/basic/BasicListUI.java Sun Aug 10 22:06:59 2014 -0300
@@ -2948,8 +2948,8 @@
for (int i = 0; i < values.length; i++) {
Object obj = values[i];
String val = ((obj == null) ? "" : obj.toString());
- plainStr.append(val + "\n");
- htmlStr.append(" <li>" + val + "\n");
+ plainStr.append(val).append('\n');
+ htmlStr.append(" <li>").append(val).append('\n');
}
// remove the last newline
diff -r dde9f5cfde5f src/share/classes/javax/swing/plaf/basic/BasicTableUI.java
--- a/src/share/classes/javax/swing/plaf/basic/BasicTableUI.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/swing/plaf/basic/BasicTableUI.java Sun Aug 10 22:06:59 2014 -0300
@@ -2230,11 +2230,11 @@
for (int col = 0; col < cols.length; col++) {
Object obj = table.getValueAt(rows[row], cols[col]);
String val = ((obj == null) ? "" : obj.toString());
- plainStr.append(val + "\t");
- htmlStr.append(" <td>" + val + "</td>\n");
+ plainStr.append(val).append('\t');
+ htmlStr.append(" <td>").append(val).append("</td>\n");
}
// we want a newline at the end of each line and not a tab
- plainStr.deleteCharAt(plainStr.length() - 1).append("\n");
+ plainStr.deleteCharAt(plainStr.length() - 1).append('\t');
htmlStr.append("</tr>\n");
}
diff -r dde9f5cfde5f src/share/classes/javax/swing/plaf/basic/BasicTreeUI.java
--- a/src/share/classes/javax/swing/plaf/basic/BasicTreeUI.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/swing/plaf/basic/BasicTreeUI.java Sun Aug 10 22:06:59 2014 -0300
@@ -3618,8 +3618,8 @@
boolean leaf = model.isLeaf(node);
String label = getDisplayString(path, true, leaf);
- plainStr.append(label + "\n");
- htmlStr.append(" <li>" + label + "\n");
+ plainStr.append(label).append('\n');
+ htmlStr.append(" <li>").append(label).append('\n');
}
// remove the last newline
diff -r dde9f5cfde5f src/share/classes/javax/swing/tree/DefaultTreeSelectionModel.java
--- a/src/share/classes/javax/swing/tree/DefaultTreeSelectionModel.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/swing/tree/DefaultTreeSelectionModel.java Sun Aug 10 22:06:59 2014 -0300
@@ -1160,15 +1160,15 @@
rows = rowMapper.getRowsForPaths(selection);
else
rows = null;
- sb.append(getClass().getName() + " " + hashCode() + " [ ");
+ sb.append(getClass().getName()).append(' ').append(hashCode()).append(" [ ");
for(int counter = 0; counter < selCount; counter++) {
if(rows != null)
- sb.append(selection[counter].toString() + "@" +
- Integer.toString(rows[counter])+ " ");
+ sb.append(selection[counter].toString()).append('@')
+ .append(Integer.toString(rows[counter])).append(' ');
else
- sb.append(selection[counter].toString() + " ");
+ sb.append(selection[counter].toString()).append(' ');
}
- sb.append("]");
+ sb.append(']');
return sb.toString();
}
-------------- next part --------------
diff -r dde9f5cfde5f src/share/classes/javax/sound/sampled/AudioFileFormat.java
--- a/src/share/classes/javax/sound/sampled/AudioFileFormat.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/sound/sampled/AudioFileFormat.java Sun Aug 10 22:06:59 2014 -0300
@@ -276,19 +276,19 @@
//$$fb2002-11-01: fix for 4672864: AudioFileFormat.toString() throws unexpected NullPointerException
if (type != null) {
- buf.append(type.toString() + " (." + type.getExtension() + ") file");
+ buf.append(type.toString()).append(" (.").append(type.getExtension()).append(") file");
} else {
buf.append("unknown file format");
}
if (byteLength != AudioSystem.NOT_SPECIFIED) {
- buf.append(", byte length: " + byteLength);
+ buf.append(", byte length: ").append(byteLength);
}
- buf.append(", data format: " + format);
+ buf.append(", data format: ").append(format);
if (frameLength != AudioSystem.NOT_SPECIFIED) {
- buf.append(", frame length: " + frameLength);
+ buf.append(", frame length: ").append(frameLength);
}
return new String(buf);
diff -r dde9f5cfde5f src/share/classes/javax/sound/sampled/DataLine.java
--- a/src/share/classes/javax/sound/sampled/DataLine.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/sound/sampled/DataLine.java Sun Aug 10 22:06:59 2014 -0300
@@ -473,17 +473,17 @@
StringBuilder sb = new StringBuilder();
if ( (formats.length == 1) && (formats[0] != null) ) {
- sb.append(" supporting format " + formats[0]);
+ sb.append(" supporting format ").append(formats[0]);
} else if (getFormats().length > 1) {
- sb.append(" supporting " + getFormats().length + " audio formats");
+ sb.append(" supporting ").append(getFormats().length).append(" audio formats");
}
if ( (minBufferSize != AudioSystem.NOT_SPECIFIED) && (maxBufferSize != AudioSystem.NOT_SPECIFIED) ) {
- sb.append(", and buffers of " + minBufferSize + " to " + maxBufferSize + " bytes");
+ sb.append(", and buffers of ").append(minBufferSize).append(" to ").append(maxBufferSize).append(" bytes");
} else if ( (minBufferSize != AudioSystem.NOT_SPECIFIED) && (minBufferSize > 0) ) {
- sb.append(", and buffers of at least " + minBufferSize + " bytes");
+ sb.append(", and buffers of at least ").append(minBufferSize).append(" bytes");
} else if (maxBufferSize != AudioSystem.NOT_SPECIFIED) {
- sb.append(", and buffers of up to " + minBufferSize + " bytes");
+ sb.append(", and buffers of up to ").append(minBufferSize).append(" bytes");
}
return new String(super.toString() + sb);
-------------- next part --------------
diff -r dde9f5cfde5f src/share/classes/javax/security/auth/kerberos/KerberosTicket.java
--- a/src/share/classes/javax/security/auth/kerberos/KerberosTicket.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/security/auth/kerberos/KerberosTicket.java Sun Aug 10 22:06:59 2014 -0300
@@ -651,8 +651,7 @@
StringBuilder caddrString = new StringBuilder();
if (clientAddresses != null) {
for (int i = 0; i < clientAddresses.length; i++) {
- caddrString.append("clientAddresses[" + i + "] = " +
- clientAddresses[i].toString());
+ caddrString.append("clientAddresses[").append(i).append("] = ").append(clientAddresses[i].toString());
}
}
return ("Ticket (hex) = " + "\n" +
-------------- next part --------------
diff -r dde9f5cfde5f src/share/classes/javax/naming/BinaryRefAddr.java
--- a/src/share/classes/javax/naming/BinaryRefAddr.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/naming/BinaryRefAddr.java Sun Aug 10 22:06:59 2014 -0300
@@ -165,11 +165,11 @@
* @return The non-null string representation of this address.
*/
public String toString(){
- StringBuilder str = new StringBuilder("Address Type: " + addrType + "\n");
-
+ StringBuilder str = new StringBuilder();
+ str.append("Address Type: ").append(addrType).append('\n');
str.append("AddressContents: ");
for (int i = 0; i<buf.length && i < 32; i++) {
- str.append(Integer.toHexString(buf[i]) +" ");
+ str.append(Integer.toHexString(buf[i])).append(' ');
}
if (buf.length >= 32)
str.append(" ...\n");
diff -r dde9f5cfde5f src/share/classes/javax/naming/NameImpl.java
--- a/src/share/classes/javax/naming/NameImpl.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/naming/NameImpl.java Sun Aug 10 22:06:59 2014 -0300
@@ -170,7 +170,7 @@
endQuote = one ? syntaxEndQuote1 : syntaxEndQuote2;
i += syntaxTypevalSeparator.length();
- answer.append(syntaxTypevalSeparator+beginQuote); // add back
+ answer.append(syntaxTypevalSeparator).append(beginQuote); // add back
// consume string until matching quote
for (i += beginQuote.length();
diff -r dde9f5cfde5f src/share/classes/javax/naming/RefAddr.java
--- a/src/share/classes/javax/naming/RefAddr.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/naming/RefAddr.java Sun Aug 10 22:06:59 2014 -0300
@@ -139,9 +139,9 @@
* @return The non-null string representation of this address.
*/
public String toString(){
- StringBuilder str = new StringBuilder("Type: " + addrType + "\n");
-
- str.append("Content: " + getContent() + "\n");
+ StringBuilder str = new StringBuilder();
+ str.append("Type: ").append(addrType).append('\n');
+ str.append("Content: ").append(getContent()).append('\n');
return (str.toString());
}
-------------- next part --------------
diff -r dde9f5cfde5f src/share/classes/javax/crypto/CryptoPermission.java
--- a/src/share/classes/javax/crypto/CryptoPermission.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/crypto/CryptoPermission.java Sun Aug 10 22:06:59 2014 -0300
@@ -370,20 +370,18 @@
*/
public String toString() {
StringBuilder buf = new StringBuilder(100);
- buf.append("(CryptoPermission " + alg + " " + maxKeySize);
+ buf.append("(CryptoPermission ").append(alg).append(' ').append(maxKeySize);
if (algParamSpec != null) {
if (algParamSpec instanceof RC2ParameterSpec) {
- buf.append(" , effective " +
- ((RC2ParameterSpec)algParamSpec).getEffectiveKeyBits());
+ buf.append(" , effective ").append(((RC2ParameterSpec) algParamSpec).getEffectiveKeyBits());
} else if (algParamSpec instanceof RC5ParameterSpec) {
- buf.append(" , rounds " +
- ((RC5ParameterSpec)algParamSpec).getRounds());
+ buf.append(" , rounds ").append(((RC5ParameterSpec) algParamSpec).getRounds());
}
}
if (exemptionMechanism != null) { // OPTIONAL
- buf.append(" " + exemptionMechanism);
+ buf.append(' ').append(exemptionMechanism);
}
- buf.append(")");
+ buf.append(')');
return buf.toString();
}
-------------- next part --------------
diff -r dde9f5cfde5f src/share/classes/javax/management/MBeanPermission.java
--- a/src/share/classes/javax/management/MBeanPermission.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/management/MBeanPermission.java Sun Aug 10 22:06:59 2014 -0300
@@ -455,11 +455,11 @@
name.append(className);
if (member == null)
member = "-";
- name.append("#" + member);
+ name.append('#').append(member);
if (objectName == null)
name.append("[-]");
else
- name.append("[").append(objectName.getCanonicalName()).append("]");
+ name.append('[').append(objectName.getCanonicalName()).append(']');
/* In the interests of legibility for Permission.toString(), we
transform the empty string into "*". */
diff -r dde9f5cfde5f src/share/classes/javax/management/modelmbean/RequiredModelMBean.java
--- a/src/share/classes/javax/management/modelmbean/RequiredModelMBean.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/management/modelmbean/RequiredModelMBean.java Sun Aug 10 22:06:59 2014 -0300
@@ -716,15 +716,14 @@
}
retStr.append("\nMBeanInfo for ModelMBean is:");
- retStr.append("\nCLASSNAME: \t"+ info.getClassName());
- retStr.append("\nDESCRIPTION: \t"+ info.getDescription());
+ retStr.append("\nCLASSNAME: \t").append(info.getClassName());
+ retStr.append("\nDESCRIPTION: \t").append(info.getDescription());
try {
- retStr.append("\nMBEAN DESCRIPTOR: \t"+
- info.getMBeanDescriptor());
+ retStr.append("\nMBEAN DESCRIPTOR: \t").append(info.getMBeanDescriptor());
} catch (Exception e) {
- retStr.append("\nMBEAN DESCRIPTOR: \t" + " is invalid");
+ retStr.append("\nMBEAN DESCRIPTOR: \t is invalid");
}
retStr.append("\nATTRIBUTES");
@@ -734,13 +733,12 @@
for (int i=0; i<attrInfo.length; i++) {
final ModelMBeanAttributeInfo attInfo =
(ModelMBeanAttributeInfo)attrInfo[i];
- retStr.append(" ** NAME: \t"+ attInfo.getName());
- retStr.append(" DESCR: \t"+ attInfo.getDescription());
- retStr.append(" TYPE: \t"+ attInfo.getType() +
- " READ: \t"+ attInfo.isReadable() +
- " WRITE: \t"+ attInfo.isWritable());
- retStr.append(" DESCRIPTOR: " +
- attInfo.getDescriptor().toString());
+ retStr.append(" ** NAME: \t").append(attInfo.getName());
+ retStr.append(" DESCR: \t").append(attInfo.getDescription());
+ retStr.append(" TYPE: \t").append(attInfo.getType())
+ .append(" READ: \t").append(attInfo.isReadable())
+ .append(" WRITE: \t").append(attInfo.isWritable());
+ retStr.append(" DESCRIPTOR: ").append(attInfo.getDescriptor().toString());
}
} else {
retStr.append(" ** No attributes **");
@@ -752,14 +750,12 @@
for (int i=0; i<constrInfo.length; i++) {
final ModelMBeanConstructorInfo ctorInfo =
(ModelMBeanConstructorInfo)constrInfo[i];
- retStr.append(" ** NAME: \t"+ ctorInfo.getName());
- retStr.append(" DESCR: \t"+
- ctorInfo.getDescription());
- retStr.append(" PARAM: \t"+
- ctorInfo.getSignature().length +
- " parameter(s)");
- retStr.append(" DESCRIPTOR: " +
- ctorInfo.getDescriptor().toString());
+ retStr.append(" ** NAME: \t").append(ctorInfo.getName());
+ retStr.append(" DESCR: \t").append(ctorInfo.getDescription());
+ retStr.append(" PARAM: \t").append(ctorInfo.getSignature().length)
+ .append(" parameter(s)");
+ retStr.append(" DESCRIPTOR: ")
+ .append(ctorInfo.getDescriptor().toString());
}
} else {
retStr.append(" ** No Constructors **");
@@ -771,13 +767,11 @@
for (int i=0; i<opsInfo.length; i++) {
final ModelMBeanOperationInfo operInfo =
(ModelMBeanOperationInfo)opsInfo[i];
- retStr.append(" ** NAME: \t"+ operInfo.getName());
- retStr.append(" DESCR: \t"+ operInfo.getDescription());
- retStr.append(" PARAM: \t"+
- operInfo.getSignature().length +
- " parameter(s)");
- retStr.append(" DESCRIPTOR: " +
- operInfo.getDescriptor().toString());
+ retStr.append(" ** NAME: \t").append(operInfo.getName());
+ retStr.append(" DESCR: \t").append(operInfo.getDescription());
+ retStr.append(" PARAM: \t").append(operInfo.getSignature().length)
+ .append(" parameter(s)");
+ retStr.append(" DESCRIPTOR: ").append(operInfo.getDescriptor().toString());
}
} else {
retStr.append(" ** No operations ** ");
@@ -790,10 +784,9 @@
for (int i=0; i<notifInfo.length; i++) {
final ModelMBeanNotificationInfo nInfo =
(ModelMBeanNotificationInfo)notifInfo[i];
- retStr.append(" ** NAME: \t"+ nInfo.getName());
- retStr.append(" DESCR: \t"+ nInfo.getDescription());
- retStr.append(" DESCRIPTOR: " +
- nInfo.getDescriptor().toString());
+ retStr.append(" ** NAME: \t").append(nInfo.getName());
+ retStr.append(" DESCR: \t").append(nInfo.getDescription());
+ retStr.append(" DESCRIPTOR: ").append(nInfo.getDescriptor().toString());
}
} else {
retStr.append(" ** No notifications **");
diff -r dde9f5cfde5f src/share/classes/javax/management/openmbean/CompositeType.java
--- a/src/share/classes/javax/management/openmbean/CompositeType.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/management/openmbean/CompositeType.java Sun Aug 10 22:06:59 2014 -0300
@@ -45,6 +45,7 @@
*
* @since 1.5
*/
+ at SuppressWarnings("StringConcatenationInsideStringBufferAppend")
public class CompositeType extends OpenType<CompositeData> {
/* Serial version */
diff -r dde9f5cfde5f src/share/classes/javax/management/relation/Role.java
--- a/src/share/classes/javax/management/relation/Role.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/management/relation/Role.java Sun Aug 10 22:06:59 2014 -0300
@@ -227,7 +227,7 @@
*/
public String toString() {
StringBuilder result = new StringBuilder();
- result.append("role name: " + name + "; role value: ");
+ result.append("role name: ").append(name).append("; role value: ");
for (Iterator<ObjectName> objNameIter = objectNameList.iterator();
objNameIter.hasNext();) {
ObjectName currObjName = objNameIter.next();
diff -r dde9f5cfde5f src/share/classes/javax/management/relation/RoleInfo.java
--- a/src/share/classes/javax/management/relation/RoleInfo.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/management/relation/RoleInfo.java Sun Aug 10 22:06:59 2014 -0300
@@ -456,13 +456,13 @@
*/
public String toString() {
StringBuilder result = new StringBuilder();
- result.append("role info name: " + name);
- result.append("; isReadable: " + isReadable);
- result.append("; isWritable: " + isWritable);
- result.append("; description: " + description);
- result.append("; minimum degree: " + minDegree);
- result.append("; maximum degree: " + maxDegree);
- result.append("; MBean class: " + referencedMBeanClassName);
+ result.append("role info name: ").append(name);
+ result.append("; isReadable: ").append(isReadable);
+ result.append("; isWritable: ").append(isWritable);
+ result.append("; description: ").append(description);
+ result.append("; minimum degree: ").append(minDegree);
+ result.append("; maximum degree: ").append(maxDegree);
+ result.append("; MBean class: ").append(referencedMBeanClassName);
return result.toString();
}
diff -r dde9f5cfde5f src/share/classes/javax/management/relation/RoleUnresolved.java
--- a/src/share/classes/javax/management/relation/RoleUnresolved.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/javax/management/relation/RoleUnresolved.java Sun Aug 10 22:06:59 2014 -0300
@@ -282,7 +282,7 @@
*/
public String toString() {
StringBuilder result = new StringBuilder();
- result.append("role name: " + roleName);
+ result.append("role name: ").append(roleName);
if (roleValue != null) {
result.append("; value: ");
for (Iterator<ObjectName> objNameIter = roleValue.iterator();
@@ -294,7 +294,7 @@
}
}
}
- result.append("; problem type: " + problemType);
+ result.append("; problem type: ").append(problemType);
return result.toString();
}
-------------- next part --------------
diff -r dde9f5cfde5f src/share/classes/java/util/Scanner.java
--- a/src/share/classes/java/util/Scanner.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/util/Scanner.java Sun Aug 10 22:06:59 2014 -0300
@@ -1304,20 +1304,20 @@
public String toString() {
StringBuilder sb = new StringBuilder();
sb.append("java.util.Scanner");
- sb.append("[delimiters=" + delimPattern + "]");
- sb.append("[position=" + position + "]");
- sb.append("[match valid=" + matchValid + "]");
- sb.append("[need input=" + needInput + "]");
- sb.append("[source closed=" + sourceClosed + "]");
- sb.append("[skipped=" + skipped + "]");
- sb.append("[group separator=" + groupSeparator + "]");
- sb.append("[decimal separator=" + decimalSeparator + "]");
- sb.append("[positive prefix=" + positivePrefix + "]");
- sb.append("[negative prefix=" + negativePrefix + "]");
- sb.append("[positive suffix=" + positiveSuffix + "]");
- sb.append("[negative suffix=" + negativeSuffix + "]");
- sb.append("[NaN string=" + nanString + "]");
- sb.append("[infinity string=" + infinityString + "]");
+ sb.append("[delimiters=").append(delimPattern).append(']');
+ sb.append("[position=").append(position).append(']');
+ sb.append("[match valid=").append(matchValid).append(']');
+ sb.append("[need input=").append(needInput).append(']');
+ sb.append("[source closed=").append(sourceClosed).append(']');
+ sb.append("[skipped=").append(skipped).append(']');
+ sb.append("[group separator=").append(groupSeparator).append(']');
+ sb.append("[decimal separator=").append(decimalSeparator).append(']');
+ sb.append("[positive prefix=").append(positivePrefix).append(']');
+ sb.append("[negative prefix=").append(negativePrefix).append(']');
+ sb.append("[positive suffix=").append(positiveSuffix).append(']');
+ sb.append("[negative suffix=").append(negativeSuffix).append(']');
+ sb.append("[NaN string=").append(nanString).append(']');
+ sb.append("[infinity string=").append(infinityString).append(']');
return sb.toString();
}
diff -r dde9f5cfde5f src/share/classes/java/util/regex/Matcher.java
--- a/src/share/classes/java/util/regex/Matcher.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/util/regex/Matcher.java Sun Aug 10 22:06:59 2014 -0300
@@ -1295,14 +1295,14 @@
public String toString() {
StringBuilder sb = new StringBuilder();
sb.append("java.util.regex.Matcher");
- sb.append("[pattern=" + pattern());
+ sb.append("[pattern=").append(pattern());
sb.append(" region=");
- sb.append(regionStart() + "," + regionEnd());
+ sb.append(regionStart()).append(',').append(regionEnd());
sb.append(" lastmatch=");
if ((first >= 0) && (group() != null)) {
sb.append(group());
}
- sb.append("]");
+ sb.append(']');
return sb.toString();
}
diff -r dde9f5cfde5f src/share/classes/java/util/regex/Pattern.java
--- a/src/share/classes/java/util/regex/Pattern.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/util/regex/Pattern.java Sun Aug 10 22:06:59 2014 -0300
@@ -1488,10 +1488,10 @@
for(int x=0; x<perms.length; x++) {
String next = base + perms[x];
if (x>0)
- result.append("|"+next);
+ result.append('|').append(next);
next = composeOneStep(next);
if (next != null)
- result.append("|"+produceEquivalentAlternation(next));
+ result.append('|').append(produceEquivalentAlternation(next));
}
return result.toString();
}
-------------- next part --------------
diff -r dde9f5cfde5f src/share/classes/java/text/ChoiceFormat.java
--- a/src/share/classes/java/text/ChoiceFormat.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/text/ChoiceFormat.java Sun Aug 10 22:06:58 2014 -0300
@@ -272,7 +272,7 @@
double tryLess = Math.abs(Math.IEEEremainder(less, 1.0d));
if (tryLessOrEqual < tryLess) {
- result.append(""+choiceLimits[i]);
+ result.append(choiceLimits[i]);
result.append('#');
} else {
if (choiceLimits[i] == Double.POSITIVE_INFINITY) {
@@ -280,7 +280,7 @@
} else if (choiceLimits[i] == Double.NEGATIVE_INFINITY) {
result.append("-\u221E");
} else {
- result.append(""+less);
+ result.append(less);
}
result.append('<');
}
-------------- next part --------------
diff -r dde9f5cfde5f src/share/classes/java/security/CodeSigner.java
--- a/src/share/classes/java/security/CodeSigner.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/security/CodeSigner.java Sun Aug 10 22:06:58 2014 -0300
@@ -156,9 +156,9 @@
public String toString() {
StringBuilder sb = new StringBuilder();
sb.append("(");
- sb.append("Signer: " + signerCertPath.getCertificates().get(0));
+ sb.append("Signer: ").append(signerCertPath.getCertificates().get(0));
if (timestamp != null) {
- sb.append("timestamp: " + timestamp);
+ sb.append("timestamp: ").append(timestamp);
}
sb.append(")");
return sb.toString();
diff -r dde9f5cfde5f src/share/classes/java/security/CodeSource.java
--- a/src/share/classes/java/security/CodeSource.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/security/CodeSource.java Sun Aug 10 22:06:58 2014 -0300
@@ -468,12 +468,12 @@
if (this.certs != null && this.certs.length > 0) {
for (int i = 0; i < this.certs.length; i++) {
- sb.append( " " + this.certs[i]);
+ sb.append(' ').append(this.certs[i]);
}
} else if (this.signers != null && this.signers.length > 0) {
for (int i = 0; i < this.signers.length; i++) {
- sb.append( " " + this.signers[i]);
+ sb.append(' ').append(this.signers[i]);
}
} else {
sb.append(" <no signer certificates>");
diff -r dde9f5cfde5f src/share/classes/java/security/KeyStore.java
--- a/src/share/classes/java/security/KeyStore.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/security/KeyStore.java Sun Aug 10 22:06:58 2014 -0300
@@ -610,8 +610,7 @@
*/
public String toString() {
StringBuilder sb = new StringBuilder();
- sb.append("Private key entry and certificate chain with "
- + chain.length + " elements:\r\n");
+ sb.append("Private key entry and certificate chain with ").append(chain.length).append(" elements:\r\n");
for (Certificate cert : chain) {
sb.append(cert);
sb.append("\r\n");
diff -r dde9f5cfde5f src/share/classes/java/security/PermissionCollection.java
--- a/src/share/classes/java/security/PermissionCollection.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/security/PermissionCollection.java Sun Aug 10 22:06:58 2014 -0300
@@ -179,12 +179,12 @@
public String toString() {
Enumeration<Permission> enum_ = elements();
StringBuilder sb = new StringBuilder();
- sb.append(super.toString()+" (\n");
+ sb.append(super.toString()).append(" (\n");
while (enum_.hasMoreElements()) {
try {
- sb.append(" ");
+ sb.append(' ');
sb.append(enum_.nextElement().toString());
- sb.append("\n");
+ sb.append('\n');
} catch (NoSuchElementException e){
// ignore
}
diff -r dde9f5cfde5f src/share/classes/java/security/ProtectionDomain.java
--- a/src/share/classes/java/security/ProtectionDomain.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/security/ProtectionDomain.java Sun Aug 10 22:06:58 2014 -0300
@@ -291,9 +291,9 @@
StringBuilder palBuf = new StringBuilder("(principals ");
for (int i = 0; i < principals.length; i++) {
- palBuf.append(principals[i].getClass().getName() +
- " \"" + principals[i].getName() +
- "\"");
+ palBuf.append(principals[i].getClass().getName())
+ .append(" \"").append(principals[i].getName())
+ .append('\"');
if (i < principals.length-1)
palBuf.append(",\n");
else
diff -r dde9f5cfde5f src/share/classes/java/security/Timestamp.java
--- a/src/share/classes/java/security/Timestamp.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/security/Timestamp.java Sun Aug 10 22:06:58 2014 -0300
@@ -143,10 +143,10 @@
public String toString() {
StringBuilder sb = new StringBuilder();
sb.append("(");
- sb.append("timestamp: " + timestamp);
+ sb.append("timestamp: ").append(timestamp);
List<? extends Certificate> certs = signerCertPath.getCertificates();
if (!certs.isEmpty()) {
- sb.append("TSA: " + certs.get(0));
+ sb.append("TSA: ").append(certs.get(0));
} else {
sb.append("TSA: <empty>");
}
diff -r dde9f5cfde5f src/share/classes/java/security/cert/CertPath.java
--- a/src/share/classes/java/security/cert/CertPath.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/security/cert/CertPath.java Sun Aug 10 22:06:58 2014 -0300
@@ -224,17 +224,16 @@
Iterator<? extends Certificate> stringIterator =
getCertificates().iterator();
- sb.append("\n" + type + " Cert Path: length = "
- + getCertificates().size() + ".\n");
+ sb.append("\n").append(type).append(" Cert Path: length = ").append(getCertificates().size()).append(".\n");
sb.append("[\n");
int i = 1;
while (stringIterator.hasNext()) {
- sb.append("=========================================="
- + "===============Certificate " + i + " start.\n");
+ sb.append("==========================================" + "===============Certificate ")
+ .append(i).append(" start.\n");
Certificate stringCert = stringIterator.next();
sb.append(stringCert.toString());
- sb.append("\n========================================"
- + "=================Certificate " + i + " end.\n\n\n");
+ sb.append("\n========================================" + "=================Certificate ")
+ .append(i).append(" end.\n\n\n");
i++;
}
diff -r dde9f5cfde5f src/share/classes/java/security/cert/CollectionCertStoreParameters.java
--- a/src/share/classes/java/security/cert/CollectionCertStoreParameters.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/security/cert/CollectionCertStoreParameters.java Sun Aug 10 22:06:58 2014 -0300
@@ -134,8 +134,8 @@
public String toString() {
StringBuilder sb = new StringBuilder();
sb.append("CollectionCertStoreParameters: [\n");
- sb.append(" collection: " + coll + "\n");
- sb.append("]");
+ sb.append(" collection: ").append(coll).append('\n');
+ sb.append(']');
return sb.toString();
}
}
diff -r dde9f5cfde5f src/share/classes/java/security/cert/LDAPCertStoreParameters.java
--- a/src/share/classes/java/security/cert/LDAPCertStoreParameters.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/security/cert/LDAPCertStoreParameters.java Sun Aug 10 22:06:58 2014 -0300
@@ -141,9 +141,9 @@
StringBuilder sb = new StringBuilder();
sb.append("LDAPCertStoreParameters: [\n");
- sb.append(" serverName: " + serverName + "\n");
- sb.append(" port: " + port + "\n");
- sb.append("]");
+ sb.append(" serverName: ").append(serverName).append("\n");
+ sb.append(" port: ").append(port).append('\n');
+ sb.append(']');
return sb.toString();
}
}
diff -r dde9f5cfde5f src/share/classes/java/security/cert/PKIXBuilderParameters.java
--- a/src/share/classes/java/security/cert/PKIXBuilderParameters.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/security/cert/PKIXBuilderParameters.java Sun Aug 10 22:06:58 2014 -0300
@@ -192,7 +192,7 @@
StringBuilder sb = new StringBuilder();
sb.append("[\n");
sb.append(super.toString());
- sb.append(" Maximum Path Length: " + maxPathLength + "\n");
+ sb.append(" Maximum Path Length: ").append(maxPathLength).append('\n');
sb.append("]\n");
return sb.toString();
}
diff -r dde9f5cfde5f src/share/classes/java/security/cert/PKIXCertPathBuilderResult.java
--- a/src/share/classes/java/security/cert/PKIXCertPathBuilderResult.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/security/cert/PKIXCertPathBuilderResult.java Sun Aug 10 22:06:58 2014 -0300
@@ -109,11 +109,11 @@
public String toString() {
StringBuilder sb = new StringBuilder();
sb.append("PKIXCertPathBuilderResult: [\n");
- sb.append(" Certification Path: " + certPath + "\n");
- sb.append(" Trust Anchor: " + getTrustAnchor().toString() + "\n");
- sb.append(" Policy Tree: " + String.valueOf(getPolicyTree()) + "\n");
- sb.append(" Subject Public Key: " + getPublicKey() + "\n");
- sb.append("]");
+ sb.append(" Certification Path: ").append(certPath).append('\n');
+ sb.append(" Trust Anchor: ").append(getTrustAnchor().toString()).append('\n');
+ sb.append(" Policy Tree: ").append(String.valueOf(getPolicyTree())).append('\n');
+ sb.append(" Subject Public Key: ").append(getPublicKey()).append('\n');
+ sb.append(']');
return sb.toString();
}
}
diff -r dde9f5cfde5f src/share/classes/java/security/cert/PKIXCertPathValidatorResult.java
--- a/src/share/classes/java/security/cert/PKIXCertPathValidatorResult.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/security/cert/PKIXCertPathValidatorResult.java Sun Aug 10 22:06:58 2014 -0300
@@ -150,10 +150,10 @@
public String toString() {
StringBuilder sb = new StringBuilder();
sb.append("PKIXCertPathValidatorResult: [\n");
- sb.append(" Trust Anchor: " + trustAnchor.toString() + "\n");
- sb.append(" Policy Tree: " + String.valueOf(policyTree) + "\n");
- sb.append(" Subject Public Key: " + subjectPublicKey + "\n");
- sb.append("]");
+ sb.append(" Trust Anchor: ").append(trustAnchor.toString()).append('\n');
+ sb.append(" Policy Tree: ").append(String.valueOf(policyTree)).append('\n');
+ sb.append(" Subject Public Key: ").append(subjectPublicKey).append('\n');
+ sb.append(']');
return sb.toString();
}
}
diff -r dde9f5cfde5f src/share/classes/java/security/cert/PKIXParameters.java
--- a/src/share/classes/java/security/cert/PKIXParameters.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/security/cert/PKIXParameters.java Sun Aug 10 22:06:58 2014 -0300
@@ -698,8 +698,7 @@
/* start with trusted anchor info */
if (unmodTrustAnchors != null) {
- sb.append(" Trust Anchors: " + unmodTrustAnchors.toString()
- + "\n");
+ sb.append(" Trust Anchors: ").append(unmodTrustAnchors.toString()).append('\n');
}
/* now, append initial state information */
@@ -707,30 +706,28 @@
if (unmodInitialPolicies.isEmpty()) {
sb.append(" Initial Policy OIDs: any\n");
} else {
- sb.append(" Initial Policy OIDs: ["
- + unmodInitialPolicies.toString() + "]\n");
+ sb.append(" Initial Policy OIDs: [").append(unmodInitialPolicies.toString()).append("]\n");
}
}
/* now, append constraints on all certificates in the path */
- sb.append(" Validity Date: " + String.valueOf(date) + "\n");
- sb.append(" Signature Provider: " + String.valueOf(sigProvider) + "\n");
- sb.append(" Default Revocation Enabled: " + revocationEnabled + "\n");
- sb.append(" Explicit Policy Required: " + explicitPolicyRequired + "\n");
- sb.append(" Policy Mapping Inhibited: " + policyMappingInhibited + "\n");
- sb.append(" Any Policy Inhibited: " + anyPolicyInhibited + "\n");
- sb.append(" Policy Qualifiers Rejected: " + policyQualifiersRejected + "\n");
+ sb.append(" Validity Date: ").append(String.valueOf(date)).append('\n');
+ sb.append(" Signature Provider: ").append(String.valueOf(sigProvider)).append('\n');
+ sb.append(" Default Revocation Enabled: ").append(revocationEnabled).append('\n');
+ sb.append(" Explicit Policy Required: ").append(explicitPolicyRequired).append('\n');
+ sb.append(" Policy Mapping Inhibited: ").append(policyMappingInhibited).append('\n');
+ sb.append(" Any Policy Inhibited: ").append(anyPolicyInhibited).append('\n');
+ sb.append(" Policy Qualifiers Rejected: ").append(policyQualifiersRejected).append('\n');
/* now, append target cert requirements */
- sb.append(" Target Cert Constraints: " + String.valueOf(certSelector) + "\n");
+ sb.append(" Target Cert Constraints: ").append(String.valueOf(certSelector)).append('\n');
/* finally, append miscellaneous parameters */
if (certPathCheckers != null)
- sb.append(" Certification Path Checkers: ["
- + certPathCheckers.toString() + "]\n");
+ sb.append(" Certification Path Checkers: [").append(certPathCheckers.toString()).append("]\n");
if (certStores != null)
- sb.append(" CertStores: [" + certStores.toString() + "]\n");
- sb.append("]");
+ sb.append(" CertStores: [").append(certStores.toString()).append("]\n");
+ sb.append(']');
return sb.toString();
}
}
diff -r dde9f5cfde5f src/share/classes/java/security/cert/PolicyQualifierInfo.java
--- a/src/share/classes/java/security/cert/PolicyQualifierInfo.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/security/cert/PolicyQualifierInfo.java Sun Aug 10 22:06:58 2014 -0300
@@ -163,10 +163,9 @@
HexDumpEncoder enc = new HexDumpEncoder();
StringBuilder sb = new StringBuilder();
sb.append("PolicyQualifierInfo: [\n");
- sb.append(" qualifierID: " + mId + "\n");
- sb.append(" qualifier: " +
- (mData == null ? "null" : enc.encodeBuffer(mData)) + "\n");
- sb.append("]");
+ sb.append(" qualifierID: ").append(mId).append('\n');
+ sb.append(" qualifier: ").append(mData == null ? "null" : enc.encodeBuffer(mData)).append('\n');
+ sb.append(']');
pqiString = sb.toString();
return pqiString;
}
diff -r dde9f5cfde5f src/share/classes/java/security/cert/TrustAnchor.java
--- a/src/share/classes/java/security/cert/TrustAnchor.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/security/cert/TrustAnchor.java Sun Aug 10 22:06:58 2014 -0300
@@ -320,14 +320,13 @@
StringBuilder sb = new StringBuilder();
sb.append("[\n");
if (pubKey != null) {
- sb.append(" Trusted CA Public Key: " + pubKey.toString() + "\n");
- sb.append(" Trusted CA Issuer Name: "
- + String.valueOf(caName) + "\n");
+ sb.append(" Trusted CA Public Key: ").append(pubKey.toString()).append('\n');
+ sb.append(" Trusted CA Issuer Name: ").append(String.valueOf(caName)).append('\n');
} else {
- sb.append(" Trusted CA cert: " + trustedCert.toString() + "\n");
+ sb.append(" Trusted CA cert: ").append(trustedCert.toString()).append('\n');
}
if (nc != null)
- sb.append(" Name Constraints: " + nc.toString() + "\n");
+ sb.append(" Name Constraints: ").append(nc.toString()).append('\n');
return sb.toString();
}
}
diff -r dde9f5cfde5f src/share/classes/java/security/cert/X509CRLSelector.java
--- a/src/share/classes/java/security/cert/X509CRLSelector.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/security/cert/X509CRLSelector.java Sun Aug 10 22:06:58 2014 -0300
@@ -572,17 +572,17 @@
sb.append(" IssuerNames:\n");
Iterator<Object> i = issuerNames.iterator();
while (i.hasNext())
- sb.append(" " + i.next() + "\n");
+ sb.append(" ").append(i.next()).append('\n');
}
if (minCRL != null)
- sb.append(" minCRLNumber: " + minCRL + "\n");
+ sb.append(" minCRLNumber: ").append(minCRL).append('\n');
if (maxCRL != null)
- sb.append(" maxCRLNumber: " + maxCRL + "\n");
+ sb.append(" maxCRLNumber: ").append(maxCRL).append('\n');
if (dateAndTime != null)
- sb.append(" dateAndTime: " + dateAndTime + "\n");
+ sb.append(" dateAndTime: ").append(dateAndTime).append('\n');
if (certChecking != null)
- sb.append(" Certificate being checked: " + certChecking + "\n");
- sb.append("]");
+ sb.append(" Certificate being checked: ").append(certChecking).append('\n');
+ sb.append(']');
return sb.toString();
}
diff -r dde9f5cfde5f src/share/classes/java/security/cert/X509CertSelector.java
--- a/src/share/classes/java/security/cert/X509CertSelector.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/security/cert/X509CertSelector.java Sun Aug 10 22:06:58 2014 -0300
@@ -1814,72 +1814,65 @@
StringBuilder sb = new StringBuilder();
sb.append("X509CertSelector: [\n");
if (x509Cert != null) {
- sb.append(" Certificate: " + x509Cert.toString() + "\n");
+ sb.append(" Certificate: ").append(x509Cert.toString()).append('\n');
}
if (serialNumber != null) {
- sb.append(" Serial Number: " + serialNumber.toString() + "\n");
+ sb.append(" Serial Number: ").append(serialNumber.toString()).append('\n');
}
if (issuer != null) {
- sb.append(" Issuer: " + getIssuerAsString() + "\n");
+ sb.append(" Issuer: ").append(getIssuerAsString()).append('\n');
}
if (subject != null) {
- sb.append(" Subject: " + getSubjectAsString() + "\n");
+ sb.append(" Subject: ").append(getSubjectAsString()).append('\n');
}
- sb.append(" matchAllSubjectAltNames flag: "
- + String.valueOf(matchAllSubjectAltNames) + "\n");
+ sb.append(" matchAllSubjectAltNames flag: ")
+ .append(String.valueOf(matchAllSubjectAltNames))
+ .append('\n');
if (subjectAlternativeNames != null) {
sb.append(" SubjectAlternativeNames:\n");
Iterator<List<?>> i = subjectAlternativeNames.iterator();
while (i.hasNext()) {
List<?> list = i.next();
- sb.append(" type " + list.get(0) +
- ", name " + list.get(1) + "\n");
+ sb.append(" type ").append(list.get(0)).append(", name ").append(list.get(1)).append('\n');
}
}
if (subjectKeyID != null) {
HexDumpEncoder enc = new HexDumpEncoder();
- sb.append(" Subject Key Identifier: " +
- enc.encodeBuffer(subjectKeyID) + "\n");
+ sb.append(" Subject Key Identifier: ").append(enc.encodeBuffer(subjectKeyID)).append('\n');
}
if (authorityKeyID != null) {
HexDumpEncoder enc = new HexDumpEncoder();
- sb.append(" Authority Key Identifier: " +
- enc.encodeBuffer(authorityKeyID) + "\n");
+ sb.append(" Authority Key Identifier: ").append(enc.encodeBuffer(authorityKeyID)).append('\n');
}
if (certificateValid != null) {
- sb.append(" Certificate Valid: " +
- certificateValid.toString() + "\n");
+ sb.append(" Certificate Valid: ").append(certificateValid.toString()).append('\n');
}
if (privateKeyValid != null) {
- sb.append(" Private Key Valid: " +
- privateKeyValid.toString() + "\n");
+ sb.append(" Private Key Valid: ").append(privateKeyValid.toString()).append('\n');
}
if (subjectPublicKeyAlgID != null) {
- sb.append(" Subject Public Key AlgID: " +
- subjectPublicKeyAlgID.toString() + "\n");
+ sb.append(" Subject Public Key AlgID: ").append(subjectPublicKeyAlgID.toString()).append('\n');
}
if (subjectPublicKey != null) {
- sb.append(" Subject Public Key: " +
- subjectPublicKey.toString() + "\n");
+ sb.append(" Subject Public Key: ").append(subjectPublicKey.toString()).append('\n');
}
if (keyUsage != null) {
- sb.append(" Key Usage: " + keyUsageToString(keyUsage) + "\n");
+ sb.append(" Key Usage: ").append(keyUsageToString(keyUsage)).append('\n');
}
if (keyPurposeSet != null) {
- sb.append(" Extended Key Usage: " +
- keyPurposeSet.toString() + "\n");
+ sb.append(" Extended Key Usage: ").append(keyPurposeSet.toString()).append('\n');
}
if (policy != null) {
- sb.append(" Policy: " + policy.toString() + "\n");
+ sb.append(" Policy: ").append(policy.toString()).append('\n');
}
if (pathToGeneralNames != null) {
sb.append(" Path to names:\n");
Iterator<GeneralNameInterface> i = pathToGeneralNames.iterator();
while (i.hasNext()) {
- sb.append(" " + i.next() + "\n");
+ sb.append(" ").append(i.next()).append('\n');
}
}
- sb.append("]");
+ sb.append(']');
return sb.toString();
}
-------------- next part --------------
diff -r dde9f5cfde5f src/share/classes/java/rmi/dgc/VMID.java
--- a/src/share/classes/java/rmi/dgc/VMID.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/rmi/dgc/VMID.java Sun Aug 10 22:06:58 2014 -0300
@@ -124,8 +124,7 @@
if (addr != null)
for (int i = 0; i < addr.length; ++ i) {
int x = addr[i] & 0xFF;
- sb.append((x < 0x10 ? "0" : "") +
- Integer.toString(x, 16));
+ sb.append(x < 0x10 ? "0" : "").append(Integer.toString(x, 16));
}
sb.append(':');
sb.append(uid.toString());
-------------- next part --------------
diff -r dde9f5cfde5f src/share/classes/java/lang/management/MemoryUsage.java
--- a/src/share/classes/java/lang/management/MemoryUsage.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/lang/management/MemoryUsage.java Sun Aug 10 22:06:58 2014 -0300
@@ -238,11 +238,10 @@
*/
public String toString() {
StringBuilder buf = new StringBuilder();
- buf.append("init = " + init + "(" + (init >> 10) + "K) ");
- buf.append("used = " + used + "(" + (used >> 10) + "K) ");
- buf.append("committed = " + committed + "(" +
- (committed >> 10) + "K) " );
- buf.append("max = " + max + "(" + (max >> 10) + "K)");
+ buf.append("init = ").append(init).append('(').append(init >> 10).append("K) ");
+ buf.append("used = ").append(used).append('(').append(used >> 10).append("K) ");
+ buf.append("committed = ").append(committed).append('(').append(committed >> 10).append("K) ");
+ buf.append("max = ").append(max).append('(').append(max >> 10).append("K)");
return buf.toString();
}
diff -r dde9f5cfde5f src/share/classes/java/lang/management/ThreadInfo.java
--- a/src/share/classes/java/lang/management/ThreadInfo.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/java/lang/management/ThreadInfo.java Sun Aug 10 22:06:58 2014 -0300
@@ -579,15 +579,17 @@
* @return a string representation of this thread info.
*/
public String toString() {
- StringBuilder sb = new StringBuilder("\"" + getThreadName() + "\"" +
- " Id=" + getThreadId() + " " +
- getThreadState());
+ StringBuilder sb = new StringBuilder();
+ sb.append('\"').append(getThreadName()).append('\"')
+ .append(" Id=").append(getThreadId()).append(' ')
+ .append(getThreadState());
+
if (getLockName() != null) {
- sb.append(" on " + getLockName());
+ sb.append(" on ").append(getLockName());
}
if (getLockOwnerName() != null) {
- sb.append(" owned by \"" + getLockOwnerName() +
- "\" Id=" + getLockOwnerId());
+ sb.append(" owned by \"").append(getLockOwnerName())
+ .append("\" Id=").append(getLockOwnerId());
}
if (isSuspended()) {
sb.append(" (suspended)");
@@ -599,21 +601,21 @@
int i = 0;
for (; i < stackTrace.length && i < MAX_FRAMES; i++) {
StackTraceElement ste = stackTrace[i];
- sb.append("\tat " + ste.toString());
+ sb.append("\tat ").append(ste.toString());
sb.append('\n');
if (i == 0 && getLockInfo() != null) {
Thread.State ts = getThreadState();
switch (ts) {
case BLOCKED:
- sb.append("\t- blocked on " + getLockInfo());
+ sb.append("\t- blocked on ").append(getLockInfo());
sb.append('\n');
break;
case WAITING:
- sb.append("\t- waiting on " + getLockInfo());
+ sb.append("\t- waiting on ").append(getLockInfo());
sb.append('\n');
break;
case TIMED_WAITING:
- sb.append("\t- waiting on " + getLockInfo());
+ sb.append("\t- waiting on ").append(getLockInfo());
sb.append('\n');
break;
default:
@@ -622,7 +624,7 @@
for (MonitorInfo mi : lockedMonitors) {
if (mi.getLockedStackDepth() == i) {
- sb.append("\t- locked " + mi);
+ sb.append("\t- locked ").append(mi);
sb.append('\n');
}
}
@@ -634,10 +636,10 @@
LockInfo[] locks = getLockedSynchronizers();
if (locks.length > 0) {
- sb.append("\n\tNumber of locked synchronizers = " + locks.length);
+ sb.append("\n\tNumber of locked synchronizers = ").append(locks.length);
sb.append('\n');
for (LockInfo li : locks) {
- sb.append("\t- " + li);
+ sb.append("\t- ").append(li);
sb.append('\n');
}
}
-------------- next part --------------
diff -r dde9f5cfde5f src/share/classes/com/sun/crypto/provider/DHParameters.java
--- a/src/share/classes/com/sun/crypto/provider/DHParameters.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/crypto/provider/DHParameters.java Sun Aug 10 22:06:58 2014 -0300
@@ -138,7 +138,7 @@
+ LINE_SEP + "g:" + LINE_SEP
+ Debug.toHexString(this.g));
if (this.l != 0)
- sb.append(LINE_SEP + "l:" + LINE_SEP + " " + this.l);
+ sb.append(LINE_SEP).append("l:").append(LINE_SEP).append(" ").append(this.l);
return sb.toString();
}
}
diff -r dde9f5cfde5f src/share/classes/com/sun/crypto/provider/DHPublicKey.java
--- a/src/share/classes/com/sun/crypto/provider/DHPublicKey.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/crypto/provider/DHPublicKey.java Sun Aug 10 22:06:58 2014 -0300
@@ -269,7 +269,7 @@
+ LINE_SEP + "g:" + LINE_SEP
+ Debug.toHexString(this.g));
if (this.l != 0)
- sb.append(LINE_SEP + "l:" + LINE_SEP + " " + this.l);
+ sb.append(LINE_SEP).append("l:").append(LINE_SEP).append(" ").append(this.l);
return sb.toString();
}
diff -r dde9f5cfde5f src/share/classes/com/sun/crypto/provider/GCMParameters.java
--- a/src/share/classes/com/sun/crypto/provider/GCMParameters.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/crypto/provider/GCMParameters.java Sun Aug 10 22:06:58 2014 -0300
@@ -140,7 +140,7 @@
= new StringBuilder(LINE_SEP + " iv:" + LINE_SEP + "["
+ encoder.encodeBuffer(iv) + "]");
- sb.append(LINE_SEP + "tLen(bits):" + LINE_SEP + tLen*8 + LINE_SEP);
+ sb.append(LINE_SEP).append("tLen(bits):").append(LINE_SEP).append(tLen * 8).append(LINE_SEP);
return sb.toString();
}
}
diff -r dde9f5cfde5f src/share/classes/com/sun/crypto/provider/OAEPParameters.java
--- a/src/share/classes/com/sun/crypto/provider/OAEPParameters.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/crypto/provider/OAEPParameters.java Sun Aug 10 22:06:58 2014 -0300
@@ -239,10 +239,9 @@
protected String engineToString() {
StringBuilder sb = new StringBuilder();
- sb.append("MD: " + mdName + "\n");
- sb.append("MGF: MGF1" + mgfSpec.getDigestAlgorithm() + "\n");
- sb.append("PSource: PSpecified " +
- (p.length==0? "":Debug.toHexString(new BigInteger(p))) + "\n");
+ sb.append("MD: ").append(mdName).append('\n');
+ sb.append("MGF: MGF1").append(mgfSpec.getDigestAlgorithm()).append('\n');
+ sb.append("PSource: PSpecified ").append(p.length == 0 ? "" : Debug.toHexString(new BigInteger(p))).append('\n');
return sb.toString();
}
}
diff -r dde9f5cfde5f src/share/classes/com/sun/crypto/provider/RC2Parameters.java
--- a/src/share/classes/com/sun/crypto/provider/RC2Parameters.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/crypto/provider/RC2Parameters.java Sun Aug 10 22:06:58 2014 -0300
@@ -221,8 +221,9 @@
+ encoder.encodeBuffer(iv) + "]");
if (version != 0) {
- sb.append(LINE_SEP + "version:" + LINE_SEP
- + version + LINE_SEP);
+ sb.append(LINE_SEP).append("version:")
+ .append(LINE_SEP).append(version)
+ .append(LINE_SEP);
}
return sb.toString();
}
diff -r dde9f5cfde5f src/share/classes/com/sun/jmx/snmp/IPAcl/NetMaskImpl.java
--- a/src/share/classes/com/sun/jmx/snmp/IPAcl/NetMaskImpl.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/jmx/snmp/IPAcl/NetMaskImpl.java Sun Aug 10 22:06:58 2014 -0300
@@ -65,7 +65,7 @@
"extractSubNet", "BINARY ARRAY :");
StringBuilder sb = new StringBuilder();
for(int i =0; i < addrLength; i++) {
- sb.append((b[i] & 0xFF) + ":");
+ sb.append(b[i] & 0xFF).append(':');
}
SNMP_LOGGER.logp(Level.FINEST, NetMaskImpl.class.getName(),
"extractSubNet", sb.toString());
diff -r dde9f5cfde5f src/share/classes/com/sun/jmx/snmp/IPAcl/ParseException.java
--- a/src/share/classes/com/sun/jmx/snmp/IPAcl/ParseException.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/jmx/snmp/IPAcl/ParseException.java Sun Aug 10 22:06:58 2014 -0300
@@ -204,7 +204,7 @@
default:
if ((ch = str.charAt(i)) < 0x20 || ch > 0x7e) {
String s = "0000" + Integer.toString(ch, 16);
- retval.append("\\u" + s.substring(s.length() - 4, s.length()));
+ retval.append("\\u").append(s.substring(s.length() - 4, s.length()));
} else {
retval.append(ch);
}
diff -r dde9f5cfde5f src/share/classes/com/sun/jmx/snmp/IPAcl/Parser.java
--- a/src/share/classes/com/sun/jmx/snmp/IPAcl/Parser.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/jmx/snmp/IPAcl/Parser.java Sun Aug 10 22:06:58 2014 -0300
@@ -379,7 +379,7 @@
}
jj_consume_token(DOT);
t = jj_consume_token(IDENTIFIER);
- jjtn000.name.append( "." + t.image);
+ jjtn000.name.append('.').append(t.image);
}
} finally {
if (jjtc000) {
@@ -408,7 +408,7 @@
}
jj_consume_token(DOT);
t = jj_consume_token(INTEGER_LITERAL);
- jjtn000.address.append( "." + t.image);
+ jjtn000.address.append('.').append(t.image);
}
} finally {
if (jjtc000) {
@@ -454,7 +454,7 @@
}
jj_consume_token(MARK);
t = jj_consume_token(INTEGER_LITERAL);
- jjtn000.address.append( "." + t.image);
+ jjtn000.address.append('.').append(t.image);
}
} finally {
if (jjtc000) {
@@ -483,7 +483,7 @@
}
jj_consume_token(DOT);
t = jj_consume_token(INTEGER_LITERAL);
- jjtn000.address.append( "." + t.image);
+ jjtn000.address.append('.').append(t.image);
}
jj_consume_token(MASK);
t = jj_consume_token(INTEGER_LITERAL);
diff -r dde9f5cfde5f src/share/classes/com/sun/jmx/snmp/IPAcl/TokenMgrError.java
--- a/src/share/classes/com/sun/jmx/snmp/IPAcl/TokenMgrError.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/jmx/snmp/IPAcl/TokenMgrError.java Sun Aug 10 22:06:58 2014 -0300
@@ -99,7 +99,7 @@
default:
if ((ch = str.charAt(i)) < 0x20 || ch > 0x7e) {
String s = "0000" + Integer.toString(ch, 16);
- retval.append("\\u" + s.substring(s.length() - 4, s.length()));
+ retval.append("\\u").append(s.substring(s.length() - 4, s.length()));
} else {
retval.append(ch);
}
diff -r dde9f5cfde5f src/share/classes/com/sun/jmx/snmp/SnmpV3Message.java
--- a/src/share/classes/com/sun/jmx/snmp/SnmpV3Message.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/jmx/snmp/SnmpV3Message.java Sun Aug 10 22:06:58 2014 -0300
@@ -479,10 +479,10 @@
*/
public String printMessage() {
StringBuilder sb = new StringBuilder();
- sb.append("msgId : " + msgId + "\n");
- sb.append("msgMaxSize : " + msgMaxSize + "\n");
- sb.append("msgFlags : " + msgFlags + "\n");
- sb.append("msgSecurityModel : " + msgSecurityModel + "\n");
+ sb.append("msgId : ").append(msgId).append('\n');
+ sb.append("msgMaxSize : ").append(msgMaxSize).append('\n');
+ sb.append("msgFlags : ").append(msgFlags).append('\n');
+ sb.append("msgSecurityModel : ").append(msgSecurityModel).append('\n');
if (contextEngineId == null) {
sb.append("contextEngineId : null");
diff -r dde9f5cfde5f src/share/classes/com/sun/jndi/cosnaming/CNNameParser.java
--- a/src/share/classes/com/sun/jndi/cosnaming/CNNameParser.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/jndi/cosnaming/CNNameParser.java Sun Aug 10 22:06:58 2014 -0300
@@ -256,7 +256,7 @@
private static String stringifyComponent(NameComponent comp) {
StringBuilder one = new StringBuilder(escape(comp.id));
if (comp.kind != null && !comp.kind.equals("")) {
- one.append(kindSeparator + escape(comp.kind));
+ one.append(kindSeparator).append(escape(comp.kind));
}
if (one.length() == 0) {
return ""+kindSeparator; // if neither id nor kind specified
diff -r dde9f5cfde5f src/share/classes/com/sun/jndi/ldap/sasl/DefaultCallbackHandler.java
--- a/src/share/classes/com/sun/jndi/ldap/sasl/DefaultCallbackHandler.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/jndi/ldap/sasl/DefaultCallbackHandler.java Sun Aug 10 22:06:58 2014 -0300
@@ -90,7 +90,7 @@
if (selected == -1) {
StringBuilder allChoices = new StringBuilder();
for (int j = 0; j < choices.length; j++) {
- allChoices.append(choices[j] + ",");
+ allChoices.append(choices[j]).append(',');
}
throw new IOException("Cannot match " +
"'java.naming.security.sasl.realm' property value, '" +
diff -r dde9f5cfde5f src/share/classes/com/sun/media/sound/WaveExtensibleFileReader.java
--- a/src/share/classes/com/sun/media/sound/WaveExtensibleFileReader.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/media/sound/WaveExtensibleFileReader.java Sun Aug 10 22:06:58 2014 -0300
@@ -167,9 +167,9 @@
for (int i = 0; i < allchannelnames.length; i++) {
if ((channelmask & m) != 0L) {
if (i < channelnames.length) {
- sb.append(channelnames[i] + " ");
+ sb.append(channelnames[i]).append(' ');
} else {
- sb.append(allchannelnames[i] + " ");
+ sb.append(allchannelnames[i]).append(' ');
}
}
m *= 2L;
diff -r dde9f5cfde5f src/share/classes/com/sun/org/apache/xml/internal/security/encryption/AbstractSerializer.java
--- a/src/share/classes/com/sun/org/apache/xml/internal/security/encryption/AbstractSerializer.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/org/apache/xml/internal/security/encryption/AbstractSerializer.java Sun Aug 10 22:06:58 2014 -0300
@@ -235,14 +235,14 @@
String nodeName = att.getNodeName();
if ((nodeName.equals("xmlns") || nodeName.startsWith("xmlns:"))
&& !storedNamespaces.containsKey(att.getNodeName())) {
- sb.append(" " + nodeName + "=\"" + att.getNodeValue() + "\"");
+ sb.append(' ').append(nodeName).append("=\"").append(att.getNodeValue()).append('\"');
storedNamespaces.put(nodeName, att.getNodeValue());
}
}
}
wk = wk.getParentNode();
}
- sb.append(">" + source + "</dummy>");
+ sb.append('>').append(source).append("</dummy>");
return sb.toString();
}
diff -r dde9f5cfde5f src/share/classes/com/sun/org/apache/xml/internal/security/transforms/params/InclusiveNamespaces.java
--- a/src/share/classes/com/sun/org/apache/xml/internal/security/transforms/params/InclusiveNamespaces.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/org/apache/xml/internal/security/transforms/params/InclusiveNamespaces.java Sun Aug 10 22:06:58 2014 -0300
@@ -85,7 +85,7 @@
if (prefix.equals("xmlns")) {
sb.append("#default ");
} else {
- sb.append(prefix + " ");
+ sb.append(prefix).append(' ');
}
}
diff -r dde9f5cfde5f src/share/classes/com/sun/org/apache/xml/internal/security/utils/RFC2253Parser.java
--- a/src/share/classes/com/sun/org/apache/xml/internal/security/utils/RFC2253Parser.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/org/apache/xml/internal/security/utils/RFC2253Parser.java Sun Aug 10 22:06:58 2014 -0300
@@ -88,7 +88,7 @@
l += countQuotes(DN, j, k);
if ((k > 0) && (DN.charAt(k - 1) != '\\') && (l % 2) == 0) {
- sb.append(parseRDN(DN.substring(i, k).trim(), toXml) + ",");
+ sb.append(parseRDN(DN.substring(i, k).trim(), toXml)).append(',');
i = k + 1;
l = 0;
@@ -121,7 +121,7 @@
l += countQuotes(str, j, k);
if ((k > 0) && (str.charAt(k - 1) != '\\') && (l % 2) == 0) {
- sb.append(parseATAV(trim(str.substring(i, k)), toXml) + "+");
+ sb.append(parseATAV(trim(str.substring(i, k)), toXml)).append('+');
i = k + 1;
l = 0;
@@ -369,7 +369,7 @@
int k;
for (int j = 0; (k = string.indexOf("\\20", j)) >= 0; j = k + 3) {
- sb.append(trim(string.substring(i, k)) + "\\ ");
+ sb.append(trim(string.substring(i, k))).append("\\ ");
i = k + 3;
}
@@ -418,7 +418,7 @@
l += countQuotes(str, j, k);
if ((k > 0) && (str.charAt(k - 1) != '\\') && (l % 2) == 0) {
- sb.append(trim(str.substring(i, k)) + replace);
+ sb.append(trim(str.substring(i, k))).append(replace);
i = k + 1;
l = 0;
diff -r dde9f5cfde5f src/share/classes/com/sun/security/sasl/CramMD5Base.java
--- a/src/share/classes/com/sun/security/sasl/CramMD5Base.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/security/sasl/CramMD5Base.java Sun Aug 10 22:06:58 2014 -0300
@@ -196,8 +196,7 @@
for (i = 0; i < digest.length; i++) {
if ((digest[i] & 0x000000ff) < 0x10) {
- digestString.append("0" +
- Integer.toHexString(digest[i] & 0x000000ff));
+ digestString.append('0').append(Integer.toHexString(digest[i] & 0x000000ff));
} else {
digestString.append(
Integer.toHexString(digest[i] & 0x000000ff));
diff -r dde9f5cfde5f src/share/classes/com/sun/security/sasl/digest/DigestMD5Base.java
--- a/src/share/classes/com/sun/security/sasl/digest/DigestMD5Base.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/security/sasl/digest/DigestMD5Base.java Sun Aug 10 22:06:58 2014 -0300
@@ -391,8 +391,7 @@
for (int i = 0; i < digest.length; i ++) {
if ((digest[i] & 0x000000ff) < 0x10) {
- digestString.append("0"+
- Integer.toHexString(digest[i] & 0x000000ff));
+ digestString.append('0').append(Integer.toHexString(digest[i] & 0x000000ff));
} else {
digestString.append(
Integer.toHexString(digest[i] & 0x000000ff));
diff -r dde9f5cfde5f src/share/classes/com/sun/tools/example/debug/expr/ParseException.java
--- a/src/share/classes/com/sun/tools/example/debug/expr/ParseException.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/tools/example/debug/expr/ParseException.java Sun Aug 10 22:06:58 2014 -0300
@@ -198,7 +198,7 @@
default:
if ((ch = str.charAt(i)) < 0x20 || ch > 0x7e) {
String s = "0000" + Integer.toString(ch, 16);
- retval.append("\\u" + s.substring(s.length() - 4, s.length()));
+ retval.append("\\u").append(s.substring(s.length() - 4, s.length()));
} else {
retval.append(ch);
}
diff -r dde9f5cfde5f src/share/classes/com/sun/tools/example/debug/expr/TokenMgrError.java
--- a/src/share/classes/com/sun/tools/example/debug/expr/TokenMgrError.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/tools/example/debug/expr/TokenMgrError.java Sun Aug 10 22:06:58 2014 -0300
@@ -107,7 +107,7 @@
default:
if ((ch = str.charAt(i)) < 0x20 || ch > 0x7e) {
String s = "0000" + Integer.toString(ch, 16);
- retval.append("\\u" + s.substring(s.length() - 4, s.length()));
+ retval.append("\\u").append(s.substring(s.length() - 4, s.length()));
} else {
retval.append(ch);
}
diff -r dde9f5cfde5f src/share/classes/com/sun/tools/example/debug/gui/ContextManager.java
--- a/src/share/classes/com/sun/tools/example/debug/gui/ContextManager.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/tools/example/debug/gui/ContextManager.java Sun Aug 10 22:06:58 2014 -0300
@@ -338,10 +338,10 @@
if (classpath.isEmpty()) {
String envcp = System.getProperty("env.class.path");
if ((envcp != null) && (envcp.length() > 0)) {
- munged.append(" -classpath " + envcp);
+ munged.append(" -classpath ").append(envcp);
}
} else {
- munged.append(" -classpath " + classpath.asString());
+ munged.append(" -classpath ").append(classpath.asString());
}
return munged.toString();
} else {
diff -r dde9f5cfde5f src/share/classes/com/sun/tools/hat/internal/model/JavaValueArray.java
--- a/src/share/classes/com/sun/tools/hat/internal/model/JavaValueArray.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/tools/hat/internal/model/JavaValueArray.java Sun Aug 10 22:06:58 2014 -0300
@@ -386,37 +386,37 @@
}
case 'B': {
int val = 0xFF & byteAt(i, value);
- result.append("0x" + Integer.toString(val, 16));
+ result.append("0x").append(Integer.toString(val, 16));
i++;
break;
}
case 'S': {
short val = shortAt(i, value);
i += 2;
- result.append("" + val);
+ result.append(val);
break;
}
case 'I': {
int val = intAt(i, value);
i += 4;
- result.append("" + val);
+ result.append(val);
break;
}
case 'J': { // long
long val = longAt(i, value);
- result.append("" + val);
+ result.append(val);
i += 8;
break;
}
case 'F': {
float val = floatAt(i, value);
- result.append("" + val);
+ result.append(val);
i += 4;
break;
}
case 'D': { // double
double val = doubleAt(i, value);
- result.append("" + val);
+ result.append(val);
i += 8;
break;
}
@@ -425,7 +425,7 @@
}
}
}
- result.append("}");
+ result.append('}');
}
return result.toString();
}
diff -r dde9f5cfde5f src/share/classes/com/sun/tools/hat/internal/util/Misc.java
--- a/src/share/classes/com/sun/tools/hat/internal/util/Misc.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/tools/hat/internal/util/Misc.java Sun Aug 10 22:06:58 2014 -0300
@@ -97,11 +97,11 @@
} else if (ch == '&') {
sb.append("&");
} else if (ch < ' ') {
- sb.append("&#" + Integer.toString(ch) + ";");
+ sb.append("&#").append(Integer.toString(ch)).append(';');
} else {
int c = (ch & 0xFFFF);
if (c > 127) {
- sb.append("&#" + Integer.toString(c) + ";");
+ sb.append("&#").append(Integer.toString(c)).append(';');
} else {
sb.append(ch);
}
diff -r dde9f5cfde5f src/share/classes/com/sun/tools/jdi/VirtualMachineImpl.java
--- a/src/share/classes/com/sun/tools/jdi/VirtualMachineImpl.java Tue Aug 05 19:29:00 2014 -0700
+++ b/src/share/classes/com/sun/tools/jdi/VirtualMachineImpl.java Sun Aug 10 22:06:58 2014 -0300
@@ -881,12 +881,12 @@
} else if (tag == JDWP.TypeTag.ARRAY) {
sb.append("ArrayType");
} else {
- sb.append("UNKNOWN TAG: " + tag);
+ sb.append("UNKNOWN TAG: ").append(tag);
}
if (signature != null) {
- sb.append(", signature='" + signature + "'");
+ sb.append(", signature='").append(signature).append('\'');
}
- sb.append(", id=" + id);
+ sb.append(", id=").append(id);
vm.printTrace(sb.toString());
}
if (id == 0) {
More information about the swing-dev
mailing list