[Lldb-commits] [lldb] [lldb][test] Fix running TestWithLimitDebugInfo.py on Windows (PR #150579)

[Pavel Labath via lldb-commits]

================
@@ -1775,16 +1775,20 @@ def no_reason(_):
                 attrvalue, "__no_debug_info_test__", False
             ):
                 # If any debug info categories were explicitly tagged, assume that list to be
-                # authoritative.  If none were specified, try with all debug
-                # info formats.
-                all_dbginfo_categories = set(
+                # authoritative.  If none were specified, try with all debug info formats.
+                test_method_categories = getattr(attrvalue, "categories", [])
+                dbginfo_categories = set(test_method_categories) & set(
----------------
labath wrote:

We should keep the type consistent, but you can do that by converting the result to a list with `list(xxx)` at an appropriate time. The thing that was bothering me was the combination of list- and set-based operations, which made it hard to follow.

Changing the property type to always be a set might also be good idea, but I'd do that separately.

https://github.com/llvm/llvm-project/pull/150579