Use RecursiveCopyVectorTable instead
The other one started using CopyVectorTable, which was pulling vtables
from the front of the log and contents from the end, making
the deduplicated config actually bigger than when it started. Recursive
copy will repack and makes it faster.
Change-Id: I4aa2f0bf102ee13942744e485a3d53884cd2ba07
diff --git a/aos/flatbuffer_merge.h b/aos/flatbuffer_merge.h
index 7518bed..537d121 100644
--- a/aos/flatbuffer_merge.h
+++ b/aos/flatbuffer_merge.h
@@ -123,14 +123,14 @@
template <class T>
inline flatbuffers::Offset<flatbuffers::Vector<flatbuffers::Offset<T>>>
-CopyVectorTable(const flatbuffers::Vector<flatbuffers::Offset<T>> *t1,
- flatbuffers::FlatBufferBuilder *fbb) {
+RecursiveCopyVectorTable(const flatbuffers::Vector<flatbuffers::Offset<T>> *t1,
+ flatbuffers::FlatBufferBuilder *fbb) {
if (t1 == nullptr) {
return 0;
}
std::vector<flatbuffers::Offset<T>> v;
for (const T *t : *t1) {
- v.emplace_back(CopyFlatBuffer(t, fbb));
+ v.emplace_back(RecursiveCopyFlatBuffer(t, fbb));
}
return fbb->CreateVector(v);
}