Skip to content

[Java] Print char fields as char instead of numeric (when primitiveType="char")#1090

Open
pcdv wants to merge 1 commit intoaeron-io:masterfrom
pcdv:master
Open

[Java] Print char fields as char instead of numeric (when primitiveType="char")#1090
pcdv wants to merge 1 commit intoaeron-io:masterfrom
pcdv:master

Conversation

@pcdv
Copy link
Copy Markdown
Contributor

@pcdv pcdv commented Sep 25, 2025

Tentative fix for #1087

@mikeb01
Copy link
Copy Markdown
Contributor

mikeb01 commented Apr 21, 2026

Is it possible to include a test for this?

@pcdv pcdv reopened this May 4, 2026
@pcdv
Copy link
Copy Markdown
Contributor Author

pcdv commented May 4, 2026

I wanted to add a test in JavaGeneratorTest but most tests are already failing with:

java.lang.IllegalStateException: Invalid property name='false' please correct the schema or consider setting system property: sbe.keyword.append.token

at uk.co.real_logic.sbe.generation.java.JavaUtil.formatForJavaKeyword(JavaUtil.java:178)
at uk.co.real_logic.sbe.generation.java.JavaGenerator.generateEnumValues(JavaGenerator.java:2243)
at uk.co.real_logic.sbe.generation.java.JavaGenerator.generateEnum(JavaGenerator.java:1980)
at uk.co.real_logic.sbe.generation.java.JavaGenerator.generateTypeStubs(JavaGenerator.java:209)
at uk.co.real_logic.sbe.generation.java.JavaGenerator.generate(JavaGenerator.java:257)
at uk.co.real_logic.sbe.generation.java.JavaGeneratorTest.shouldGenerateRepeatingGroupCountLimits(JavaGeneratorTest.java:494)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants