Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@paulovap
Copy link
Contributor

@paulovap paulovap commented Sep 3, 2019

This PR adds more documentation for FlexBuffers in Java.

Additionally adds a new constructor to FlexBufferBuilder to specify buffer size.

@aardappel
Copy link
Collaborator

There appear to be some compile errors in the CI, e.g.

./../java/com/google/flatbuffers/FlexBuffers.java:160: error: cannot find symbol
        case 1: return byteToUnsignedInt(buff.get(end));

Also add a FlexBuffer constructor to simplify usage
@paulovap
Copy link
Contributor Author

paulovap commented Sep 3, 2019

There appear to be some compile errors in the CI, e.g.

./../java/com/google/flatbuffers/FlexBuffers.java:160: error: cannot find symbol
        case 1: return byteToUnsignedInt(buff.get(end));

Yes, I reduced the visibility of class from package-private to private, which triggered the error. I reverted the change. Hopefully it will be back to normal

@paulovap
Copy link
Contributor Author

paulovap commented Sep 10, 2019

Hello @aardappel. Do you want to see any more changes on this MR?

@aardappel
Copy link
Collaborator

No, sorry.. the remaining CI errors are unrelated.

@aardappel aardappel merged commit b5560fc into google:master Sep 10, 2019
LuckyRu pushed a commit to LuckyRu/flatbuffers that referenced this pull request Oct 2, 2020
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