boostorg / range

Boost.org range module
http://boost.org/libs/range
43 stars 104 forks source link

Resolve cyclic dependency between range and algorithm #93

Closed jeking3 closed 5 years ago

jeking3 commented 5 years ago

algorithm depends on range quite a bit, however range also depends on algorithm in:

include/boost/range/algorithm/min_element.hpp:#include <boost/algorithm/minmax_element.hpp>
include/boost/range/algorithm/max_element.hpp:#include <boost/algorithm/minmax_element.hpp>

This cyclic dependency should be resolved. Perhaps minmax_element.hpp should be moved from algorithm to range? Nothing else in algorithm includes it. This would break the cyclic dependency between these modules.

cc: @mclow

jeking3 commented 5 years ago

Looks like this was resolved by 4ec46acdf585d421770228950a271473345a54a0