Skip to content

Conversation

@TimothyMakkison
Copy link
Contributor

@TimothyMakkison TimothyMakkison commented Dec 26, 2025

Replace redundant repeat allocations with Linq. TUnit has a lot of code that replaces a 24 byte Linq allocation (sometimes Linq doesn't even allocate) with a O(n) temporary collection allocation.

There are way more of these for instance 0b1df1a, I don't think the AI understand Linq, as a rule of thumb, any comment starting with the word Optimize or mentions the word Linq is suspect

@thomhurst
Copy link
Owner

Thanks!

@thomhurst thomhurst merged commit be92dac into thomhurst:main Dec 26, 2025
9 of 10 checks passed
This was referenced Dec 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants