On 10/04/2013 06:04 PM, Paolo Carlini wrote:
In particular, the new *_testsuite functions, isn't immediately clear
in what they differ from the non-_testsuite variants. In any case we
should figure out a better name, maybe even *_internal, if we can't
find anything more accurate, but it should be self-explaining in terms
of their C++ semantics.
Thus, the functions are comparing two different executors. Thus say that
in the name, like: __regex_search_debug, or __regex_search_compare. And
please say what they are for in a comment before at least the "primary"
overload, the one actually carrying out the comparison.
All the other comments live.
Paolo.
