Один день из жизни разработчика PVS-Studio, или как я отлаживал диагностику, оказавшуюся внимательнее трёх программистов

Спасибо за замечание по поводу сравнения, поправил в документации. По поводу отсутствия 0-символа, мне кажется, это уже совсем частный случай)

Один день из жизни разработчика PVS-Studio, или как я отлаживал диагностику, оказавшуюся внимательнее трёх программистов

Да, если написать условие, то мы перестанем выдавать ложное срабатывание.

После обнаружения данного срабатывания мы добавили его, как пример, в документацию V645 и там предлагаем такой вариант фикса:

size_t textSize = strlen(a.consoleText);
if (sizeof(a.consoleText) - textSize > 5)
{
     strncat(a.consoleText, inputBuffer, 
              sizeof(a.consoleText) - textSize - 5);
}

Один день из жизни разработчика PVS-Studio, или как я отлаживал диагностику, оказавшуюся внимательнее трёх программистов

Спасибо за фидбек! Учту при написании следующих статей.

Free Heroes of might and magic 2 – open-source проект, в котором хочется участвовать

Спасибо, учту

Free Heroes of might and magic 2 – open-source проект, в котором хочется участвовать

Также, если не хочется создавать строчку через итераторы, как я в примере, то можно позвать ещё одну перегрузку и написать, так:

list.emplace_back(str, pos1, pos2 - pos1);

Free Heroes of might and magic 2 – open-source проект, в котором хочется участвовать

Спасибо за замечание, вы действительно правы, просто замена push_back на emplace_back не даст выигрыша в производительности и по моему описанию предупреждения было не особо понятно что я имел в виду. Более подробно расписал пример N1.

Free Heroes of might and magic 2 – open-source проект, в котором хочется участвовать

Здравствуйте, с точки зрения человека, читающего код — это безусловно полезная практика, как минимум может защитить от двойного освобождения памяти (выстрел в ногу, защита). Но по факту такое обнуление компилятор, скорее всего, оптимизирует. А, вообще, я бы порекомендовал пользоваться умными указателми с явной семантикой владения и выделения памяти.

Топ 10 ошибок в C++ проектах за 2020 год

В среднесрочной перспективе поддержка плагина не планируется, но сам способ запуска анализатора таким образом можем исследовать и написать по итогу статью-гайд. Там, скорее всего, достаточно будет позвать консольную версию анализатора по аналогии с GitLab, Azure, TeamCity и т.д.

Топ 10 ошибок в C++ проектах за 2020 год

Спасибо за замечание, поправил.

Топ 10 ошибок в C++ проектах за 2020 год

Вы имеете в виду интеграцию с GitHub Actions?