R
Size: a a a
2020 June 11
AS
lua_strlen там должен быть походу
может быть
AS
вообще имеет смысл сделать отдельные методы, которые работают только с msgpack'ом
R
вообще имеет смысл сделать отдельные методы, которые работают только с msgpack'ом
ну прям типа для msgp совсем отдельные не знаю, а вот для buf точно можно , (https://www.tarantool.io/ru/doc/2.4/reference/reference_lua/buffer/)
строки в lua без нуля в конце, так что строки и msgp это одно и тоже в lua
строки в lua без нуля в конце, так что строки и msgp это одно и тоже в lua
AS
а не может быть такого что бы здесь value_len неправильно считался если встроке лежит результат msgpack.encode ?
выше lua_tostring переводит луашную строку в сишную, поэтому strlen работает как задуманно
R
выше lua_tostring переводит луашную строку в сишную, поэтому strlen работает как задуманно
Делать этого не нужно потому что,
AS
Делать этого не нужно потому что,
ну это делается специально, потому что здесь мы принимаем только строку
тут же проблема в том, что мы не можем передать в payload произвольный тип из луа
тут же проблема в том, что мы не можем передать в payload произвольный тип из луа
AS
кто-нибудь запихает луашную таблицу и хз во что это выльется
AS
или что похуже
R
кто-нибудь запихает луашную таблицу и хз во что это выльется
Есть проврки которые нужно делать в начале функции
AS
Да просто посчитай длинну строки как луа говорит и правильно указатель передай, и всё заработает
проверка на тип всё равно нужна
AS
ibuf эти проверки не пройдёт
R
проверка на тип всё равно нужна
lua_isstring
R
ibuf эти проверки не пройдёт
Для него отдельно, я так и написал
AS
Посути там ветвление может быть, если строка то так, если буфер то по-другому, это же луа си стиль
сделай, пожалуйста, тикет в репозитории