- 25 Mar, 2019 4 commits
-
-
Austin Schuh authored
$ cat /home/austin/.cache/bazel/_bazel_austin/4b3182bfa237d7e256d9f18ffe58322f/execroot/com_github_google_flatbuffers/bazel-out/k8-opt/testlogs/flatbuffers_test/test.log exec ${PAGER:-/usr/bin/less} "$0" || exit 1 Executing tests from //:flatbuffers_test ----------------------------------------------------------------------------- VALUE: "0" EXPECTED: "1" TEST FAILED: tests/test.cpp:2029, flatbuffers::LoadFile( (test_data_path + "union_vector/union_vector.json").c_str(), false, &jsonfile) in VALUE: "0" EXPECTED: "1" TEST FAILED: tests/test.cpp:2100, parser.Parse(jsonfile.c_str()) in VALUE: "0" EXPECTED: "1" TEST FAILED: tests/test.cpp:2103, VerifyMovieBuffer(jverifier) in
-
László Csomor authored
Use the "simple format" defined here: https://github.com/bazelbuild/continuous-integration/blob/69cf7ec23d984dcbb41fffe946e07ebf5ab7e0f1/buildkite/bazelci.py#L1453 This will unbreak the "Bazel / FlatBuffers" pipeline on Bazel CI, see https://buildkite.com/bazel/flatbuffers
-
Nikolay Kuznetsov authored
-
LouisP authored
-
- 21 Mar, 2019 3 commits
-
-
Henry Lee authored
* Correct the usage in the flathash program As it is possible to have -- before the occurrence of the first input STRING. * Exit with 1 in the flathash program when an error occurs
-
Ashay Shirwadkar authored
Path specified in tutorial file was pointing to invalid location.
-
Wouter van Oortmerssen authored
Change-Id: I92c257ec4ab4a0cf4676bd98523b766ce25bf4f6
-
- 18 Mar, 2019 3 commits
-
-
Vladimir Glavnyy authored
* Unify line ending rules in '.editorconfig' and '.gitattributes' * Revert '.gitattributes' - fix invalid comments in the check-source.py
-
Wouter van Oortmerssen authored
Change-Id: I3e758d44b9845d6df91332bb609b4b7ad88659ac
-
Wouter van Oortmerssen authored
Change-Id: Ia4d56a5eb086f86eb5d1ad6ddae64e4a51bf3aa3
-
- 14 Mar, 2019 1 commit
-
-
Wouter van Oortmerssen authored
This to support code that relied on tables being multiline, but not vectors. This behavior was changed in: https://github.com/google/flatbuffers/commit/b1a925dfc2ac78508aee89029e950057405f0a0a#diff-c45c8fbffbc64f7ff4aa2978612b10d8 Change-Id: I4c95471b643b2b3fab95e06b1294e19d686b492c
-
- 11 Mar, 2019 5 commits
-
-
Wouter van Oortmerssen authored
Original change by: https://github.com/alexames Change-Id: Ib65bd02156d1c3637ed278a8334a2307caacaa44
-
Wouter van Oortmerssen authored
This was incompatible with -Wc++98-c++11-compat on some platforms, due to local variables in the function. Change-Id: Idef510c2cefe944eef2e0656f5a219c2158063e6
-
Vladimir Glavnyy authored
-
Randal Stevens authored
-
Wouter van Oortmerssen authored
implement better custom string type constructor alternative for Unpack() and fix bug with vector of custom string types in Pack(). Squashed commit of the following: commit e9519c647ef949b22756ed660839dd3af854881c Author: Luca Longinotti <luca.longinotti@inivation.com> Date: Tue Mar 5 18:24:49 2019 +0100 tests: regenerate code, reverts change to CreateVectorOfStrings(). commit 117e3b0679209f2aa55cbee18c4036e7da4bd4b3 Author: Luca Longinotti <luca.longinotti@inivation.com> Date: Tue Mar 5 18:15:05 2019 +0100 idl_gen_cpp.cpp: move clang-format on/off outside of declaration, so they are kept properly aligned automatically. commit 4791923806965637d5b13f7003329bfbb2fdf18b Author: Luca Longinotti <luca.longinotti@inivation.com> Date: Tue Mar 5 18:11:40 2019 +0100 idl_gen_cpp.cpp: full clang-format run with provided Google format file, enforce 80 lines width. commit 2f0402f9ff43b1af0a29806932e08e6d64373345 Author: Luca Longinotti <luca.longinotti@inivation.com> Date: Tue Mar 5 18:09:32 2019 +0100 CppUsage: address requested changes. idl_gen_cpp.cpp: fix formatting, keep CreateVectorOfStrings for normal string cases. commit 371d4e0b7972a59e5cff418c44e4043c016ce56a Author: Luca Longinotti <luca.longinotti@inivation.com> Date: Fri Mar 1 16:35:29 2019 +0100 Fix compile error with a vector of non-std::strings. CreateVectorOfStrings() expects a vector of std::string types, but that's not always the case. commit 92b90d7f0fbcfc837a94aa06bedccec0f7b4bb62 Author: Luca Longinotti <luca.longinotti@inivation.com> Date: Fri Mar 1 16:15:36 2019 +0100 Document requirement for custom string types to implement empty() and be constructible from std::string. Add new option --cpp-str-flex-ctor to construct custom string types not via std::string, but (char * + length). commit 28cb2e92d5b7684b5df5184da3a3fad2d0cda733 Author: Luca Longinotti <luca.longinotti@inivation.com> Date: Fri Mar 1 14:31:17 2019 +0100 idl_gen_cpp.cpp: clang-format run, to better separate changes in content from formatting. Change-Id: I4887ba2f2c632b9e7a8c938659b088cd95690870
-
- 09 Mar, 2019 1 commit
-
-
bspeice authored
* Don't use inner attributes for `allow` Messes with being able to easily include elsewhere * Regenerate tests * No-op to retrigger CI * Add the rest of the `allow` attributes
-
- 08 Mar, 2019 2 commits
-
-
Mathias Svensson authored
-
tymcauley authored
Thanks for tackling this, @tymcauley ! * big endian docker test -- wip * tweaks * tweaks * tweaks * docker tweaks * fix conditional compilation issues * reactivate other docker tests * try some more cross-platform config (from tymcauley) * Update tests/docker/languages/Dockerfile.testing.rust.big_endian.1_30_1 Co-Authored-By:
rw <rw@users.noreply.github.com> * Update tests/docker/languages/Dockerfile.testing.rust.big_endian.1_30_1 Co-Authored-By:
rw <rw@users.noreply.github.com> * Update tests/docker/languages/Dockerfile.testing.rust.big_endian.1_30_1 Co-Authored-By:
rw <rw@users.noreply.github.com> * Resolved Rust warnings during big-endian builds. * Unify Rust test suites for x86 and MIPS builds. Note that I had to add four extra packages to the MIPS `Dockerfile`: `libexpat1`, `libmagic1`, `libmpdec2`, and `libreadline7`. For a reason I couldn't identify, even the simplest Rust MIPS binaries run with `qemu-mips` would fail with a segfault when run through this `Dockerfile`. After installing the `gdb-multiarch` package to attempt to debug the issue, the binaries ran successfully. I pared down the packages installed by `gdb-multiarch`, and these four packages are the minimum subset necessary to get Rust MIPS binaries running under `qemu-mips`. * Changed Rust tests to use `Vector`s instead of direct-slice-access. The direct-slice-access method is not available on big-endian targets, but `flatbuffers::Vector`s provide an array interface that is available on all platforms. * Resolved FooStruct endianness issues using explicit struct constructor. This more closely resembles how FlatBuffers structs are constructed in generated Rust code. * Added explanation of how `FooStruct` parallels generated struct code. Also collected duplicate implementations of `FooStruct` into a common location.
-
- 07 Mar, 2019 3 commits
-
-
Wouter van Oortmerssen authored
Change-Id: I6dfadb5289a4396ad2f3d16baf1bdb99c7534174
-
Austin Schuh authored
* Stop building for Windows until the build passes ERROR: D:/b/bk-windows-java8-bd0z/bazel/flatbuffers/BUILD:123:1: Couldn't build file _objs/flatbuffers_test/test.obj: undeclared inclusion(s) in rule '//:flatbuffers_test': this rule is missing dependency declarations for the following files included by 'tests/test.cpp': 'tests/monster_test_generated.h' 'tests/monster_extra_generated.h' The files in tests are being found instead of the generated files since Windows doesn't have any sandboxing. For now, let's disable the rules and come back to it. * Fix buildifier warnings Clean up docstrings and add a module docstring.
-
Wouter van Oortmerssen authored
This for some reason never had a test case, and was broken. Change-Id: If832f5eb8b6c5ba8a75257464892634b38719c55
-
- 06 Mar, 2019 2 commits
-
-
jean-airoldie authored
-
David Reiss authored
Fixes #5206
-
- 04 Mar, 2019 2 commits
-
-
ll-antn authored
* Clean-up nested_parser on all paths (#5179) * Added test for parsing json with invalid nested flatbuffer * Removed utf-8 BOM from test.cpp
-
Laurent Stacul authored
-
- 28 Feb, 2019 1 commit
-
-
csukuangfj authored
-
- 27 Feb, 2019 1 commit
-
-
naure authored
* Fix lifetime in union _as_ accessors In the accessors for union field, the return value is implicitly taking the lifetime of &self. This is irrelevant and prevents usages of the returned value, because it is needlessly bound to the parent field lifetime. This patch makes the return value inherit the lifetime of the data, like other methods do.
-
- 25 Feb, 2019 8 commits
-
-
Kulikov Alexey authored
vtable and vtable size depends only on `Table#bb_pos` but calculated in `Table#_offset` method on each field lookup. Doing this with every call of `Table#__offset` is redundant. These values can be read once with change of `Table#bb_pos` and reused for any field lookup.
-
Kulikov Alexey authored
-
Искандаров Егор authored
Fixed a bug that prevented vtable reuse during buffer construction in the lua library. Also fixed a bug in vtable equality check that was revealed after the first fix.
-
Wouter van Oortmerssen authored
Change-Id: Icdff8a90338bd2ff79ff76fe1ebaa3481da8dea4
-
Marcus Comstedt authored
This fixes the testcase MiniReflectFlatBuffersTest.
-
Brian Wellington authored
On Solaris Sparc, calling NumToString() with a char called the primary version, not the signed char or unsigned char specializations, which caused integer to string conversions to be missed.
-
Wouter van Oortmerssen authored
And fixed 2 more extra semi-colons. Change-Id: I1ee08e5b2f67cd0c886847c05cd4d63cfcafec3e
-
Wouter van Oortmerssen authored
Also fixed warnings this generated. Change-Id: If3607907b2dd609f854b7b81225d9cf66af75a60
-
- 22 Feb, 2019 1 commit
-
-
Brian Wellington authored
For some reason, Offset<T> is being considered a scalar, which causes EndianSwap to be passed an Offset<T>. This doesn't work, as it does not support types with non-trivial constructors. This change adds an overload to WriteScalar(), which works around this.
-
- 21 Feb, 2019 3 commits
-
-
Eric Erhardt authored
* Remove byte* property in ByteBufferAllocator. This allows consumers to read/write into native memory, but without having to always pin the managed `byte[]` when working with managed memory. This allows for users to not need to Dispose() ByteBuffers when they are using the default ByteArrayAllocator class. Instead, we use `Span<byte> GetSpan()` methods to get access to the underlying memory buffer. Fix #5181 * Add a set of benchmark tests. * Add ReadOnly spans. This allows consumers to use ReadOnlyMemory<byte> as the backing storage for ByteBuffers, which is useful in read-only scenarios. * Run tests using ENABLE_SPAN_T in appveyor. * Fix FlatBuffers.Test.csproj to work on older MSBuild versions. * Change the test script to test UNSAFE_BYTEBUFFER * Address PR feedback. Remove IDisposable from ByteBuffer. * Respond to PR feedback.
-
bwelling authored
Add support for using __BYTE_ORDER__, to support platforms where g++ doesn't explicitly define __BIG_ENDIAN__, like Solaris Sparc.
-
Marc Butler authored
* Add RPM packaging support Using the existing PackageDebian as template add support for generating an rpm with the package target. * Restore debian package target Also add an option to advertise the fact. * Update package description C-n-p from README.md * Update rpm package maintainer
-