Skip to content

Commit 84fd561

Browse files
authored
Rollup merge of #155738 - Walnut356:tuple_fields, r=jieyouxu
Pass fields to `is_tuple_fields` instead of `SBValue` object straightforward fix for a logic error. `is_tuple_fields` expects a `list`, so we pass that in instead of the value object. Coincidentally, this also fixes one of the 3 DI tests that fails on `x86_64-pc-windows-gnu` (`tests/debuginfo/union-smoke.rs`)
2 parents 0c27f91 + 180bfef commit 84fd561

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

src/etc/lldb_providers.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -268,7 +268,7 @@ def aggregate_field_summary(valobj: SBValue, _dict) -> Generator[str, None, None
268268
if summary is None:
269269
summary = child.value
270270
if summary is None:
271-
if is_tuple_fields(child):
271+
if is_tuple_fields(child.GetType().fields):
272272
summary = TupleSummaryProvider(child, _dict)
273273
else:
274274
summary = StructSummaryProvider(child, _dict)

0 commit comments

Comments
 (0)