certify icon indicating copy to clipboard operation
certify copied to clipboard

Improved locking performance on .NET 9.0+

Open MarkCiliaVincenti opened this issue 1 year ago • 5 comments

The new System.Threading.Lock offers greater performance, as independent benchmarks show (eg https://steven-giesel.com/blogPost/4cf6d48a-ec9d-4c68-961c-31fd8d8c1340)

MarkCiliaVincenti avatar Sep 01 '24 08:09 MarkCiliaVincenti

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Sep 01 '24 08:09 CLAassistant

Thanks Mark! We don't typically struggle with lock performance but we'll definitely consider integrating this for modernity, the backported dependency is potentially a little risky for us as all of our users are currently on net462 builds but we'll see.

webprofusion-chrisc avatar Sep 02 '24 00:09 webprofusion-chrisc

Are you still interested in this @webprofusion-chrisc? In the meantime, the library can now be used as a source generator so that you do not add an external dependency.

MarkCiliaVincenti avatar Nov 30 '24 08:11 MarkCiliaVincenti

@webprofusion-chrisc would you like the source generator version?

MarkCiliaVincenti avatar Dec 14 '24 09:12 MarkCiliaVincenti

Hi Mark we'll definitely consider this for next year and yes a source generator would be preferable to a run-time library dependency.

webprofusion-chrisc avatar Dec 16 '24 00:12 webprofusion-chrisc