status changed; resolution set

Marshall Clow — Wed, 26 Oct 2016 18:28:14 GMT

status new → closed
resolution → wontfix

First, I apologize for the lack of documentation of the changes. That has been corrected, and will appear in the 1.63.0 release, along with some porting notes. You can see those docs in the develop branch now.

As for "why did I make this change", it's because the previous interface was throwing away useful information - to be precise, the end of the matched range. If you know the length of the pattern that you are searching for, and you have random-access iterators, then recovering the end of the matched range is simple and cheap. However, that's not always the case. Consider a matcher that matches against a regular expression.

I wrote a blog post about this titled "sometimes you get things wrong" here: https://cplusplusmusings.wordpress.com/2016/02/01/sometimes-you-get-things-wrong/

antoinep92@… — Wed, 02 Nov 2016 13:59:19 GMT

Just for people looking for the porting notes mentioned by marshall, they're available on github here: https://github.com/boostorg/algorithm/blob/develop/doc/boyer_moore.qbk (markup version -- haven't been able to find compiled html).

(please note that the version on SVN trunk is not up to date)

Boost C++ Libraries: Ticket #12552: Change of the interface of Searching Algorithms breaks client code

status changed; resolution set