ukoethe / vigra

a generic C++ library for image analysis
http://ukoethe.github.io/vigra/
Other
410 stars 191 forks source link

Vigra pollutes builds with pragmas that suppress warnings #482

Closed samkhal closed 1 year ago

samkhal commented 3 years ago

config.hxx currently uses compiler-specific warning suppression pragmas, such as here. This pollutes the build of anyone including this file who relies on the suppressed warnings, particularly for anyone also using -Werror, as this introduces compile errors that are dependent on where vigra is first included.

If suppressing these warnings from within vigra is desired, push and pop pragmas should be used to apply and reset these suppressions when entering and leaving vigra code.

hmaarrfk commented 1 year ago

We finally got our CIs running again. if you want to make a PR to address this, I would be happy to review it.