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.

Sign up for GitHub

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

Jump to bottom

PVS Studio scan (6 high / 22 medium level warnings) #49

Closed
JustAndreww opened this issue Feb 5, 2020 · 1 comment
Closed

PVS Studio scan (6 high / 22 medium level warnings) #49

JustAndreww opened this issue Feb 5, 2020 · 1 comment

Comments

@JustAndreww
Copy link

JustAndreww commented Feb 5, 2020 •
edited

Did a quick scan for the latest master (ccf567a) with PVS Studio:
image

In attachment you can found a full scan log in HTML format. Hope that it's not a lot of false-positive alarms :)

Thanks!
fullhtml.zip
jpcy added a commit that referenced this issue Feb 6, 2020
@jpcy
Cleanup RadixSort. #49
a50033a
jpcy added a commit that referenced this issue Feb 6, 2020
@jpcy
Fix misc. PVS studio warnings. #49
7204faa
jpcy added a commit that referenced this issue Feb 6, 2020
@jpcy
Fix boundary intersection test grid traversal. #49
09b15bb
jpcy added a commit that referenced this issue Feb 6, 2020
@jpcy
Fix some PVS studio performance warnings. #49
4484035
@jpcy
Copy link
Owner

jpcy commented Feb 6, 2020

Thanks. I fixed some of these. Others were intentional - like vector default ctors not initializing data - or too trivial to bother with.

@jpcy jpcy closed this as completed Feb 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jpcy @JustAndreww