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

Coverity Bugs and Errors #8

Open
SAnsell opened this issue Feb 17, 2014 · 0 comments
Open

Coverity Bugs and Errors #8

SAnsell opened this issue Feb 17, 2014 · 0 comments
Assignees

Comments

@SAnsell
Copy link
Owner

SAnsell commented Feb 17, 2014

Coverity that produce what I think of as an excellent static analysis tool, have scanned the project. That has produced a number of warnings errors. The majority are warnings about constructors not initializing values, however, I am ignoring all those because they are initialized in populate and you can't fail to call populate as it is called in very call to createAll().

Issues that do count:

There is an error with makeCNF -- definitely my fault -- I delete memory and then assume it will be ok. But it is complex and I am going to need a bit of time to figure out a fix. [I need to write some tests first to make sure I don't break things!] I believe it has the possibility to corrupt output.

Then there have been a number of other minor issues that I have been fixing along the way, e.g. incorrect copy constructors. Since much of the code doesn't really rely on them because of the createAll system, I am not sure you will see an impact on output -- but it is better to fix them than not.

Anyway after my first pass I am going to see if I can resubmit and have another go.

If anyone wants to help.....

@SAnsell SAnsell self-assigned this Feb 17, 2014
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

1 participant