Skip to content

Commit ac16086

Browse files
committed
[Java] Clean up after merge of PR #603.
1 parent a0d777a commit ac16086

File tree

1 file changed

+17
-16
lines changed

1 file changed

+17
-16
lines changed

sbe-tool/src/main/java/uk/co/real_logic/sbe/generation/java/JavaGenerator.java

Lines changed: 17 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -2269,7 +2269,7 @@ private CharSequence generateFlyweightCode(
22692269
" private %11$s buffer;\n" +
22702270
" protected int offset;\n" +
22712271
" protected int limit;\n" +
2272-
"%13$s" +
2272+
"%13$s" +
22732273
"\n" +
22742274
" public %1$s sbeBlockLength()\n" +
22752275
" {\n" +
@@ -2340,48 +2340,49 @@ private CharSequence generateEncoderFlyweightCode(final String className, final
23402340
" }\n\n",
23412341
className,
23422342
mutableBuffer);
2343-
final String indent = " ";
2344-
String wrapAndApplyHeaderMethodFmtString =
2343+
2344+
final StringBuilder builder = new StringBuilder(
23452345
" public %1$s wrapAndApplyHeader(\n" +
23462346
" final %2$s buffer, final int offset, final %3$s headerEncoder)\n" +
23472347
" {\n" +
23482348
" headerEncoder\n" +
2349-
" .wrap(buffer, offset)";
2349+
" .wrap(buffer, offset)");
2350+
23502351
for (final Token headerToken : ir.headerStructure().tokens())
23512352
{
23522353
if (!headerToken.isConstantEncoding())
23532354
{
2354-
final String fieldName = headerToken.name();
2355-
switch (fieldName)
2355+
switch (headerToken.name())
23562356
{
23572357
case "blockLength":
2358-
wrapAndApplyHeaderMethodFmtString += '\n' + indent + ".blockLength(BLOCK_LENGTH)";
2358+
builder.append("\n .blockLength(BLOCK_LENGTH)");
23592359
break;
2360+
23602361
case "templateId":
2361-
wrapAndApplyHeaderMethodFmtString += '\n' + indent + ".templateId(TEMPLATE_ID)";
2362+
builder.append("\n .templateId(TEMPLATE_ID)");
23622363
break;
2364+
23632365
case "schemaId":
2364-
wrapAndApplyHeaderMethodFmtString += '\n' + indent + ".schemaId(SCHEMA_ID)";
2366+
builder.append("\n .schemaId(SCHEMA_ID)");
23652367
break;
2368+
23662369
case "version":
2367-
wrapAndApplyHeaderMethodFmtString += '\n' + indent + ".version(SCHEMA_VERSION)";
2370+
builder.append("\n .version(SCHEMA_VERSION)");
23682371
break;
23692372
}
23702373
}
23712374
}
23722375

2376+
builder.append(";\n\n return wrap(buffer, offset + %3$s.ENCODED_LENGTH);\n" + " }\n\n");
23732377

2374-
wrapAndApplyHeaderMethodFmtString += ";\n" +
2375-
" return wrap(buffer, offset + %3$s.ENCODED_LENGTH);\n" +
2376-
" }\n\n";
2377-
final String wrapAndApplyHeaderMethod = String.format(
2378-
wrapAndApplyHeaderMethodFmtString,
2378+
final String wrapAndApplyMethod = String.format(
2379+
builder.toString(),
23792380
className,
23802381
mutableBuffer,
23812382
formatClassName(ir.headerStructure().tokens().get(0).applicableTypeName() + "Encoder"));
23822383

23832384
return generateFlyweightCode(
2384-
CodecType.ENCODER, className, token, wrapMethod + wrapAndApplyHeaderMethod, mutableBuffer);
2385+
CodecType.ENCODER, className, token, wrapMethod + wrapAndApplyMethod, mutableBuffer);
23852386
}
23862387

23872388
private CharSequence generateEncoderFields(

0 commit comments

Comments
 (0)