acts-project / algebra-plugins

Mozilla Public License 2.0
3 stars 10 forks source link

Replaced casts to `void` with the `[[maybe_unused]]` attribute. #94

Closed wermos closed 1 year ago

wermos commented 1 year ago

Casts to void (to suppress unused variable warnings) look like black boxes to unsuspecting readers of the code. [[maybe_unused]] is a much better way to convey intent.

Moreover, since we are using C++17, there is no reason not to use [[maybe_unused]] as it available in any C++17-standard compliant compiler.