the minmax creates dangling references too easily

[anonymous]

First. minmax's example codes has created dangling references.

boost::tuple<int const&, int const&> result1 = boost::minmax(1, 0); //actual code
boost::tuple<int, int> result1 = boost::minmax(1, 0); //expected code

http://www.boost.org/doc/libs/1_60_0/libs/algorithm/minmax/index.html
http://www.boost.org/doc/libs/1_60_0/libs/algorithm/minmax/example/minmax_ex.cpp

Second. The design of minmax is unsuitable for that of C++11 era. Consider the following.

auto tp = minmax(1,2);
tp.get<0>(); // Undefined behavior

This code looks like fairy normal, but it causes undefined behavior. Since the minmax returns tuple<T const&, T const&>, the 'auto tp' has dangling references.

I think we should change the return type from tuple<T const&, T const&> to tuple<T,T>.

See also: ​https://groups.google.com/a/isocpp.org/forum/#!topic/std-discussion/HLiJJBRNYSw

[Marshall Clow]

This is the same interface that std::min has.

int const &x = std::min(0,1);

[Marshall Clow]

I think we should change the return type from tuple<T const&, T const&> to tuple<T,T>.

That would add the requirement that T be copyable.