[Tarantool-patches] [PATCH 0/2] Improve json error message.

roman roman.habibov at tarantool.org
Tue Nov 3 13:01:19 MSK 2020


Hi!
On 12.10.2020 11:51, Leonid Vasiliev wrote:
> Hi! Thank you for the patch.
> Add @ChangeLog.
>
> On 09.10.2020 00:59, Roman Khabibov wrote:
>> I already have LGTM from Vlad.
>>
>> Branch: 
>> https://github.com/tarantool/tarantool/tree/romanhabibov/gh-4339-json-err
>> Issue: https://github.com/tarantool/tarantool/issues/4339
>>
>> Roman Khabibov (2):
>>    json: make error messages more readable
>>    json: print context in error mesages
>>
>>   test/app-tap/json.test.lua        | 50 +++++++++++++++-
>>   third_party/lua-cjson/lua_cjson.c | 97 +++++++++++++++++++++++--------
>>   2 files changed, 122 insertions(+), 25 deletions(-)
>>
@ChangeLog:
* Print json tokens themselves instead of token names "T_*" in the error 
messages (gh-4339).
* Print context in json error mesages (gh-4339).


More information about the Tarantool-patches mailing list