[Tarantool-patches] [PATCH] Increase limits for debug.traceback

Sergey Voinov sergeiv at tarantool.org
Tue Jan 21 11:08:09 MSK 2020


Hi Igor!

Thanks for your elaborate review!
Please see my answers.


>>  /* Note: changing the following defines breaks the Lua 5.1 ABI. */
>> -#define LUA_INTEGERptrdiff_t
>> -#define LUA_IDSIZE60/* Size of lua_Debug.short_src. */
>> +#define LUA_INTEGER    ptrdiff_t
>> +#define LUA_IDSIZE    128 /* Size of lua_Debug.short_src. */
>> +#define LUA_IDSIZE_ERR  60  /* Size of error message */
>
>I don't get the reason to introduce a separate constant for a single
>line sources. Could you please provide a bit more extended rationale?

The problem here is that if I allow 128 for size of error messages, all error outputs will be longer and there will be need to fix vast amount of test result outputs (.result files). Also, these long outputs will flood tarantool output. I don't think there is a reason to increase error message size. Do you?

Thanks.

-- 
Sergey Voinov


More information about the Tarantool-patches mailing list