cppcheck 2.6

https://imgur.com/a/TDYCWTC

Ага. Благодарю!

Ну, если это не нравится, то можно убрать варнинг, отключив `useStlAlgorithm` в параметрах cppcheck-а.
Или можно прямо в коде перед той строкой вписать:

// cppcheck-suppress useStlAlgorithm

Но вообще-то...
Совет cppcheck-а не бесполезен.
Он заставляет задуматься о будущих оптимизациях.

Возьмём пример:

std::vector<int> lor( const std::vector<int> & v )
{
   std::vector<int> result;

   for (auto item : v)
   {
      result.push_back( item / 2 );
   }

   return result;
}

Сначала, как предложил `cppcheck`, перепишем его в std::transform:

std::vector<int> lor( const std::vector<int> & v )
{
   std::vector<int> result;

   std::transform(
       v.begin(),
       v.end(),
       std::back_inserter(result),
       [](int x){return x/2;} );

   return result;
}

Можно:

std::vector<int> lor( const std::vector<int> & v )
{
   std::vector<int> result( v.size() );  // <--

   std::transform(
       v.begin(),
       v.end(),
       result.begin(),                   // <--
       [](int x){return x/2;} );

   return result;
}

Это мелкое изменение избавило нас от перевыделения памяти, и сделало код быстрее.

Но и это ещё не всё!

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

std::vector<int> lor( const std::vector<int> & v )
{
   std::vector<int> result( v.size() );

   std::transform(
       std::execution::par_unseq,        // <--
       v.begin(),
       v.end(),
       result.begin(),
       [](int x){return x/2;} );

   return result;
}

Такой код современные компиляторы полностью инлайнят.
Это позволяет им использовать всякие MMX/SSE/AVX/etc для параллельных вычислений.
Из внешних вызовов остаётся только new и memset.

А при обычных циклах об этих оптимизациях даже не задумываешься.

Cppcheck - неинтересно обсуждать как MISRA на регулярках делается, хотя, при случае, может и напишем статью. Тяжеловесы интереснее :) Выявляем ошибки в релизе LLVM 13.0.0.

Пулл-реквесты бы лучше оформляли, чем вести спамоблоги с рекламой. Из-за передоза рекламы уважающие свое время люди, которые не любят тратить свое время на рекламу, ненавидят PVS. Главным образом потому, что реклама тут маскируется под приличные статьи и пробивается через фильтры. Cppcheck интересно обсуждать, потому что он открытый и бесплатный, у него нет миллиона вариантов лицензирования, нет фонтана пиара. Ей-богу, вместо рекламоблогов писали бы пулреквесты в открытые проекты, закрывали бы критические баги, деньги бы получали с каких-нибудь bug bounty - уважаемые были бы люди, и к PVS, как к вашей разработке, был бы интерес. Еще бы с офсайта убрать свистелки, счетчики, анимации и плашку про куки (вообще не используйте куки и тогда вроде таких ужасных плашек не надо). Чтобы сайт выглядел не как одна большая реклама, которую хочется закрыть, а как приложение для телефона или там статья какая. Цены тоже хотя бы примерные указать, обычную кнопку покупки, чтобы можно было в пару кликов купить, а не ждать неделю, пока пришлют цены и пока происходит человеческий диалог о покупке (вместо пары кликов на сайте!)

PVS-Studio является B2B решением и запрос цены является естественным процессом. Для индивидуального использования существует ряд бесплатных лицензий: https://pvs-studio.com/ru/blog/posts/0614/