add LambdaSerializer and fix array length issue

arangodb / velocypack

A fast and compact format for serialization and storage

Other

420 stars 40 forks source link

add LambdaSerializer and fix array length issue #81

Closed siilike closed 3 years ago

siilike commented 3 years ago

Fixes #80.

coveralls commented 3 years ago

Coverage increased (+0.009%) to 94.482% when pulling c2371edeb80e76ae25c23a094df1e23d384553fe on siilike:master into 5bed1dacd896dfa442092d44d65e811d2a5db01d on arangodb:master.

jsteemann commented 3 years ago

@siilike : please note that we switched the main branch for this repository from master to main some time ago. Can you rebase your PR against the main branch, please? Thanks!

siilike commented 3 years ago

Sorry, didn't notice. Unfortunately I don't have time to debug this again (and you already have a partial solution).

Still, would be great if you included the LambdaSerializable, either the original version or the one suggested by @maierlars, I don't know enough C++ to take it to the next level.