aeron-io
diff --git a/‎sbe-tool/src/main/java/uk/co/real_logic/sbe/generation/java/JavaGenerator.java
Copy file name to clipboard
+44-25Lines changed: 44 additions & 25 deletions b/‎sbe-tool/src/main/java/uk/co/real_logic/sbe/generation/java/JavaGenerator.java
Copy file name to clipboard
+44-25Lines changed: 44 additions & 25 deletions
@@ -1101,12 +1101,13 @@ private void generateComposite(final List<Token> tokens) throws IOException
                 final StringBuilder sb = new StringBuilder();
                 generateEncodingOffsetMethod(sb, propertyName, encodingToken.offset(), BASE_INDENT);
                 generateEncodingLengthMethod(sb, propertyName, encodingToken.encodedLength(), BASE_INDENT);
+                generateFieldSinceVersionMethod(sb, encodingToken, BASE_INDENT);
 
                 switch (encodingToken.signal())
                 {
                     case ENCODING:
                         out.append(sb).append(generatePrimitiveDecoder(
-                            true, encodingToken.name(), encodingToken, BASE_INDENT));
+                            true, encodingToken.name(), encodingToken, encodingToken, BASE_INDENT));
                         break;
 
                     case BEGIN_ENUM:
@@ -1524,19 +1525,24 @@ private static CharSequence generateEnumDeclaration(final String name, final Tok
     }
 
     private CharSequence generatePrimitiveDecoder(
-        final boolean inComposite, final String propertyName, final Token token, final String indent)
+        final boolean inComposite,
+        final String propertyName,
+        final Token propertyToken,
+        final Token encodingToken,
+        final String indent)
     {
         final StringBuilder sb = new StringBuilder();
 
-        sb.append(generatePrimitiveFieldMetaData(propertyName, token, indent));
+        sb.append(generatePrimitiveFieldMetaData(propertyName, encodingToken, indent));
 
-        if (token.isConstantEncoding())
+        if (encodingToken.isConstantEncoding())
         {
-            sb.append(generateConstPropertyMethods(propertyName, token, indent));
+            sb.append(generateConstPropertyMethods(propertyName, encodingToken, indent));
         }
         else
         {
-            sb.append(generatePrimitivePropertyDecodeMethods(inComposite, propertyName, token, indent));
+            sb.append(generatePrimitivePropertyDecodeMethods(
+                inComposite, propertyName, propertyToken, encodingToken, indent));
         }
 
         return sb;
@@ -1562,11 +1568,16 @@ private CharSequence generatePrimitiveEncoder(
     }
 
     private CharSequence generatePrimitivePropertyDecodeMethods(
-        final boolean inComposite, final String propertyName, final Token token, final String indent)
+        final boolean inComposite,
+        final String propertyName,
+        final Token propertyToken,
+        final Token encodingToken,
+        final String indent)
     {
-        return token.matchOnLength(
-            () -> generatePrimitivePropertyDecode(inComposite, propertyName, token, indent),
-            () -> generatePrimitiveArrayPropertyDecode(inComposite, propertyName, token, indent));
+        return encodingToken.matchOnLength(
+            () -> generatePrimitivePropertyDecode(inComposite, propertyName, propertyToken, encodingToken, indent),
+            () -> generatePrimitiveArrayPropertyDecode(
+                inComposite, propertyName, propertyToken, encodingToken, indent));
     }
 
     private CharSequence generatePrimitivePropertyEncodeMethods(
@@ -1617,12 +1628,16 @@ private CharSequence generatePrimitiveFieldMetaData(
     }
 
     private CharSequence generatePrimitivePropertyDecode(
-        final boolean inComposite, final String propertyName, final Token token, final String indent)
+        final boolean inComposite,
+        final String propertyName,
+        final Token propertyToken,
+        final Token encodingToken,
+        final String indent)
     {
-        final Encoding encoding = token.encoding();
+        final Encoding encoding = encodingToken.encoding();
         final String javaTypeName = javaTypeName(encoding.primitiveType());
 
-        final int offset = token.offset();
+        final int offset = encodingToken.offset();
         final String byteOrderStr = byteOrderString(encoding);
 
         return String.format(
@@ -1634,7 +1649,7 @@ private CharSequence generatePrimitivePropertyDecode(
             indent + "    }\n\n",
             javaTypeName,
             propertyName,
-            generateFieldNotPresentCondition(inComposite, token.version(), encoding, indent),
+            generateFieldNotPresentCondition(inComposite, propertyToken.version(), encoding, indent),
             generateGet(encoding.primitiveType(), "offset + " + offset, byteOrderStr));
     }
 
@@ -1723,13 +1738,17 @@ private static CharSequence generatePropertyNotPresentCondition(
     }
 
     private CharSequence generatePrimitiveArrayPropertyDecode(
-        final boolean inComposite, final String propertyName, final Token token, final String indent)
+        final boolean inComposite,
+        final String propertyName,
+        final Token propertyToken,
+        final Token encodingToken,
+        final String indent)
     {
-        final Encoding encoding = token.encoding();
+        final Encoding encoding = encodingToken.encoding();
         final String javaTypeName = javaTypeName(encoding.primitiveType());
-        final int offset = token.offset();
+        final int offset = encodingToken.offset();
         final String byteOrderStr = byteOrderString(encoding);
-        final int fieldLength = token.arrayLength();
+        final int fieldLength = encodingToken.arrayLength();
         final int typeSize = sizeOfPrimitive(encoding);
 
         final StringBuilder sb = new StringBuilder();
@@ -1750,7 +1769,7 @@ private CharSequence generatePrimitiveArrayPropertyDecode(
             javaTypeName,
             propertyName,
             fieldLength,
-            generateFieldNotPresentCondition(inComposite, token.version(), encoding, indent),
+            generateFieldNotPresentCondition(inComposite, propertyToken.version(), encoding, indent),
             offset,
             typeSize,
             generateGet(encoding.primitiveType(), "pos", byteOrderStr)));
@@ -1774,7 +1793,7 @@ private CharSequence generatePrimitiveArrayPropertyDecode(
                 indent + "    }\n",
                 Generators.toUpperFirstChar(propertyName),
                 fieldLength,
-                generateArrayFieldNotPresentCondition(token.version(), indent),
+                generateArrayFieldNotPresentCondition(propertyToken.version(), indent),
                 offset));
 
             sb.append(String.format("\n" +
@@ -1788,7 +1807,7 @@ private CharSequence generatePrimitiveArrayPropertyDecode(
                 indent + "        return new String(dst, 0, end, %s);\n" +
                 indent + "    }\n\n",
                 formatPropertyName(propertyName),
-                generateStringNotPresentCondition(token.version(), indent),
+                generateStringNotPresentCondition(propertyToken.version(), indent),
                 fieldLength, offset,
                 fieldLength, fieldLength,
                 charset(encoding.characterEncoding())));
@@ -2332,7 +2351,7 @@ private CharSequence generateDecoderFields(final List<Token> tokens, final Strin
                 switch (typeToken.signal())
                 {
                     case ENCODING:
-                        sb.append(generatePrimitiveDecoder(false, propertyName, typeToken, indent));
+                        sb.append(generatePrimitiveDecoder(false, propertyName, fieldToken, typeToken, indent));
                         break;
 
                     case BEGIN_ENUM:
@@ -2464,7 +2483,7 @@ private CharSequence generateEnumDecoder(
                 indent + "    }\n\n",
                 enumName,
                 propertyName,
-                generatePropertyNotPresentCondition(inComposite, DECODER, token.version(), indent),
+                generatePropertyNotPresentCondition(inComposite, DECODER, signalToken.version(), indent),
                 enumName,
                 generateGet(encoding.primitiveType(), "offset + " + token.offset(), byteOrderString(encoding)));
         }
@@ -2522,7 +2541,7 @@ private CharSequence generateBitSetProperty(
             generateFlyweightPropertyJavadoc(indent + INDENT, propertyToken, bitSetName),
             bitSetName,
             propertyName,
-            generatePropertyNotPresentCondition(inComposite, codecType, bitsetToken.version(), indent),
+            generatePropertyNotPresentCondition(inComposite, codecType, propertyToken.version(), indent),
             propertyName,
             bitsetToken.offset(),
             propertyName));
@@ -2558,7 +2577,7 @@ private CharSequence generateCompositeProperty(
             generateFlyweightPropertyJavadoc(indent + INDENT, propertyToken, compositeName),
             compositeName,
             propertyName,
-            generatePropertyNotPresentCondition(inComposite, codecType, compositeToken.version(), indent),
+            generatePropertyNotPresentCondition(inComposite, codecType, propertyToken.version(), indent),
             propertyName,
             compositeToken.offset(),
             propertyName));