From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Tue, 30 Apr 2019 16:03:17 +0300 From: Vladimir Davydov Subject: Re: [PATCH v3 4/7] lua: add non-recursive msgpack decoding functions Message-ID: <20190430130317.bgekx24eldpr45id@esperanza> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: To: Alexander Turenko Cc: tarantool-patches@freelists.org List-ID: 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.