Skip to content

Commit 77b036f

Browse files
committed
[Java] Remove unnecessary allocation.
1 parent 1c80149 commit 77b036f

File tree

1 file changed

+10
-26
lines changed

1 file changed

+10
-26
lines changed

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

Lines changed: 10 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -437,7 +437,7 @@ private CharSequence generateCompositePropertyElements(final List<Token> tokens,
437437
final Token token = tokens.get(i);
438438
final String propertyName = formatPropertyName(token.name());
439439

440-
// FIXME: do I need to pass classname down here for diambiguation
440+
// FIXME: do I need to pass classname down here for disambiguation
441441
switch (token.signal())
442442
{
443443
case ENCODING:
@@ -616,12 +616,10 @@ private CharSequence generatePrimitiveFieldMetaData(
616616
final Token token,
617617
final String indent)
618618
{
619-
final StringBuilder sb = new StringBuilder();
620-
621619
final PrimitiveType primitiveType = token.encoding().primitiveType();
622620
final String typeName = cSharpTypeName(primitiveType);
623621

624-
sb.append(String.format(
622+
return String.format(
625623
"\n" +
626624
indent + "public const %1$s %2$sNullValue = %3$s;\n" +
627625
indent + "public const %1$s %2$sMinValue = %4$s;\n" +
@@ -630,9 +628,7 @@ private CharSequence generatePrimitiveFieldMetaData(
630628
toUpperFirstChar(propertyName),
631629
generateLiteral(primitiveType, token.encoding().applicableNullValue().toString()),
632630
generateLiteral(primitiveType, token.encoding().applicableMinValue().toString()),
633-
generateLiteral(primitiveType, token.encoding().applicableMaxValue().toString())));
634-
635-
return sb;
631+
generateLiteral(primitiveType, token.encoding().applicableMaxValue().toString()));
636632
}
637633

638634
private CharSequence generateSingleValueProperty(
@@ -646,9 +642,7 @@ private CharSequence generateSingleValueProperty(
646642
final ByteOrder byteOrder = token.encoding().byteOrder();
647643
final String byteOrderStr = generateByteOrder(byteOrder, token.encoding().primitiveType().size());
648644

649-
final StringBuilder sb = new StringBuilder();
650-
651-
sb.append(String.format("\n" +
645+
return String.format("\n" +
652646
indent + "public %1$s %2$s\n" +
653647
indent + "{\n" +
654648
indent + INDENT + "get\n" +
@@ -666,9 +660,7 @@ private CharSequence generateSingleValueProperty(
666660
generateFieldNotPresentCondition(token.version(), token.encoding(), indent),
667661
typePrefix,
668662
offset,
669-
byteOrderStr));
670-
671-
return sb;
663+
byteOrderStr);
672664
}
673665

674666
private CharSequence generateFieldNotPresentCondition(
@@ -1105,9 +1097,7 @@ private CharSequence generateEnumProperty(final String propertyName, final Token
11051097
final ByteOrder byteOrder = token.encoding().byteOrder();
11061098
final String byteOrderStr = generateByteOrder(byteOrder, token.encoding().primitiveType().size());
11071099

1108-
final StringBuilder sb = new StringBuilder();
1109-
1110-
sb.append(String.format("\n" +
1100+
return String.format("\n" +
11111101
indent + INDENT + "public %1$s %2$s\n" +
11121102
indent + INDENT + "{\n" +
11131103
indent + INDENT + INDENT + "get\n" +
@@ -1127,23 +1117,19 @@ private CharSequence generateEnumProperty(final String propertyName, final Token
11271117
typePrefix,
11281118
offset,
11291119
byteOrderStr,
1130-
enumUnderlyingType));
1131-
1132-
return sb;
1120+
enumUnderlyingType);
11331121
}
11341122

1135-
private Object generateBitSetProperty(final String propertyName, final Token token, final String indent)
1123+
private String generateBitSetProperty(final String propertyName, final Token token, final String indent)
11361124
{
1137-
final StringBuilder sb = new StringBuilder();
1138-
11391125
final String bitSetName = formatClassName(token.applicableTypeName());
11401126
final int offset = token.offset();
11411127
final String typePrefix = toUpperFirstChar(token.encoding().primitiveType().primitiveName());
11421128
final ByteOrder byteOrder = token.encoding().byteOrder();
11431129
final String byteOrderStr = generateByteOrder(byteOrder, token.encoding().primitiveType().size());
11441130
final String typeName = cSharpTypeName(token.encoding().primitiveType());
11451131

1146-
sb.append(String.format("\n" +
1132+
return String.format("\n" +
11471133
indent + INDENT + "public %1$s %2$s\n" +
11481134
indent + INDENT + "{\n" +
11491135
indent + INDENT + INDENT + "get\n" +
@@ -1163,9 +1149,7 @@ private Object generateBitSetProperty(final String propertyName, final Token tok
11631149
typePrefix,
11641150
offset,
11651151
byteOrderStr,
1166-
typeName));
1167-
1168-
return sb;
1152+
typeName);
11691153
}
11701154

11711155
private Object generateCompositeProperty(final String propertyName, final Token token, final String indent)

0 commit comments

Comments
 (0)