Skip to content

Metadata macro/query cleanups#156131

Open
nnethercote wants to merge 2 commits intorust-lang:mainfrom
nnethercote:rm-unnecessary-decode
Open

Metadata macro/query cleanups#156131
nnethercote wants to merge 2 commits intorust-lang:mainfrom
nnethercote:rm-unnecessary-decode

Conversation

@nnethercote
Copy link
Copy Markdown
Contributor

Details in individual commits.

r? @Zalathar

- Improve a comment.
- Remove unused `[decode]` annotations on some arenas.
- Improve `impl_arena_allocatable_decoder`:
  - Make the first rule more like the second rule.
  - Remove unnecessary brackets.
  - Remove unused support for attributes other than `decode`.
The table names usually match the name of the corresponding query.
The `trait_impl_trait_tys` table is an exception; this commit renames it
`collect_return_position_impl_trait_in_trait_tys` to match the query.
@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 May 4, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented May 4, 2026

Zalathar is not on the review rotation at the moment.
They may take a while to respond.

@Zalathar
Copy link
Copy Markdown
Member

Zalathar commented May 6, 2026

Looks good.

I have mixed feelings about the second commit, because collect_return_position_impl_trait_in_trait_tys is kind of a weird table name, but matching the closely-linked query does seem good. And it would be easy enough to rename both the table and the query later, if a better name is found.

@bors r+ rollup

@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented May 6, 2026

📌 Commit cdefdd0 has been approved by Zalathar

It is now in the queue for this repository.

@rust-bors rust-bors Bot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 6, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request May 6, 2026
… r=Zalathar

Metadata macro/query cleanups

Details in individual commits.

r? @Zalathar
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request May 6, 2026
… r=Zalathar

Metadata macro/query cleanups

Details in individual commits.

r? @Zalathar
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request May 6, 2026
… r=Zalathar

Metadata macro/query cleanups

Details in individual commits.

r? @Zalathar
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.

3 participants