Skip to content

Commit 584bdbc

Browse files
author
Vincent Potucek
committed
fix Reports loops that can be replaced with a single String.repeat() method (available since Java 11).
1 parent 8e73be5 commit 584bdbc

File tree

3 files changed

+4
-12
lines changed

3 files changed

+4
-12
lines changed

lib/src/main/java/com/diffplug/spotless/generic/IndentStep.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -102,14 +102,10 @@ String format(String raw) {
102102
if (numSpaces > 0) {
103103
switch (state.type) {
104104
case SPACE:
105-
for (int i = 0; i < numSpaces; ++i) {
106-
builder.append(' ');
107-
}
105+
builder.append(" ".repeat(numSpaces));
108106
break;
109107
case TAB:
110-
for (int i = 0; i < numSpaces / state.numSpacesPerTab; ++i) {
111-
builder.append('\t');
112-
}
108+
builder.append("\t".repeat(Math.max(0, numSpaces / state.numSpacesPerTab)));
113109
if (mightBeMultiLineComment && (numSpaces % state.numSpacesPerTab == 1)) {
114110
builder.append(' ');
115111
}

lib/src/main/java/com/diffplug/spotless/sql/dbeaver/DBeaverSQLFormatterConfiguration.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -67,9 +67,7 @@ public DBeaverSQLFormatterConfiguration(Properties properties) {
6767
private String getIndentString(String indentType, int indentSize) {
6868
char indentChar = indentType.equals("space") ? ' ' : '\t';
6969
StringBuilder stringBuilder = new StringBuilder();
70-
for (int i = 0; i < indentSize; i++) {
71-
stringBuilder.append(indentChar);
72-
}
70+
stringBuilder.append(String.valueOf(indentChar).repeat(Math.max(0, indentSize)));
7371
return stringBuilder.toString();
7472
}
7573

lib/src/main/java/com/diffplug/spotless/sql/dbeaver/SQLTokenizedFormatter.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -398,9 +398,7 @@ private int insertReturnAndIndent(final List<FormatterToken> argList, final int
398398
try {
399399
final String defaultLineSeparator = getDefaultLineSeparator();
400400
StringBuilder s = new StringBuilder(defaultLineSeparator);
401-
for (int index = 0; index < argIndent; index++) {
402-
s.append(formatterCfg.getIndentString());
403-
}
401+
s.append(String.valueOf(formatterCfg.getIndentString()).repeat(Math.max(0, argIndent)));
404402
if (argIndex > 0) {
405403
final FormatterToken token = argList.get(argIndex);
406404
final FormatterToken prevToken = argList.get(argIndex - 1);

0 commit comments

Comments
 (0)