Merge pull request #3903 from senior7515/master

Add env var for flatc schema compiler cmake module

Add env var for flatc schema compiler cmake module

Merge branch 'master' of https://github.com/google/flatbuffers into fuzzer2

Added a way to get mutable tables out of a mutable vector of tables.

Change-Id: I8f9ef1d14f86671ced929c7a159115b1d8510327
Tested: on Linux.

Fixed LoadFile crashing on directory arguments.

Change-Id: I737f6fd0bb1302ab7cfa6ab8b06108e221ebb63c
Tested: on Linux.

Fixed compile errors on Windows

Merge pull request #3885 from Lakedaemon/shareWarningAndTwoMethods

Share warning and two methods

Merge pull request #3870 from martinus/master

Added helpers to access objects while creating the flatbuffer.

Merge pull request #3891 from bog-dan-ro/master

More bit fields patches

renamed functions

More descriptive name, show that it's really just a temporary object.

Don't generate MIN & MAX enum values if "--no-prefix" is used.

Close #3892

MIN & MAX are useless for bit_flags

Instead we need NONE (0) and ANY (all orred values), if the user didn't
already defined them

Add ! operator

We already know the underlying_type.

We just need to pass it to DEFINE_BITMASK_OPERATORS macro

Merge pull request #3888 from bog-dan-ro/master

Define bit mask operators for scoped enums bitfields

Merge pull request #3889 from PSPDFKit-labs/james/avoid-msvc-align-error

MSVC doesn't allow alignment on function params.

Define bit mask operators for scoped enums bitfields

Close #3887

sharing namespace_dir and the namespace string methods

sharing the Flatbuffers warning

Add simple libfuzzer based test.

This tests the verifier and the parser.

Tested: on Linux.
Bug: 27230459
Change-Id: I3417faba9385dc941199c57a08d8a50c2faec1aa

Merge pull request #3880 from Lakedaemon/IsEverythingGenerated

shared method that exits early if everything is generated

shared method that exits early if everything is generated

MSVC doesn't allow alignment on function params. This prevents the copy and thus avoids this issue https://msdn.microsoft.com/en-us/library/373ak2y1.aspx

Merge "Uncomment the corpus attribute since b/28762769 is fixed, add more items to the corpus (taken from the ClusterFuzz-generated corpus)." into ub-games-master

Merge "Add a fuzzer for flatbuffers, derived from the one used in Chromium: https://code.google.com/p/chromium/codesearch#chromium/src/testing/libfuzzer/fuzzers/flatbuffers_verifier_fuzzer.cc" into ub-games-master

Merge pull request #3865 from bhamiltoncx/surrogate-pairs

Handle \u-escaped surrogate pairs correctly in IDL parser

Merge "Removed reference to deprecated NDK toolchain version." into ub-games-master

Handle \u-escaped surrogate pairs correctly in IDL parser

Uncomment the corpus attribute since b/28762769 is fixed,
add more items to the corpus (taken from the ClusterFuzz-generated corpus).

(mirrored from cl 122690857)

Change-Id: I98c7672169a9a27fe18bedae8501e6fb85a0681d

Add a fuzzer for flatbuffers, derived from the one used in Chromium:
https://code.google.com/p/chromium/codesearch#chromium/src/testing/libfuzzer/fuzzers/flatbuffers_verifier_fuzzer.cc

(mirrored from cl 122676317)

Change-Id: I0074af435cf37f5e6368b040563c3733d9e02345

Removed reference to deprecated NDK toolchain version.

Change-Id: I8d39a66ca5642427619c60f349fa78917aec2455

Merge branch 'master' of https://github.com/google/flatbuffers

Fixed Google Groups link in documentation.

Change-Id: I64307ccd0a6f23a7c3860d602813e637415e7c91

Made .sh files all have same comment formatting.

Change-Id: Ie7cc268a8823aec1d06c8b7cbef54068d832de3a

Merge pull request #3878 from pjulien/no_alloc_on_cb

Remove Remaining FBB#createString Allocation

Clarify documentation based on review comments

Removed BOM marker from some files.

Also removed some trailing whitespace.

Change-Id: I40bf4f776ca4d467b7ee6f2c9a7500c13700f9d5

Merge branch 'master' of https://github.com/google/flatbuffers

Added missing licenses to some sh/cs/py files.

Bug: https://github.com/google/flatbuffers/issues/3872
Change-Id: I5d551168e9bc925e867e5e4ddf5d809418fd44eb
Tested: on Linux.

If the user passes in a {{CharBuffer}} himser, reward him by
eliminating the last remaining allocation in string creation

Merge pull request #3867 from dreifachstein/master

Use the build output flatc to compile test schemas by default.

Merge pull request #3866 from bhamiltoncx/fix-undefined-bit-shift

Fix undefined behavior when CheckBitsFit left-shifts 64-bit value by 64 bits

Merge branch 'master' of https://github.com/google/flatbuffers

Fixes verifier not catching odd vtable sizes.

Bug: 28762769
Change-Id: Iec370482f67c4585e97dd5175c1a4258ddba47fe
Tested: on Linux.

Merge pull request #3877 from ozkuran/patch-1

Fixed Library Location in CppUsage.md

Fixed Library Location in CppUsage.md

Fixed Library Location
"https://github.com/google/flatbuffers/tree/master/include/flatbuffers"
in CppUsage.md

Added helpers to access objects while creating the flatbuffer.

GetObject and GetMutableObject are similar to GetRoot and GetMutableRoot,
and can be useful when wanting to access data that has just been created.
Unfortunately there is a danger in using these methods, as it is possible
that the buffer reallocates which will invalidate the pointers.

Use the build output flatc to compile test schemas by default.

Fix undefined behavior in CheckBitsFit bit-shift on size_t

Merge pull request #3806 from mediagraph/fix-create-string

Fix CreateString with already-encoded string or bytearray in Python 2.7.

Merge pull request #3854 from pjulien/3853

Fix for #3853

Merge pull request #3843 from Lakedaemon/refactoring

Transition 1 (nice diffs) Go, C++ and General generators with class

Merge pull request #3863 from pjulien/cseq

Can now use ``CharSequence`` of ``String``.  Works with

memory leak fix + php/js/python transition to class based generator

Can now use ``CharSequence`` of ``String``.  Works with

1. [AsciiString]
(http://netty.io/4.1/api/io/netty/util/AsciiString.html)
2. [CharBuffer]
(https://docs.oracle.com/javase/8/docs/api/java/nio/CharBuffer.html)
3. [StringBuilder]
(https://docs.oracle.com/javase/8/docs/api/java/lang/StringBuilder.html)

Merge pull request #3859 from rw/go-zero-alloc-strings

Go: Fix heap allocation when reading a string.

Fix heap allocation when reading a string.

Merge pull request #3850 from pjulien/3849

Fix for #3849

Merge branch 'master' of https://github.com/google/flatbuffers

Fixed a verifier check that didn't return from its parent.

Change-Id: I1aef5fdba72de0c3bb8276c9f50fffa494f26e29
Tested: on Linux.

Fixed SkipByteOrderMark advancing cursor_ too early.

Change-Id: Ie60f01d935ca6b4aa6ce0eab7598602ac0758342
Tested: on Linux.

Fix for #3849

This avoids the following allocations over repeated use:

- A ``CharsetEncoder``
- A byte array to contain the contents of the string
- A wrapper, via ``ByteBuffer#wrap``, for the previously mentioned byte
  array

This also removes a copy of the temporary byte array.  This extra copy
is needed because ``String#getBytes`` returns a byte array and its
length must match the exact size of the contents.

This implementation requires that the builder retain a ``ByteBuffer``
and ``CharEncoder``.  This is considered slower only for users that
will only allocate a single string in their buffers or none at all.

The improvement is especially potent if the builder is constantly reused
by its caller.  This also eliminates the penalty for users that do not
use strings at all since the cost of allocating these fields is
now amortized.

The only allocation left in this code is a ``CharBuffer`` wrapper.  It's
possible to eliminate this allocation using an additional copy but need
further profiling to see which is the bigger problem.

Fixed float suffix not being added on implicit float defaults.

Change-Id: I8dcbb5e93836356511d411ef86452babd3dd3c0b
Tested: on Linux.

size the allocation to the required bytes

fix file permission

Fix for #3853

Removes the following allocations:
- ``CharsetDecoder`` is reused between calls
- ``CharBuffer#wrap`` removed in favor of heap
  based char buffer that is reused
- Temporary ``char[]``, an intermediate copy inside ``StringCoding``
- Another ``char[]``, this is needed because ``StringCoding`` uses
  a ``CharBuffer`` internally but returns a ``char[]``.  Extra
  characters need to be trimmed so this means yet another allocation
- Yet another ``char[]`` directly from ``__string`` for non-heap
  based buffers

Removes the following copies
- No copy is performed to trim the allocation since a ``CharBuffer``
  is used directly
- For non-heap based byte buffers, removes the copy that was
  previously done in the __string function

This does need to get the TLS entry which implies at least some
contention on the thread object table and a fence.

Merge pull request #3848 from pjulien/3847

Fix for 3847

Fixed whole-number float default values missing a .0 suffix.

A previous commit that added "f" for C++/Java/C# would break
on gcc of constants like 3f, which are now output as 3.0f

Tested: on Linux

Change-Id: If9cabbe3c6d6948a5050b8b123bda9c06e181f52

3847

Enforcing the google C++ style guide

Merge branch 'master' of https://github.com/google/flatbuffers

Fixed missing "f" on float default values in C++/Java/C#

Bug: 28145201
Change-Id: I5a1818c40c2f4fbd00cfec3362040407976f7a39
Tested: on Linux.

Merge pull request #3844 from Nnamdi/CreateUninitializedVector_fix

Stop CreateUninitializedVector returning a pointer to invalid memory.

Stop CreateUninitializedVector returning a pointer to invalid memory.

CreateUninitializedVector was performing the following actions:
    1. call StartVector.
    2. call make_space, and set buf to point to the reserved space.
    3. call EndVector.

The problem is that a call to EndVector can ultimately call make_space, which
if the buffer is full, will cause a reallocation, invalidating the value stored
in buf.  So setting buf needs to be delayed until after EndVector.

The following code, when run under valgrind shows a write to free'd memory before
the change, but no such error after:

int main()
{
    flatbuffers::FlatBufferBuilder fbb(128);
    char *buf = nullptr;
    fbb.CreateUninitializedVector(128, &buf);
    *buf = 0;
}

Transition 1 (with nice diffs) Go, Cpp and General code generators with class

Fixed struct -> class pre-declaration error.

Change-Id: I3730a6823d603982c4250d514e827e3f93d1f81e
Tested: on Linux.

Made user-defined attributes available in the reflection data.

Tested: on Linux.

Bug: 27923233
Change-Id: Ic16675650e4a8e138c163f1e2131a3aad7008ada

Merge branch 'master' of https://github.com/google/flatbuffers

Fixed JSON integers as strings not being parsed correctly.

Change-Id: If34aef46ef619e06de799cff14948b02e98da86d
Tested: on Linux.

Fixed pre-declarations of included types in generated code.

This was harmless, but unnecessary.

Change-Id: I1365a725e254d4ebbda081110a50277cb5118323
Tested: on Linux.

Merge pull request #3789 from Chippiewill/master

Allow builds on platforms with unsigned chars by default

Merge pull request #3839 from klapek/net/bytebuffer-assert-optimization

Remove unnecessary condition in AssertOffsetAndLength.

Remove unnecessary condition in AssertOffsetAndLength.

Condition is already checked and covered in the following condition, the second one was excess.

Signed-off-by: Wojciech Jaszczak <jaszczakw@gmail.com>

Merge pull request #3832 from mikkelfj/c_docs

Documentation for C bindings

Update C documentation with supported MSVC versions, and other minor updates.

Added missing util.cpp to VS project

Adding missing util.cpp to Xcode project

Merge branch 'master' of https://github.com/google/flatbuffers

Merge pull request #3836 from tkelman/cygwinbuild

Fix build on Cygwin

add setloadfile and setfileexists functions to flatbuffers

to allow for custom file loaders. The targeted use case is android runtime.

(from CL 116980408)

Change-Id: I8785c0acf714fab41d8f6fc9f1c52875423b8f5b

Fix flatbuffers conversion warnings.

(from CL 116785833)

Change-Id: I0fb0e1318fd70cba51e229a93f535c6639bbb331

Fix flatc build for msvc.

(from CL 117010760)

Change-Id: I0ee77cf87d0cb00b94887356745b85423b1375a6

Fixed potential integer overflow in the Verifier.

This overflow could allow malformed FlatBuffers to pass the verifier.

Change-Id: Ia993299a761b00b93e53e8aff2689b631755763c
Tested: on Linux.
Bug: 27903580

Fixed VS assignment constructor warning.

Remove details on FlatCC builds that would easily become outdated

Document FlatCC support for Windows

Fix build on Cygwin

use -std=gnu++11 instead of c++0x on cygwin to avoid
error: realpath was not declared in this scope

ref http://www.cygwin.com/ml/cygwin/2016-03/msg00005.html

Merge pull request #3816 from armen/master

Implement __vector_as_bytes and methods to get [ubyte] efficiently

Merge pull request #3834 from Nnamdi/gen_name_strings

Added --gen-name-strings command line option.

Added --gen-name-strings command line option.

To support the use case described in issue google/flatbuffers#3826, a new command line option --gen-name-strings
has been added, which will cause a static GetFullyQualifiedName function to be added
to the C++ output for tables/structs.