Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Kotlin] Control the generation of reflection with --reflect-names #7775

Merged
merged 3 commits into from Jan 10, 2023
Merged

[Kotlin] Control the generation of reflection with --reflect-names #7775

merged 3 commits into from Jan 10, 2023

Conversation

anton-bobukh
Copy link
Contributor

Tested:

$ cmake -G "Unix Makefiles" && make && ./tests/flatc/main.py
...
KotlinTests.EnumValAttributes
 [PASSED]
KotlinTests.EnumValAttributes_ReflectNames
 [PASSED]
KotlinTests: 2 of 2 passsed
...

35 of 35 tests passed

@github-actions github-actions bot added c++ codegen Involving generating code from schema kotlin python labels Jan 9, 2023
@anton-bobukh
[Kotlin] Control the generation of reflection with --reflect-names.
0bd312c 
Tested:
```
$ cmake -G "Unix Makefiles" && make && ./tests/flatc/main.py
...
KotlinTests.EnumValAttributes
 [PASSED]
KotlinTests.EnumValAttributes_ReflectNames
 [PASSED]
KotlinTests: 2 of 2 passsed
...

35 of 35 tests passed
```
@anton-bobukh
[Kotlin] Fix SampleBinary by converting Byte to UByte for ubyte fields.
586f599
@anton-bobukh
[Kotlin] Annotate all generated classes with kotlin.ExperimentalUnsig…
34eedaf 
…nedTypes.
@dbaileychess dbaileychess merged commit b50b6be into google:master Jan 10, 2023
@aaronriekenberg aaronriekenberg mentioned this pull request Mar 8, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dbaileychess dbaileychess dbaileychess approved these changes

@paulovap paulovap Awaiting requested review from paulovap

Assignees
No one assigned
Labels
c++ codegen Involving generating code from schema kotlin python
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@anton-bobukh @dbaileychess