[libcxx] r299652 - Restore Missing awk regex tests. Thanks to dexonsmith for noticing, and proposing this as https://reviews.llvm.org/D16541

Mehdi Amini via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 6 09:11:34 PDT 2017


Hi,

I reverted in r299656. MacOS bot is broken both on 64 and 32 bits:

http://green.lab.llvm.org/green/job/libcxx_master_cmake/83/
http://green.lab.llvm.org/green/job/libcxx_master_cmake_32/61/

It shouldn’t be too hard to fix though, it is full of the same error:

/Users/buildslave/jenkins/sharedspace/libcxx/libcxx.src/test/std/re/re.alg/re.alg.match/awk.pass.cpp:266:28: error: comparison of integers of different signs: 'difference_type' (aka 'long') and 'size_t' (aka 'unsigned long') [-Werror,-Wsign-compare]

        assert(m.length(0) == std::char_traits<char>::length(s));
               ~~~~~~~~~~~ ^  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~


— 
Mehdi






> On Apr 6, 2017, at 7:32 AM, Marshall Clow via cfe-commits <cfe-commits at lists.llvm.org> wrote:
> 
> Author: marshall
> Date: Thu Apr  6 09:32:42 2017
> New Revision: 299652
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=299652&view=rev
> Log:
> Restore Missing awk regex tests. Thanks to dexonsmith for noticing, and proposing this as https://reviews.llvm.org/D16541
> 
> Modified:
>    libcxx/trunk/test/std/re/re.alg/re.alg.match/awk.pass.cpp
> 
> Modified: libcxx/trunk/test/std/re/re.alg/re.alg.match/awk.pass.cpp
> URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/re/re.alg/re.alg.match/awk.pass.cpp?rev=299652&r1=299651&r2=299652&view=diff
> ==============================================================================
> --- libcxx/trunk/test/std/re/re.alg/re.alg.match/awk.pass.cpp (original)
> +++ libcxx/trunk/test/std/re/re.alg/re.alg.match/awk.pass.cpp Thu Apr  6 09:32:42 2017
> @@ -26,7 +26,7 @@
> 
> int main()
> {
> -/*    {
> +    {
>         std::cmatch m;
>         const char s[] = "a";
>         assert(std::regex_match(s, m, std::regex("a", std::regex_constants::awk)));
> @@ -616,13 +616,12 @@ int main()
>         assert(m.size() == 0);
>     }
>     std::locale::global(std::locale(LOCALE_cs_CZ_ISO8859_2));
> -*/    {
> -    /*
> +    {
>         std::cmatch m;
>         const char s[] = "m";
> -        assert(std::regex_match(s, m, std::regex("[a[=M=]z]",
> -                                                 std::regex_constants::awk);
> -       assert(m.size() == 1);
> +        assert(std::regex_match(s, m, 
> +                      std::regex("[a[=M=]z]", std::regex_constants::awk)));
> +        assert(m.size() == 1);
>         assert(!m.prefix().matched);
>         assert(m.prefix().first == s);
>         assert(m.prefix().second == m[0].first);
> @@ -632,8 +631,8 @@ int main()
>         assert(m.length(0) == std::char_traits<char>::length(s));
>         assert(m.position(0) == 0);
>         assert(m.str(0) == s);
> -*/    }
> -/*    {
> +    }
> +    {
>         std::cmatch m;
>         const char s[] = "Ch";
>         assert(std::regex_match(s, m, std::regex("[a[.ch.]z]",
> @@ -1389,4 +1388,4 @@ int main()
>         assert(m.position(0) == 0);
>         assert(m.str(0) == s);
>     }
> -*/}
> +}
> 
> 
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170406/87e059c7/attachment.html>


More information about the cfe-commits mailing list