Skip to content

Conversation

@woodruffw
Copy link
Member

@woodruffw woodruffw commented Dec 5, 2025

WIP. Still need to do the same for DockerUses.

The idea here is to get rid of a few of the low-level allocations that happen over and over, namely in things like Uses parsing. The main trick here is using self_cell to hold onto the backing String (instead of destroying it through from_str).

We can actually go even further than this, since serde_yaml supports borrowing from the original input. But that'll be a bigger refactor, since we lose track of the original loaded input at the moment.

Signed-off-by: William Woodruff <[email protected]>
Signed-off-by: William Woodruff <[email protected]>
Signed-off-by: William Woodruff <[email protected]>
Signed-off-by: William Woodruff <[email protected]>
@woodruffw woodruffw self-assigned this Dec 8, 2025
@woodruffw woodruffw added the refactor Refactoring tasks label Dec 8, 2025
Signed-off-by: William Woodruff <[email protected]>
@woodruffw woodruffw merged commit 1e25953 into main Dec 8, 2025
9 checks passed
@woodruffw woodruffw deleted the ww/uses-parsing branch December 8, 2025 01:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performance refactor Refactoring tasks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants