commit f575b02fda04fe579fb23442234feb8129b77ee2
author Wouter van Oortmerssen <aardappel@gmail.com> Fri Nov 16 16:22:18 2018 -0800
committer Wouter van Oortmerssen <aardappel@gmail.com> Fri Nov 16 17:08:35 2018 -0800
tree 3574a6c6474316723d41ed650821ba94a8a3e3a2
parent 5f32f948102e65eaeea461b44f3b43f96c7a7a5a

Make string/vector field serialization order not depend on optimizer.

Multiple calls of e.g. CreateString inside a call to a CreateTable
could cause those strings to end up in different locations in the
wire format, since order or argument evaluation is undefined.

This is allowed by the FlatBuffer format, but it is not helpful,
especially when debugging the contents of binaries, or comparing
against a "golden" binary for tests etc.

Now making sure that all the CreateTableDirect calls first serialize
sub strings/vectors before calling CreateTable.

Also made similar changes to the serialization of "binary schemas".

Change-Id: I5747c4038b37a0d400aca2bc592bec751cf5c172