Chore: Fix unused variable warnings

[HunterBelanger]

I found a few places in the code where function arguments were not being used, and gcc/clang was giving warnings regarding unused variables. I have commented out the variable names here to silence these warnings. I don't think these are bugs, but I definitely don't know enough about the project to be sure, so you should definitely check to see if this is really the case.

[brmarkus]

What about using [[maybe_unused]] (from C++11)?
See https://en.cppreference.com/w/cpp/language/attributes.html for details.

[HunterBelanger]

The [[maybe_unused]] attribute actually requires C++17 or greater, which I think probably exceeds the projects targeted standard?

[brmarkus]

The [[maybe_unused]] attribute actually requires C++17 or greater, which I think probably exceeds the projects targeted standard?

Yeah, you are right: https://en.cppreference.com/w/cpp/language/attributes/maybe_unused.html
Even "CONTRIBUTING.md" mentions

Only use C++ features available in C++11.

Sorry for the confusion.

[brenocq]

Thanks for sharing this warning @HunterBelanger! I've removed the unused arguments and added the flag -Wextra to the workflows to make sure these warnings are caught.