search

ukoethe / vigra

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

c++17 support? #481

Closed k-dominik closed 2 years ago

k-dominik commented 3 years ago

The clang folks have implemented c++17 a bit more aggressively than gcc: some members that were deprecated in c++17 are removed, like std::bind1st, std::bind2nd as used in e.g. vigra/include/vigra/numpy_array_traits.hxx.

gcc seems to be keeping those for the time being.

For projects that link against vigra this poses a problem: on osx those projects cannot be compiled using c++17. Are there any plans to add support for this? Or, are there plans to support standards older than c++11? Otherwise one can use lambdas instead...

BSeppke commented 3 years ago

Would be interesting for me as well. MS also dropped these functions in their C++ compiler when switching to the C++17 langange standard.

constantinpape commented 3 years ago

I ran into this again recently. Would be great to enable c++17 builds with vigra as dependency. @ukoethe @hmeine I would not mind making these changes; is there any interest from your side in having this? Also, if I do this I would need a new vigra release to get this merged into conda forge to use the changes in other conda builds.