Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PVS-Studio check #1291

Open
boris768 opened this issue Dec 6, 2020 · 4 comments
Open

PVS-Studio check #1291

boris768 opened this issue Dec 6, 2020 · 4 comments
Assignees
Labels
Confirmed Reproduced but not fixed yet

Comments

@boris768
Copy link

boris768 commented Dec 6, 2020

PVS-Studio is a tool for detecting bugs and security weaknesses in the source code of programs, written in C, C++, C# and Java. It works under 64-bit systems in Windows, Linux and macOS environments, and can analyze source code intended for 32-bit, 64-bit and embedded ARM platforms.
https://www.viva64.com/en/pvs-studio/

Please, check this log, i think, some code should be reviewed.
fullhtml.zip

@MeridianOXC MeridianOXC added the Confirmed Reproduced but not fixed yet label Aug 9, 2022
@MeridianOXC
Copy link
Member

Many of them are irrelevant, but some definitely deserve a review.

@SupSuper probably best to do via a call with Warboy and Yankes too, there's code from all of us in the report.

@boris768
Copy link
Author

boris768 commented Aug 9, 2022

Many of them are irrelevant, but some definitely deserve a review.

@SupSuper probably best to do via a call with Warboy and Yankes too, there's code from all of us in the report.

i can generate new report, this one is too old may be

@MeridianOXC
Copy link
Member

yes, if it's not too much trouble, I'd appreciate it

@boris768
Copy link
Author

yes, if it's not too much trouble, I'd appreciate it

Code from fb87108, builded on windows, win32 build with deps from Github Actions
xcom_report.zip
also I splitted report to two categories : General and Optimization

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Confirmed Reproduced but not fixed yet
Projects
None yet
Development

No branches or pull requests

3 participants