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 issues #45

Closed
375gnu opened this issue May 5, 2018 · 5 comments
Closed

Coverity issues #45

375gnu opened this issue May 5, 2018 · 5 comments
Assignees

Comments

@375gnu
Copy link
Member

375gnu commented May 5, 2018

Hi!

I've scanned the latest Celestia code using Coverity tool and it has found 182 issues.

Scan results can be found here

I'll try to fix them.

@Alexell
Copy link
Contributor

Alexell commented May 7, 2018

@375gnu Coverity does not allow viewing on your link (at least without authorization). But I'm aware of the presence of unfinished places in the code. I would be grateful if you could help me finish them.

@Alexell
Copy link
Contributor

Alexell commented May 7, 2018

On our forum there are a lot of messages about Celestia QT bugs, but I do not have enough time to go to the forum yet.

@375gnu
Copy link
Member Author

375gnu commented May 7, 2018

Indeed, it requires authentication. I've sent you an invitation. https://scan.coverity.com/projects/celestia.

I'm not familiar with QT, 3D and astronomic programming, but I like Celestia so I just do what I can.

@CelestiaProject
Copy link
Collaborator

@375gnu Please check the successful Celestia compilation and correct work after your changes before sending pull requests. Thank you.

@375gnu 375gnu changed the title Coverity issue Coverity issues Aug 11, 2018
@375gnu
Copy link
Member Author

375gnu commented Nov 11, 2018

fixed in master

@375gnu 375gnu closed this as completed Nov 11, 2018
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

2 participants