Йо-хо-хо, и бутылка рому! Разбираем ошибки игрового движка Storm Engine / Comments / Habr

pda0 Jun 25 2021 at 23:18

PVS-Studio: V724 Испааанцы!!!

ertaquo Jun 26 2021 at 19:54

А можно где-то посмотреть полный отчет, на предмет исправления этих багов?

SvyatoslavMC Jun 29 2021 at 19:58

Оно немножко не так работает, как вы написали. Вот Issue в их репозитории. А там как пойдёт, зависит от разработчиков. Иногда они проводят активную работу в рамках тикета, прикрепляя коммиты, их интересно смотреть. Иногда молча закрывают. Для крупных проектов нормой (печальной) является исправление бага из статьи года через 2.

wslc Jun 30 2021 at 16:15

Про производительность deque очень сомнительный совет: источников света вряд ли очень много, а движущихся (aMovingLight) и того меньше. Если размер бакета deque большой, то ничего не выиграется, а если маленький (как в MSVC — 16 байт), то получится много аллокаций — сомнительно, что это лучше компактного вектора. Опять же остается поиск каждого aMovingLight — с полным перепроходом всего списка источников. В целом, полезность всего этого кода сомнительная с учетом дальнейшего отбора по расстоянию.
Рекомендация про deque создает ложную иллюзию исправления. Лучше, наверное, предлагать пересмотреть весь цикл с вставкой в начало, чем пытаться угадать решение.