Skip to content

Commit cd8af55

Browse files
committed
Apply checkstyle
1 parent 966c452 commit cd8af55

13 files changed

Lines changed: 20 additions & 22 deletions

File tree

handlebars-guava-cache/src/main/java/com/github/jknack/handlebars/io/GuavaCachedTemplateLoader.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,7 @@ public void setSuffix(final String suffix) {
126126
delegate.setSuffix(suffix);
127127
}
128128

129-
@Override public void setCharset(Charset charset) {
129+
@Override public void setCharset(final Charset charset) {
130130
delegate.setCharset(charset);
131131
}
132132

handlebars/src/main/java/com/github/jknack/handlebars/helper/DefaultHelperRegistry.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -262,7 +262,7 @@ public DefaultHelperRegistry with(final HandlebarsJs handlebarsJs) {
262262
return this;
263263
}
264264

265-
@Override public DefaultHelperRegistry setCharset(Charset charset) {
265+
@Override public DefaultHelperRegistry setCharset(final Charset charset) {
266266
this.charset = notNull(charset, "Charset required.");
267267
return this;
268268
}

handlebars/src/main/java/com/github/jknack/handlebars/internal/Block.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -316,9 +316,10 @@ public String text() {
316316
* Build a text version of this block.
317317
*
318318
* @param complete True if the inner block should be added.
319+
* @param close If we must close the block.
319320
* @return A string version of this block.
320321
*/
321-
private String text(final boolean complete, boolean close) {
322+
private String text(final boolean complete, final boolean close) {
322323
StringBuilder buffer = new StringBuilder();
323324
buffer.append(startDelimiter).append(type).append(name);
324325
String params = paramsToString(this.params);

handlebars/src/main/java/com/github/jknack/handlebars/internal/HbsParserFactory.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,8 @@ public Template parse(final TemplateSource source) throws IOException {
7171
final ANTLRErrorListener errorReporter = new HbsErrorReporter(source.filename());
7272

7373
// 1. Lexer
74-
final HbsLexer lexer = newLexer(newStream(source.filename(), source.content(handlebars.getCharset())),
75-
startDelimiter, endDelimiter);
74+
final HbsLexer lexer = newLexer(newStream(source.filename(),
75+
source.content(handlebars.getCharset())), startDelimiter, endDelimiter);
7676
configure(lexer, errorReporter);
7777

7878
// 2. Parser

handlebars/src/main/java/com/github/jknack/handlebars/internal/Partial.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,6 @@
2424
import java.io.IOException;
2525
import java.io.Writer;
2626
import java.nio.charset.Charset;
27-
import java.nio.charset.StandardCharsets;
28-
import java.util.Arrays;
2927
import java.util.Collections;
3028
import java.util.HashMap;
3129
import java.util.LinkedList;
@@ -240,7 +238,7 @@ public String filename() {
240238
}
241239

242240
@Override
243-
public String content(Charset charset) throws IOException {
241+
public String content(final Charset charset) throws IOException {
244242
return partialInput(source.content(charset), indent);
245243
}
246244

handlebars/src/main/java/com/github/jknack/handlebars/internal/js/RhinoHandlebars.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -334,6 +334,9 @@ private org.mozilla.javascript.Context newContext() {
334334
return ctx;
335335
}
336336

337+
/**
338+
* @return Context factory.
339+
*/
337340
private static ContextFactory newFactory() {
338341
return new ContextFactory() {
339342
@Override

handlebars/src/main/java/com/github/jknack/handlebars/io/AbstractTemplateLoader.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -103,8 +103,8 @@ public void setSuffix(final String suffix) {
103103
}
104104

105105
@Override
106-
public void setCharset(Charset charset) {
107-
this.charset = charset;
106+
public void setCharset(final Charset charset) {
107+
this.charset = notNull(charset, "Charset required.");
108108
}
109109

110110
@Override

handlebars/src/main/java/com/github/jknack/handlebars/io/AbstractTemplateSource.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,6 @@
1919

2020
import org.apache.commons.lang3.builder.HashCodeBuilder;
2121

22-
import java.io.IOException;
23-
import java.nio.charset.StandardCharsets;
24-
2522
/**
2623
* Base class for {@link TemplateSource} with default implementation of {@link #equals(Object)} and
2724
* {@link #hashCode()}.

handlebars/src/main/java/com/github/jknack/handlebars/io/CompositeTemplateLoader.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ public void setSuffix(final String suffix) {
115115
}
116116

117117
@Override
118-
public void setCharset(Charset charset) {
118+
public void setCharset(final Charset charset) {
119119
for (TemplateLoader delegate : delegates) {
120120
delegate.setCharset(charset);
121121
}

handlebars/src/main/java/com/github/jknack/handlebars/io/ForwardingTemplateSource.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ public ForwardingTemplateSource(final TemplateSource source) {
4444
this.source = notNull(source, "The source is required.");
4545
}
4646

47-
@Override public String content(Charset charset) throws IOException {
47+
@Override public String content(final Charset charset) throws IOException {
4848
return source.content(charset);
4949
}
5050

0 commit comments

Comments
 (0)