Re: [boost] Proposed interface change to boost::algorithm::copy_while

...

I'm proposing to change the interface to:

template<typename InputIterator, typename OutputIterator, typename Predicate> std::pair<InputIterator, OutputIterator> copy_while ( InputIterator first, InputIterator last, OutputIterator result, Predicate p );

i.e, changing the return type to return both iterators.

I think that certainly a change needs to be made to accomodate the extra information. I wonder how wise it is to return a pair that when the InputIterator type is the same as the OutputIterator type that it could be mistaken for, and accepted by, range algorithms as input? (That's my fault - doh!)

An alternative would be to take the 'first' iterator by reference. Boost.Spirit uses this approach. Regards, Nate