References inside messageHeader not working

[ksergey]

Hello Devs,

Example:

        <set name="UpdateFlags" encodingType="uint8">
            <choice name="Begin">0</choice>
            <choice name="End">1</choice>
        </set>
        <composite name="messageHeader" description="Template ID and length of message root">
            <type name="blockLength" primitiveType="uint16"/>
            <type name="templateId" primitiveType="uint16"/>
            <type name="schemaId" primitiveType="uint16"/>
            <type name="version" primitiveType="uint16"/>
            <ref name="flags" type="UpdateFlags"/>
        </composite>

Generated C++ code looks like:

private:                                                                                        
    Flags m_flags;

public:

    Flags &flags()
    {   
        m_flags.wrap(m_buffer, m_offset + 8, m_actingVersion, m_bufferLength);
        return m_flags;
    }   

But class Flags not generated and I believe class name should be UpdateFlags (not Flags)

Thanks

[mjpt777]

@billsegall Worth tracking this one to see the changes that need to be applied to the Golang version.

[tmontgomery]

C++ has been changed to follow Java (as of commit 323ece0). Might be good to check and see if that addresses the issue.

[tmontgomery]

now it should be working, I think.

[ksergey]

@tmontgomery not fully resolved - MessageHeader.h do not include required headers.
Example above should include "UpdateFlags.h"

[tmontgomery]

MessageHeader.h is only designed to include the MessageHeader type itself. It doesn't include any others.

[mjpt777]

@tmontgomery I think a message header should support the minimum four fields and it is OK to add on additional fields at the end. The Java implementation now allows this. The spec does not say it must only be these four fields.

[tmontgomery]

@mjpt777 true. The spec does not forbid it. But the IR HeaderStructure class only picks up those 4 token names. So, the IR itself doesn't contain anything else for the header structure.

[tmontgomery]

I believe this would effect the OTF as well, for example.

[tmontgomery]

i.e. https://github.com/real-logic/simple-binary-encoding/blob/master/sbe-tool/src/main/java/uk/co/real_logic/sbe/ir/HeaderStructure.java#L52

[mjpt777]

There is a default header with a minimum set of requirements. An OTF implementation could use its own extended header decoder.