SuperElastix / elastix

Official elastix repository
http://elastix.dev
Apache License 2.0
449 stars 110 forks source link

Enable and address `-Wunused-parameter` warnings from both GCC and clang #1136

Closed N-Dekker closed 3 weeks ago

N-Dekker commented 3 weeks ago

Suggested by Bradley Lowekamp (@blowekamp) at https://github.com/SimpleITK/SimpleITK/pull/2104#issuecomment-2075743756, as part of "Update elastix to April 19 2024 hash"

The warnings are addressed, either by removing those parameter names (if they were not really informative), by using the itkNotUsed(x) macro (in RecursiveBSplineTransformImplementation), or by actually using the specific parameter (specifically in GTestUtilities::InitializeMetric).

Follow-up to:

Note: some of those warnings were addressed already by separate pull requests: