Boost C++ Libraries: Ticket #7237: Make sub_match a proper range

status changed

Eric Niebler — Wed, 29 Aug 2012 02:23:59 GMT

status new → assigned

xpressive's sub_match is closely modeled on std::sub_match, which doesn't have the members you refer to. But xpressive doesn't slavishly follow the standard, and what you suggest is reasonable. I would prefer to non-intrusively make sub_match a valid range by hooking Boost.Range's customization points.

But would sub_match's const_iterator be a typedef for iterator, or would it const-ify the iterator (if necessary)? And do I need rbegin/rend, cbegin/cend, crbegin/crend and the associated typedefs, too? Sheesh! Opinions?

zeratul976@… — Wed, 29 Aug 2012 03:20:26 GMT

xpressive's sub_match is closely modeled on std::sub_match, which doesn't have the members you refer to

Right, because ranges haven't been fully standardized. I expect that when they are, care will be taken to make things like std::sub_match be a Range.

I would prefer to non-intrusively make sub_match a valid range by hooking Boost.Range's customization points.

That's reasonable.

would sub_match's const_iterator be a typedef for iterator, or would it const-ify the iterator (if necessary)?

const_iterator should be the type returned when calling begin() on a 'const sub_match', so the answer depends on whether you think it should be allowed to modify the string through a 'const sub_match'. I have no strong feelings either way.

And do I need rbegin/rend, cbegin/cend, crbegin/crend and the associated typedefs, too?

No. Those members are required by the standard Container concept, which is a (large) superset of the Range concept. I am not proposing that sub_match model Container.

Thanks! Nate

Eric Niebler — Wed, 29 Aug 2012 18:01:05 GMT

Fixed on trunk in [80300]. I've taken the boost::iterator_range and std::pair route and kept const_iterator the same as iterator. Thanks!

status changed; resolution set

Eric Niebler — Thu, 04 Oct 2012 18:23:22 GMT

status assigned → closed
resolution → fixed

Fixed on release in [80581]. Will be part of Boost 1.52.

Michel Morin — Wed, 06 Feb 2013 09:47:28 GMT

begin/end member functions might be better, since they also support C++11 range-based for.