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

Unnecessary check subcommand #2145

Open
bruntib opened this issue May 23, 2019 · 3 comments
Open

Unnecessary check subcommand #2145

bruntib opened this issue May 23, 2019 · 3 comments

Comments

@bruntib
Copy link
Contributor

bruntib commented May 23, 2019

Why do we need CodeChecker check subcommand? The only remarkable difference is the location of the output. Couldn't we eliminate CodeChecker check? Instead we could print the results to the standard output in case there is no -o flag for CodeChecker analyze.

@gyorb
Copy link
Contributor

gyorb commented Jun 25, 2019

The check command is kept for backward compatibility and it adds easier usage to analyze some small files.
It calls log/analyze/parse through one command. So you do not have to call all three commands separately.
If we would change the output behavior for analyze you still need to call log before that.

@bruntib
Copy link
Contributor Author

bruntib commented Jun 25, 2019

Unless you give the build command with -b to analyze like in case of check.

@vodorok
Copy link
Collaborator

vodorok commented Jul 26, 2019

CodeCheckerEclipsePlugin was changed, to use analyze a while back.

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

No branches or pull requests

3 participants