[PATCH v3 4/7] lua: add non-recursive msgpack decoding functions

Alexander Turenko alexander.turenko at tarantool.org
Tue Apr 30 21:38:55 MSK 2019


On Tue, Apr 30, 2019 at 04:03:17PM +0300, Vladimir Davydov wrote:
> On Wed, Apr 10, 2019 at 06:21:22PM +0300, Alexander Turenko wrote:
> > Needed for #3276.
> > 
> > @TarantoolBot document
> > Title: Non-recursive msgpack decoding functions
> > 
> > Contracts:
> > 
> > ```
> > msgpack.decode_array(buf.rpos, buf:size()) -> arr_len, new_rpos
> > msgpack.decode_map(buf.rpos, buf:size()) -> map_len, new_rpos
> > ```
> 
> Let's rename to decode_array_header and decode_map_header, as agreed
> with Kostja. Other than that looks good to me.

Done. Also updated tarantool-merger-examples repo and graphql PRs.

Removed 'Consider merger's docbot comment for usage examples' from the
commit message, because the patch likely will not land into master at
the same time as merger.

Pushed to Totktonada/gh-3276-on-board-merger.



More information about the Tarantool-patches mailing list