8067207: Replace concat String to append in StringBuilder parameters (client)

Reviewed-by: serb
This commit is contained in:
Otavio Santana 2014-12-11 15:19:12 +08:00 committed by Weijun Wang
parent eaff0bd40d
commit 6e791e3f31
8 changed files with 38 additions and 39 deletions

View File

@ -167,9 +167,9 @@ public final class WaveExtensibleFileReader extends AudioFileReader {
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;

View File

@ -1213,15 +1213,15 @@ public class GroupLayout implements LayoutManager2 {
registerComponents(horizontalGroup, HORIZONTAL);
registerComponents(verticalGroup, VERTICAL);
}
StringBuffer buffer = new StringBuffer();
buffer.append("HORIZONTAL\n");
createSpringDescription(buffer, horizontalGroup, " ", HORIZONTAL);
buffer.append("\nVERTICAL\n");
createSpringDescription(buffer, verticalGroup, " ", VERTICAL);
return buffer.toString();
StringBuilder sb = new StringBuilder();
sb.append("HORIZONTAL\n");
createSpringDescription(sb, horizontalGroup, " ", HORIZONTAL);
sb.append("\nVERTICAL\n");
createSpringDescription(sb, verticalGroup, " ", VERTICAL);
return sb.toString();
}
private void createSpringDescription(StringBuffer buffer, Spring spring,
private void createSpringDescription(StringBuilder sb, Spring spring,
String indent, int axis) {
String origin = "";
String padding = "";
@ -1239,20 +1239,19 @@ public class GroupLayout implements LayoutManager2 {
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");
sb.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 += " ";
for (int counter = 0; counter < springs.size(); counter++) {
createSpringDescription(buffer, springs.get(counter), indent,
createSpringDescription(sb, springs.get(counter), indent,
axis);
}
}

View File

@ -543,17 +543,17 @@ public class JColorChooser extends JComponent implements Accessible {
* @return a string representation of this <code>JColorChooser</code>
*/
protected String paramString() {
StringBuilder chooserPanelsString = new StringBuilder("");
for (int i=0; i<chooserPanels.length; i++) {
chooserPanelsString.append("[" + chooserPanels[i].toString()
+ "]");
StringBuilder chooserPanelsString = new StringBuilder();
for (AbstractColorChooserPanel panel : chooserPanels) {
chooserPanelsString.append('[').append(panel)
.append(']');
}
String previewPanelString = (previewPanel != null ?
previewPanel.toString() : "");
String previewPanelString = (previewPanel != null ? previewPanel
.toString() : "");
return super.paramString() +
",chooserPanels=" + chooserPanelsString.toString() +
",previewPanel=" + previewPanelString;
return super.paramString() + ",chooserPanels="
+ chooserPanelsString.toString() + ",previewPanel="
+ previewPanelString;
}
/////////////////

View File

@ -1330,8 +1330,8 @@ public class BasicFileChooserUI extends FileChooserUI {
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

View File

@ -2948,8 +2948,8 @@ public class BasicListUI extends ListUI
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

View File

@ -2230,11 +2230,11 @@ public class BasicTableUI extends TableUI
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('\n');
htmlStr.append("</tr>\n");
}

View File

@ -3618,8 +3618,8 @@ public class BasicTreeUI extends TreeUI
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

View File

@ -1894,9 +1894,9 @@ public class StandardGlyphVector extends GlyphVector {
}
}
catch(Exception e) {
buf.append(" " + e.getMessage());
buf.append(' ').append(e.getMessage());
}
buf.append("}");
buf.append('}');
return buf;
}