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.

Sign up for GitHub

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

Jump to bottom

clang-tidy: Ignored function return values in tests #269

Open
billpittman opened this issue Aug 7, 2024 · 1 comment
Open

clang-tidy: Ignored function return values in tests #269

billpittman opened this issue Aug 7, 2024 · 1 comment
Labels
quality Code quality and linter issues

Comments

@billpittman
Copy link
Contributor

/data/ultifi/up-cpp/test/coverage/datamodel/UUriSerializerTest.cpp
/data/ultifi/up-cpp/test/coverage/datamodel/UuidSerializerTest.cpp

Several tests ignore function return values and that cause errors for clang-tidy.
@gregmedd gregmedd added help wanted Extra attention is needed quality Code quality and linter issues and removed help wanted Extra attention is needed labels Aug 9, 2024
@gregmedd
Copy link
Contributor

gregmedd commented Aug 9, 2024

std::ignore = calledFunction() seems to be the recommended way to deal with this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
quality Code quality and linter issues
Projects
C++ uProtocol Development
Status: Backlog
Milestone
No milestone
Development

No branches or pull requests
2 participants
@billpittman @gregmedd