F
эта штука позволит не зависеть от си компилятора проектам с си зависимостями, что супер круто!
Size: a a a
2020 January 07
DM
Звучит хорошо, ждём shaderc
F
миниквад завёлся на этой штуке!
F
линукс версия работает без си компилятора!1
F
λ
эта штука позволит не зависеть от си компилятора проектам с си зависимостями, что супер круто!
Фантастика. Не позволит. Будешь ещё полгода переписывать после )
F
прямо сейчас позволило ;)
F
большая сишная зависимость превратилась в обычный кусок раст кода, но всё продолжило работать как и было.
По сути - поменялся вызов bindgen на вызов c2rust да и всё
По сути - поменялся вызов bindgen на вызов c2rust да и всё
BD
Я пробовал с2раст, там было все плохо с документацией и примерами. И код по дефолту генерит чудовищный.
BD
Блин, раст нашел им реальную ошибку в памяти и они пофиксили c2rust чтобы он не находил. Srsly?
MD
Блин, раст нашел им реальную ошибку в памяти и они пофиксили c2rust чтобы он не находил. Srsly?
Статью внимательно читай. В си так можно, в расте нельзя
BD
Memory Doctor
Статью внимательно читай. В си так можно, в расте нельзя
Сам внимательно читай. Там была настоящая ошибка в сишном коде
MD
Я из-за твоего коммента и прочитал. Ты про память после последнего элемента массива?
BD
Memory Doctor
Я из-за твоего коммента и прочитал. Ты про память после последнего элемента массива?
The Rust compiler also flagged a similar but actually buggy example in G_TryPushingEntity where the conditional is
>, not >=. The out of bounds pointer was then dereferenced after the conditional, which is an actual memory safety bug.
BD
Очевидно что если нельзя >= &array[1024] это заменяется на эквивалентное > &array[1023] в арифметике указателей. Это лучше, чем скрывать такие ошибки.
BD
Судя по их тексту они все пофиксили так, что ошибочный код транспилится теперь без предупреждений. С телефона тяжело читать, но ща проверю хистори кода.
BD
Да, в хистори этот кусок кода так и остался. Теперь можно словить off-by-one memory overwrite. Круто!
ДС
hrtf
BD
Дмитрий Степанов
hrtf
Я пока в отпуске без компа)