Skip to content

ctest: Use askama comments rather than source comments.#4872

Merged
tgross35 merged 1 commit intorust-lang:mainfrom
tgross35:ctest-comments
Dec 3, 2025
Merged

ctest: Use askama comments rather than source comments.#4872
tgross35 merged 1 commit intorust-lang:mainfrom
tgross35:ctest-comments

Conversation

@tgross35
Copy link
Contributor

@tgross35 tgross35 commented Dec 3, 2025

We don't need the doc comments to be repeated for each invocation, so move them to either once-per-loop comments or to askama comments that get deleted from the template. On one target, this reduced the lines of generated C from 82700 to 62931, and Rust from 255569 to 227735.

We don't need the doc comments to be repeated for each invocation, so
move them to either once-per-loop comments or to askama comments that
get deleted from the template. On one target, this reduced the lines of
generated C from 82700 to 62931, and Rust from 255569 to 227735.
@rustbot rustbot added the ctest Issues relating to the ctest crate label Dec 3, 2025
@tgross35 tgross35 marked this pull request as ready for review December 3, 2025 22:36
@tgross35 tgross35 enabled auto-merge December 3, 2025 22:36
@tgross35 tgross35 added this pull request to the merge queue Dec 3, 2025
Merged via the queue into rust-lang:main with commit 826f0aa Dec 3, 2025
51 checks passed
@tgross35 tgross35 deleted the ctest-comments branch December 3, 2025 23:39
@tgross35 tgross35 added the stable-unneeded This PR is applied to main but already exists on libc-0.2 label Jan 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ctest Issues relating to the ctest crate stable-unneeded This PR is applied to main but already exists on libc-0.2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants