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 bugs #232

Closed
jacob1 opened this issue Dec 25, 2014 · 4 comments
Closed

PVS-Studio bugs #232

jacob1 opened this issue Dec 25, 2014 · 4 comments

Comments

@jacob1
Copy link
Member

jacob1 commented Dec 25, 2014

Looks like they used us as an example, mniip found it but didn't ping me, so then savask found it. All these fail bugs and warnings should be fixed sometime: http://www.viva64.com/en/b/0298/ . We shouldn't be famous just for having such buggy code :P.

And speaking of warnings I was going through and fixing many warnings, never finished since it would probably take hours ... but after I do finish we should probably compile with warnings enabled so that they don't pile up again.

@Departing
Copy link
Contributor

Sounds like a plan!

@mniip
Copy link
Member

mniip commented Dec 25, 2014

@jacksonmj
Copy link
Member

A selection of these have been fixed in 236ff08

cppcheck gives a few hundred more things to look at though... https://gist.github.com/jacksonmj/efb4b6caa667b0bbfd3a

@jacob1
Copy link
Member Author

jacob1 commented Feb 14, 2015

All the PVS-Studio bugs are fixed now at least. Not the cppcheck ones though. Not sure what the command to get that output is exactly ... when I ran it I got a ton of warnings (mostly things which aren't likely to cause bugs).

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

No branches or pull requests

4 participants