[PATCH] D16467: [libcxx] re.results.form: Format out-of-range subexpression references as null

[Duncan P. N. Exon Smith via cfe-commits]

Ping.

Marshall, does this look okay?

> On 2016-Jan-25, at 10:32, Duncan P. N. Exon Smith via cfe-commits <cfe-commits at 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 at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits