search

YairHalberstadt / stronginject

compile time dependency injection for .NET
MIT License
845 stars 24 forks source link

Optimize memory #182

Closed sharwell closed 2 years ago

sharwell commented 2 years ago

This is intended to fix allocation items highlighted in #179.

📝 Note that I haven't tested this locally.

📝 Based on the Codacy results for this pull request, I would consider disabling it. The suggestions it made ranged from questionable to outright wrong (execution correctness requires the author to know that Codacy suggestion must not be followed).

YairHalberstadt commented 2 years ago

Thanks this all looks great!

I moved this over to #183 so that I could reset the last commit, and that's now been merged.