Skip to content

Remove deprecated std::binary_function - #13

Merged
jzmaddock merged 1 commit into
boostorg:developfrom
pavelkryukov:patch-1
Jun 1, 2021
Merged

Remove deprecated std::binary_function#13
jzmaddock merged 1 commit into
boostorg:developfrom
pavelkryukov:patch-1

Conversation

@pavelkryukov

Copy link
Copy Markdown
Contributor

std::binary_function was removed in C++17 and is no longer available while compiling with MS Visual Studio 2017

@fearsomepirate

Copy link
Copy Markdown

@pavelkryukov Have you tried using boost::functional::binary_function instead?

https://www.boost.org/doc/libs/1_71_0/boost/functional.hpp

@pavelkryukov

Copy link
Copy Markdown
Contributor Author

I believe it should be deprecated as well since C++11 has type deduction and perfect forwarding.

@fearsomepirate

Copy link
Copy Markdown

Right, but this is failing integration checks. The minimum viable fix is probably to just replace std::binary_function with boost::functional::binary_function, and then those checks would probably stop failing.

@pavelkryukov

Copy link
Copy Markdown
Contributor Author

The minimum viable fix is probably to just replace std::binary_function with boost::functional::binary_function,

This is not the way the issue has been fixed in other Boost libraries, for instance:

and then those checks would probably stop failing.

Could you please point to the place where checks fail due to absence of std::binary_function inheritance?

@jzmaddock
jzmaddock merged commit 1348a5f into boostorg:develop Jun 1, 2021
@pavelkryukov
pavelkryukov deleted the patch-1 branch June 1, 2021 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants