-
Notifications
You must be signed in to change notification settings - Fork 600
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 -Wunused-parameter #13138
Fix -Wunused-parameter #13138
Conversation
Hi! Thank you for contributing! |
0716373
to
ca0a247
Compare
Fixed all warnings, waiting for approval. |
@niksaveliev, could you review this PR or assign someone else? |
0893035
to
bc1d123
Compare
Rebased once more to fix merge conflicts in |
⚪ Test history | Ya make output | Test bloat
⚪ Test history | Ya make output | Test bloat | Test bloat
⚪ Test history | Ya make output | Test bloat | Test bloat | Test bloat
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
⚪ Test history | Ya make output | Test bloat
⚪ Test history | Ya make output | Test bloat | Test bloat
⚪ Test history | Ya make output | Test bloat | Test bloat | Test bloat
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
Changelog category
Additional information
This PR fixes
-Wunused-parameter
warning that is currently silenced. I'm going to restore this warning in the near future (https://nda.ya.ru/t/ysFv8tqj7Ao7Rm)