================
@@ -78,14 +78,12 @@ def cleanup():
             "u32_string", type="std::u32string_view", summary='U"🍄🍅🍆🍌"'
         )
         self.expect_var_path("u32_empty", type="std::u32string_view", 
summary='U""')
-        self.expect_var_path(
-            "oops", type="std::string_view", summary='"Hellooo World\\n"'
-        )
 
         # GetSummary returns None so can't be checked by expect_var_path, so we
         # use the str representation instead
         null_obj = self.frame().GetValueForVariablePath("null_str")
-        self.assertEqual(null_obj.GetSummary(), "Summary Unavailable")
+        null_summary = null_obj.GetSummary()
+        self.assertTrue(null_summary == "Summary Unavailable" or null_summary 
is None)
----------------
Michael137 wrote:

I prefer being consistent with the other formatters and printing "Summary 
Unavailable". Not printing anything might get confused for "string is empty".

https://github.com/llvm/llvm-project/pull/150318
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to