Skip to content

main.cpp: added -q (quiet mode)#234

Merged
danmar merged 2 commits intocppcheck-opensource:masterfrom
firewave:quiet
Aug 23, 2022
Merged

main.cpp: added -q (quiet mode)#234
danmar merged 2 commits intocppcheck-opensource:masterfrom
firewave:quiet

Conversation

@firewave
Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread main.cpp Outdated
if (std::strncmp(arg, "-std=",5)==0)
dui.std = arg + 5;
break;
case 'q':
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tabs

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

Comment thread main.cpp
std::cout << " -include=FILE Include FILE." << std::endl;
std::cout << " -UNAME Undefine NAME." << std::endl;
std::cout << " -std=STD Specify standard." << std::endl;
std::cout << " -q Quiet mode (no output)." << std::endl;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why would you run it in quiet mode if there is no output?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Profiling (less code being executed) and crash testing.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also if we would change the exitcode based on if the parsing was successful or not it could use used as a command-line tool and in that case you also might not want all the output.

@danmar danmar merged commit 43f1733 into cppcheck-opensource:master Aug 23, 2022
@firewave firewave deleted the quiet branch August 23, 2022 19:24
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

Successfully merging this pull request may close these issues.

2 participants