-
Notifications
You must be signed in to change notification settings - Fork 3.4k
FlatBuffers 64 for C++ #7935
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.
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
Merged
Merged
FlatBuffers 64 for C++ #7935
Changes from all commits
Commits
Show all changes
62 commits
Select commit
Hold shift + click to select a range
1f53a53
First working hack of adding 64-bit. Don't judge :)
dbaileychess 0033e15
Made vector_downward work on 64 bit types
dbaileychess a91e67d
vector_downward uses size_t, added offset64 to reflection
dbaileychess 8a4b283
cleaned up adding offset64 in parser
dbaileychess 00a5c35
Add C++ testing skeleton for 64-bit
dbaileychess b08d3ed
working test for CreateVector64
dbaileychess e396b34
working >2 GiB buffers
dbaileychess 44b9b91
support for large strings
dbaileychess d02a187
simplified CreateString<> to just provide the offset type
dbaileychess a00d388
generalize CreateVector template
dbaileychess 7add255
update test_64.afb due to upstream format change
dbaileychess 9ef353a
Added Vector64 type, which is just an alias for vector ATM
dbaileychess 1ca9995
Switch to Offset64 for Vector64
dbaileychess 181ba16
Update for reflection bfbs output change
dbaileychess c130cd2
Starting to add support for vector64 type in C++
dbaileychess e90f0a5
made a generic CreateVector that can handle different offsets and vec…
dbaileychess 35d1df0
Support for 32-vector with 64-addressing
dbaileychess b76bfa0
Vector64 basic builder + tests working
dbaileychess 6d310d6
basic support for json vector64 support
dbaileychess 2a4014b
renamed fields in test_64bit.fbs to better reflect their use
dbaileychess f597385
working C++ vector64 builder
dbaileychess e63aff9
Apply --annotate-sparse-vector to 64-bit tests
dbaileychess 3935ce4
Enable Vector64 for --annotate-sparse-vectors
dbaileychess 5175a82
Merged from upstream
dbaileychess aa96cb6
Add `near_string` field for testing 32-bit offsets alongside
dbaileychess 1cce400
keep track of where the 32-bit and 64-bit regions are for flatbufferb…
dbaileychess 59d84e4
move template<> outside class body for GCC
dbaileychess 574be24
update run.sh to build and run tests
dbaileychess eeb3347
basic assertion for adding 64-bit offset at the wrong time
dbaileychess b8b9bfc
started to separate `FlatBufferBuilder` into two classes, 1 64-bit aw…
dbaileychess eddc21e
add test for nested flatbuffer vector64, fix bug in alignment of big …
dbaileychess b459abf
fixed CreateDirect method by iterating by Offset64 first
dbaileychess 5d355f3
internal refactoring of flatbufferbuilder
dbaileychess c9aaff3
block not supported languages in the parser from using 64-bit
dbaileychess 3ae01bd
evolution tests for adding a vector64 field
dbaileychess 3f0e2ed
conformity tests for adding/removing offset64 attributes
dbaileychess 0830233
ensure test is for a big buffer
dbaileychess 2421d33
add parser error tests for `offset64` and `vector64` attributes
dbaileychess 490dac5
add missing static that GCC only complains about
dbaileychess 99c4700
remove stdint-uintn.h header that gets automatically added
dbaileychess fa456d5
move 64-bit CalculateOffset internal
dbaileychess 3e7bbf9
fixed return size of EndVector
dbaileychess f3ba945
various fixes on windows
dbaileychess cae57e3
add SizeT to vector_downward
dbaileychess b28d0e9
minimze range of size changes in vector and builder
dbaileychess a75b3fb
reworked how tracking if 64-offsets are added
dbaileychess 6d7a8fe
Add ReturnT to EndVector
dbaileychess b51a18d
small cleanups
dbaileychess 38e98d1
remove need for second Array definition
dbaileychess 699b618
combine IndirectHelpers into one definition
dbaileychess 2cc133b
started support for vector of struct
dbaileychess 46f2068
Support for 32/64-vectors of structs + Offset64
dbaileychess d2641a3
small cleanups
dbaileychess d075a35
add verification for vector64
dbaileychess 040cea8
add sized prefix for 64-bit buffers
dbaileychess cb64fef
add fuzzer for 64-bit
dbaileychess 66c8059
add example of adding many vectors using a wrapper table
dbaileychess e7f1afe
run the new -bfbs-gen-embed logic on the 64-bit tests
dbaileychess 6e52dbb
remove run.sh and fix cmakelist issue
dbaileychess b3e3fad
fixed bazel rules
dbaileychess 46a0ea8
fixed some PR comments
dbaileychess 1ac0550
add 64-bit tests to cmakelist
dbaileychess File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.