All streams
Search
Write a publication
Pull to refresh

Comments 14

По четвертой ошибке есть же простой совет — enum class. И там уже даже компилятор не позволит явно присвоить.
>>Предупреждение PVS-Studio: V612 An unconditional 'break' within a loop. BlackJack_Client140 messagetypes.h 213

А оператор в цикле не может иметь побочного эффекта, меняющего EIP?
> Как видите, случайно два раза инициализировали член low, но при этом забыли инициализировать high. Глубокомысленный комментарий здесь написать сложно.

Чего ж сложного — Вы ж сами писали про эффект последней строки.
image
Однако, объект по-прежнему может быть скопирован с помощью конструктора копирования, создаваемого компилятором по умолчанию.

Как исправить эту ошибку?
private:
_acquire_protector& operator=(const _acquire_protector&);
_acquire_protector(const _acquire_protector&); (или = delete)
Я правильно понимаю, что все ошибки найденные в статье, были в директории samples (т.е. примеры)?
enum BJHandState — вообще странно, что там значения имеют префикс HR_. Ну и да, сам проект вроде как использует C++11, а тут на enum class поскупились.

В общем, чтобы не давать хлеба разработчикам PVS-Studio, уважаемые разработчики библиотек, тщательнее вылизывайте примеры! :D
Я правильно понимаю, что все ошибки найденные в статье, были в директории samples (т.е. примеры)?
Кажется, да. Проект небольшой, поэтому искали где могли… :)
Я правильно понимаю, что все ошибки найденные в статье, были в директории samples (т.е. примеры)?
Так это ж ещё хуже, когда ошибки в «смотрите, дети, как надо писАть»!
Вне зависимости от ударения в слове «писать» это плохо )
Благодаря статье узнал что сказка про паровозик имеет не только версию, озвученную в «Майоре Пэйне».
Это уже к любителям пони.
Sign up to leave a comment.

Information

Website
pvs-studio.ru
Registered
Founded
2008
Employees
51–100 employees
Location
Россия
Representative
Андрей Карпов