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

Fix issues reported by Coverity Scan #192

Closed
vitaut opened this issue Aug 4, 2015 · 8 comments
Closed

Fix issues reported by Coverity Scan #192

vitaut opened this issue Aug 4, 2015 · 8 comments

Comments

@vitaut
Copy link
Contributor

vitaut commented Aug 4, 2015

Coverity Scan report: https://scan.coverity.com/projects/5671

The issues are not critical and only affect tests. Some of them are in Google Mock, but a bug in Coverity Scan prevents adding the latter as a component and ignoring its issues: https://communities.coverity.com/message/7086#7086.

@vitaut
Copy link
Contributor Author

vitaut commented Aug 5, 2015

Another issue with Coverity Scan: https://communities.coverity.com/thread/3462

@vitaut
Copy link
Contributor Author

vitaut commented Aug 11, 2015

Nothing to do here until Coverity Scan is fixed.

@vitaut vitaut closed this as completed Aug 11, 2015
@vitaut
Copy link
Contributor Author

vitaut commented Aug 31, 2015

Now that Coverity Scan issue is fixed, remaining issues can be addressed.

@vitaut vitaut reopened this Aug 31, 2015
@vitaut
Copy link
Contributor Author

vitaut commented Oct 18, 2015

Two bogus Coverity warnings should be fixed in e81c0ae.

@vitaut
Copy link
Contributor Author

vitaut commented Oct 21, 2015

As of e104ba5 we are down to 3 issues.

@vitaut
Copy link
Contributor Author

vitaut commented Oct 22, 2015

Have to wait till 25 October because of the submission limit.

@vitaut
Copy link
Contributor Author

vitaut commented Oct 22, 2015

And reported a bug in Coverity Scan in https://communities.coverity.com/thread/3484.

@vitaut
Copy link
Contributor Author

vitaut commented Oct 28, 2015

As of 2d727e7 all warnings fixed or suppressed (if false positive).

@vitaut vitaut closed this as completed Oct 28, 2015
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