build: собирать emmyluacodestyle с оптимизацией - #3804
Conversation
Форматтер Lua подключался без optimization в default_options, а верхний уровень собирается с buildtype=plain -- то есть подпроект не получал никакого -O вообще и на свежем каталоге тоже, в отличие от fmt и yaml-cpp. Заодно скрипт проверки знает про него и теперь помечает как непорядок. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XUwDWDnYdXrJdvjDVd36QH
|
@vkhanayev, это по твоей части — форматтер Lua. Коротко: Насколько это заметно, зависит от того, как часто его дёргают. Для сравнения, на В PR одна строка в Посмотри, пожалуйста, — если форматтер где-то в горячем пути (например, при сохранении триггера), эффект может оказаться заметным. |
Скрипт из #3803 на боевом каталоге показал:
fmtиyaml-cppв порядке, а форматтер Lua собран без оптимизации — и это, в отличие от моего случая, не следствие старого каталога. У остальных подпроектов вmeson.buildстоитoptimization=2вdefault_options, а уemmyluacodestyle— толькоwarning_level=0. Верхний уровень собирается сbuildtype=plain, meson сам никаких-Oне добавляет, так что подпроект не получал оптимизации никогда, в том числе на свежем каталоге.Что сделано
meson.build—optimization=2вdefault_optionsдляemmyluacodestyle, как уfmt,gtestи остальных;tools/check_build_optimization.py— знает про этот подпроект и помечает его как непорядок (раньше просто печатал «нет -O» без пометки, потому что его не было в списке ожидаемых).Подключается форматтер только при
-Dlua_formatter=true, так что в обычный игровой бинарь он не попадает — но раз собирается, пусть собирается по-человечески.🤖 Generated with Claude Code
https://claude.ai/code/session_01XUwDWDnYdXrJdvjDVd36QH