From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from localhost (localhost [127.0.0.1]) by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTP id 5B4B624F71 for ; Mon, 21 Jan 2019 21:08:47 -0500 (EST) Received: from turing.freelists.org ([127.0.0.1]) by localhost (turing.freelists.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id MUWF_hiBHw2I for ; Mon, 21 Jan 2019 21:08:47 -0500 (EST) Received: from smtpng3.m.smailru.net (smtpng3.m.smailru.net [94.100.177.149]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTPS id AD14F23F22 for ; Mon, 21 Jan 2019 21:08:46 -0500 (EST) Date: Tue, 22 Jan 2019 05:08:45 +0300 From: Alexander Turenko Subject: [tarantool-patches] Re: [PATCH 2/2] lua-yaml: fix yaml encode/decode type inferring Message-ID: <20190122020845.kxzknrjvi5jikhb3@tkn_work_nb> References: <9fab4f4496d3c255ed1de3c7642817d07992a7af.1538663611.git.avkhatskevich@tarantool.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <9fab4f4496d3c255ed1de3c7642817d07992a7af.1538663611.git.avkhatskevich@tarantool.org> Sender: tarantool-patches-bounce@freelists.org Errors-to: tarantool-patches-bounce@freelists.org Reply-To: tarantool-patches@freelists.org List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-Id: tarantool-patches List-subscribe: List-owner: List-post: List-archive: To: Alex Khatskevich Cc: kyukhin@tarantool.org, tarantool-patches@freelists.org Changes I made: * Moved str_type.h to lyaml.cc. * Simplified cases in app-tap/console.test.lua, added more. * Extracted an empty document / value treatment change into a separate commit. * Removed hacking around console.eval() result in test/vinyl/txn_proxy.lua. * Rebased at top of 2.1. * Better describe changes in commit messages. I'll push the patchset to kh/gh-3662-yaml-2.1 and will resend it as v2. WBR, Alexander Turenko. On Thu, Oct 04, 2018 at 05:36:14PM +0300, AKhatskevich wrote: > Changes: > * encode boolean values in single-quoted style > * encode/decode nulls as '', '~', 'null', 'Null', "NULL", according > to the yaml 1.0/1.1/1.2 standard > * fix affected tests > > Related to #3476 > Closes #3662 #3583 > --- > test/app-tap/console.test.lua | 25 +- > test/app/fio.result | 2 +- > test/app/socket.result | 26 +- > test/box/access.result | 8 +- > test/box/misc.result | 2 +- > test/box/role.result | 8 +- > test/box/sequence.result | 2 +- > test/vinyl/errinj.result | 10 +- > test/vinyl/hermitage.result | 122 ++--- > test/vinyl/mvcc.result | 1062 +++++++++++++++++++-------------------- > test/vinyl/tx_conflict.result | 4 +- > test/vinyl/tx_conflict.test.lua | 4 +- > test/vinyl/tx_gap_lock.result | 186 +++---- > test/vinyl/tx_serial.result | 4 +- > test/vinyl/tx_serial.test.lua | 4 +- > test/vinyl/txn_proxy.lua | 6 + > third_party/lua-yaml/lyaml.cc | 43 +- > third_party/lua-yaml/str_type.h | 33 ++ > 18 files changed, 802 insertions(+), 749 deletions(-) > create mode 100644 third_party/lua-yaml/str_type.h