Ping. Marshall, does this look okay?
> On 2016-Jan-25, at 10:32, Duncan P. N. Exon Smith via cfe-commits > <cfe-commits@lists.llvm.org> wrote: > > dexonsmith updated this revision to Diff 45879. > dexonsmith added a comment. > > Addressed Marshall's review comments: deferring to > match_results::operator[]() rather than duplicating the logic. > > > http://reviews.llvm.org/D16467 > > Files: > include/regex > test/std/re/re.results/re.results.form/form1.pass.cpp > > Index: test/std/re/re.results/re.results.form/form1.pass.cpp > =================================================================== > --- test/std/re/re.results/re.results.form/form1.pass.cpp > +++ test/std/re/re.results/re.results.form/form1.pass.cpp > @@ -38,6 +38,31 @@ > { > std::match_results<const char*> m; > const char s[] = "abcdefghijk"; > + assert(std::regex_search(s, m, std::regex("cd((e)fg)hi", > + > std::regex_constants::nosubs))); > + > + char out[100] = {0}; > + const char fmt[] = "prefix: $`, match: $&, suffix: $', m[1]: $1, > m[2]: $2"; > + char* r = m.format(output_iterator<char*>(out), > + fmt, fmt + std::char_traits<char>::length(fmt)).base(); > + assert(r == out + 54); > + assert(std::string(out) == "prefix: ab, match: cdefghi, suffix: jk, > m[1]: , m[2]: "); > + } > + { > + std::match_results<const char*> m; > + const char s[] = "abcdefghijk"; > + assert(std::regex_search(s, m, std::regex("cdefghi"))); > + > + char out[100] = {0}; > + const char fmt[] = "prefix: $`, match: $&, suffix: $', m[1]: $1, > m[2]: $2"; > + char* r = m.format(output_iterator<char*>(out), > + fmt, fmt + std::char_traits<char>::length(fmt)).base(); > + assert(r == out + 54); > + assert(std::string(out) == "prefix: ab, match: cdefghi, suffix: jk, > m[1]: , m[2]: "); > + } > + { > + std::match_results<const char*> m; > + const char s[] = "abcdefghijk"; > assert(std::regex_search(s, m, std::regex("cd((e)fg)hi"))); > > char out[100] = {0}; > @@ -61,6 +86,33 @@ > assert(r == out + 34); > assert(std::string(out) == "match: cdefghi, m[1]: efg, m[2]: e"); > } > + { > + std::match_results<const char*> m; > + const char s[] = "abcdefghijk"; > + assert(std::regex_search(s, m, std::regex("cd((e)fg)hi", > + > std::regex_constants::nosubs))); > + > + char out[100] = {0}; > + const char fmt[] = "match: &, m[1]: \\1, m[2]: \\2"; > + char* r = m.format(output_iterator<char*>(out), > + fmt, fmt + std::char_traits<char>::length(fmt), > + std::regex_constants::format_sed).base(); > + assert(r == out + 30); > + assert(std::string(out) == "match: cdefghi, m[1]: , m[2]: "); > + } > + { > + std::match_results<const char*> m; > + const char s[] = "abcdefghijk"; > + assert(std::regex_search(s, m, std::regex("cdefghi"))); > + > + char out[100] = {0}; > + const char fmt[] = "match: &, m[1]: \\1, m[2]: \\2"; > + char* r = m.format(output_iterator<char*>(out), > + fmt, fmt + std::char_traits<char>::length(fmt), > + std::regex_constants::format_sed).base(); > + assert(r == out + 30); > + assert(std::string(out) == "match: cdefghi, m[1]: , m[2]: "); > + } > > { > std::match_results<const wchar_t*> m; > Index: include/regex > =================================================================== > --- include/regex > +++ include/regex > @@ -5387,8 +5387,8 @@ > if ('0' <= *__fmt_first && *__fmt_first <= '9') > { > size_t __i = *__fmt_first - '0'; > - __out = _VSTD::copy(__matches_[__i].first, > - __matches_[__i].second, __out); > + __out = _VSTD::copy((*this)[__i].first, > + (*this)[__i].second, __out); > } > else > { > @@ -5439,8 +5439,8 @@ > ++__fmt_first; > __i = 10 * __i + *__fmt_first - '0'; > } > - __out = _VSTD::copy(__matches_[__i].first, > - __matches_[__i].second, __out); > + __out = _VSTD::copy((*this)[__i].first, > + (*this)[__i].second, __out); > } > else > { > > > <D16467.45879.patch>_______________________________________________ > cfe-commits mailing list > cfe-commits@lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
