Skip to content

Add missing closing brackets to THIR output.#151215

Merged
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
AlexCeleste:thir-mismatched-brackets-and-field-names
Jan 16, 2026
Merged

Add missing closing brackets to THIR output.#151215
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
AlexCeleste:thir-mismatched-brackets-and-field-names

Conversation

@AlexCeleste

Copy link
Copy Markdown
Contributor

Closing brackets were missing on AdtDef, the field_types list in FruInfo, and InlineAsmExpr, breaking folding in some editors; Fields were incorrectly (?) indexed in the list for functional update syntax, showing the (implicit, irrelevant) iteration index instead of the field index; also spurious colon after Pat.

Closing brackets were missing on AdtDef, the field_types list in FruInfo, and InlineAsmExpr, breaking folding in some editors;
Fields were incorrectly (?) indexed in the list for functional update syntax, showing the (implicit, irrelevant) iteration index instead of the field index;
also spurious colon after Pat.
@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jan 16, 2026
@rustbot

rustbot commented Jan 16, 2026

Copy link
Copy Markdown
Collaborator

r? @jackh726

rustbot has assigned @jackh726.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@bjorn3

bjorn3 commented Jan 16, 2026

Copy link
Copy Markdown
Member

@bors r+ rollup

r? @bjorn3

@rust-bors

rust-bors Bot commented Jan 16, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 1f691f7 has been approved by bjorn3

It is now in the queue for this repository.

@rust-bors rust-bors Bot added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jan 16, 2026
@rust-bors rust-bors Bot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 16, 2026
}
}
}
,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

interesting, there usually aren't comma separators when fields are laid out vertically

rust-bors Bot pushed a commit that referenced this pull request Jan 16, 2026
Rollup of 4 pull requests

Successful merges:

 - #151196 (Ship LLVM (`rust-dev`) in fast try builds again)
 - #151198 (Add an additional help note to the ambiguity lint error)
 - #151215 (Add missing closing brackets to THIR output.)
 - #151218 (compiletest: Add `AuxCrate` struct with docs.)

r? @ghost
@rust-bors rust-bors Bot merged commit cfa8f97 into rust-lang:main Jan 16, 2026
11 checks passed
@rustbot rustbot added this to the 1.94.0 milestone Jan 16, 2026
rust-timer added a commit that referenced this pull request Jan 16, 2026
Rollup merge of #151215 - thir-mismatched-brackets-and-field-names, r=bjorn3

Add missing closing brackets to THIR output.

Closing brackets were missing on AdtDef, the field_types list in FruInfo, and InlineAsmExpr, breaking folding in some editors; Fields were incorrectly (?) indexed in the list for functional update syntax, showing the (implicit, irrelevant) iteration index instead of the field index; also spurious colon after Pat.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants