Revert "Generalize regex matching std::string variants to compensate for recent"
authorRaphael Isemann <teemperor@gmail.com>
Thu, 12 Nov 2020 13:01:09 +0000 (14:01 +0100)
committerRaphael Isemann <teemperor@gmail.com>
Thu, 12 Nov 2020 13:01:22 +0000 (14:01 +0100)
This reverts commit 856fd98a176240470dcc2b8ad54b5c17ef6a75b3. The type formatters
use inline namespaces to find the formatter that fits the type ABI, so they
can't just ignore the inline namespaces.

The failing tests should be fixed by da121fff1184267a405f81a87f7314df2d474e1c .

lldb/source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp

index 7bc5b19..83a140e 100644 (file)
@@ -461,52 +461,52 @@ static void LoadLibCxxFormatters(lldb::TypeCategoryImplSP cpp_category_sp) {
   AddCXXSummary(cpp_category_sp,
                 lldb_private::formatters::LibcxxStringSummaryProviderASCII,
                 "std::string summary provider",
-                ConstString("^std::(__[[:alnum:]]+::)?string$"), stl_summary_flags,
+                ConstString("^std::__[[:alnum:]]+::string$"), stl_summary_flags,
                 true);
   AddCXXSummary(cpp_category_sp,
                 lldb_private::formatters::LibcxxStringSummaryProviderASCII,
                 "std::string summary provider",
-                ConstString("^std::(__[[:alnum:]]+::)?basic_string<char(, "
-                            "std::(__[[:alnum:]]+::)?char_traits<char>, "
-                            "std::(__[[:alnum:]]+::)?allocator<char> )?>$"),
+                ConstString("^std::__[[:alnum:]]+::basic_string<char, "
+                            "std::__[[:alnum:]]+::char_traits<char>, "
+                            "std::__[[:alnum:]]+::allocator<char> >$"),
                 stl_summary_flags, true);
   AddCXXSummary(cpp_category_sp,
                 lldb_private::formatters::LibcxxStringSummaryProviderASCII,
                 "std::string summary provider",
-                ConstString("^std::(__[[:alnum:]]+::)?basic_string<unsigned char(, "
-                            "std::(__[[:alnum:]]+::)?char_traits<unsigned char>, "
-                            "std::(__[[:alnum:]]+::)?allocator<unsigned char> )?>$"),
+                ConstString("^std::__[[:alnum:]]+::basic_string<unsigned char, "
+                            "std::__[[:alnum:]]+::char_traits<unsigned char>, "
+                            "std::__[[:alnum:]]+::allocator<unsigned char> >$"),
                 stl_summary_flags, true);
 
   AddCXXSummary(cpp_category_sp,
                 lldb_private::formatters::LibcxxStringSummaryProviderUTF16,
                 "std::u16string summary provider",
                 ConstString(
-                    "^std::(__[[:alnum:]]+::)?basic_string<char16_t(, "
-                    "std::(__[[:alnum:]]+::)?char_traits<char16_t>, "
-                    "std::(__[[:alnum:]]+::)?allocator<char16_t> )?>$"),
+                    "^std::__[[:alnum:]]+::basic_string<char16_t, "
+                    "std::__[[:alnum:]]+::char_traits<char16_t>, "
+                    "std::__[[:alnum:]]+::allocator<char16_t> >$"),
                 stl_summary_flags, true);
 
   AddCXXSummary(cpp_category_sp,
                 lldb_private::formatters::LibcxxStringSummaryProviderUTF32,
                 "std::u32string summary provider",
                 ConstString(
-                    "^std::(__[[:alnum:]]+::)?basic_string<char32_t(, "
-                    "std::(__[[:alnum:]]+::)?char_traits<char32_t>, "
-                    "std::(__[[:alnum:]]+::)?allocator<char32_t> )?>$"),
+                    "^std::__[[:alnum:]]+::basic_string<char32_t, "
+                    "std::__[[:alnum:]]+::char_traits<char32_t>, "
+                    "std::__[[:alnum:]]+::allocator<char32_t> >$"),
                 stl_summary_flags, true);
 
   AddCXXSummary(cpp_category_sp,
                 lldb_private::formatters::LibcxxWStringSummaryProvider,
                 "std::wstring summary provider",
-                ConstString("^std::(__[[:alnum:]]+::)?wstring$"),
+                ConstString("^std::__[[:alnum:]]+::wstring$"),
                 stl_summary_flags, true);
   AddCXXSummary(cpp_category_sp,
                 lldb_private::formatters::LibcxxWStringSummaryProvider,
                 "std::wstring summary provider",
-                ConstString("^std::(__[[:alnum:]]+::)?basic_string<wchar_t(, "
-                            "std::(__[[:alnum:]]+::)?char_traits<wchar_t>, "
-                            "std::(__[[:alnum:]]+::)?allocator<wchar_t> )?>$"),
+                ConstString("^std::__[[:alnum:]]+::basic_string<wchar_t, "
+                            "std::__[[:alnum:]]+::char_traits<wchar_t>, "
+                            "std::__[[:alnum:]]+::allocator<wchar_t> >$"),
                 stl_summary_flags, true);
 
   SyntheticChildren::Flags stl_synth_flags;
@@ -788,10 +788,6 @@ static void LoadLibStdcppFormatters(lldb::TypeCategoryImplSP cpp_category_sp) {
                   "std::allocator<char> >"),
       cxx11_string_summary_sp);
   cpp_category_sp->GetTypeSummariesContainer()->Add(
-      ConstString("std::basic_string<unsigned char, std::char_traits<unsigned char>, "
-                  "std::allocator<unsigned char> >"),
-      cxx11_string_summary_sp);
-  cpp_category_sp->GetTypeSummariesContainer()->Add(
       ConstString("std::__cxx11::basic_string<unsigned char, std::char_traits<unsigned char>, "
                   "std::allocator<unsigned char> >"),
       cxx11_string_summary_sp);