search

boostorg / algorithm

Boost.org algorithm module
http://boost.org/libs/algorithm
Boost Software License 1.0
114 stars 105 forks source link

Remove std::*nary_function #18

Closed kundor closed 7 years ago

kundor commented 8 years ago

You asked me to remind you about PR #17 / commit 3aef0ab so you could merge to master.

Also, in C++17, std::unary_function and std::binary_function will be removed from the standard, so we should also remove their uses.

I brought this up on the mailing list: What to do about std::binary_function and std::unary_function? There was not a lot of response, but the slight preponderance of opinion seemed to be to just remove the inheritance: everything should keep working, but maybe provide a typedef for result_type for use with boost::bind.

I did that. I didn't add the typedef for the structs in detail since they are "internal."

Lastique commented 7 years ago

This seems to be fixed in develop. I think this PR can be closed.

mclow commented 7 years ago

I'll merge to master after the beta.

Lastique commented 7 years ago

When you do, please, have a look at https://github.com/boostorg/algorithm/pull/34, which may make it easier for you. Or you can just close that PR if you don't need it.

mclow commented 7 years ago

This is all done in develop. Closing.