Reflection cache

[karljj1]

Calling GetMembers is slow and generates a lot of garbage. This introduces a simple cache so that we only need to do the query once and then any subsequent queries can come from the cache. Its quite simple but gave a nice performance improvement and reduced the garbage by quite a lot for us :)

[codecov]

Codecov Report

Merging #155 (bb4e2df) into main (aa9e428) will increase coverage by 0%.
The diff coverage is 100%.

@@         Coverage Diff         @@
##           main   #155   +/-   ##
===================================
  Coverage    94%    94%           
===================================
  Files        42     42           
  Lines      1749   1762   +13     
===================================
+ Hits       1651   1664   +13     
  Misses       98     98           

Impacted Files	Coverage Δ
src/SmartFormat/Extensions/ReflectionSource.cs	100% <100%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update aa9e428...bb4e2df. Read the comment docs.

[axunonb]

Thanks, nice PR, will review this week.

[axunonb]

Did you measure the performance gain with the type cache?
My first rough test loop says performance doubled.

[karljj1]

I did test it on our version but I have not tested this one.

This was my results which also includes the object pooling.

I also disabled the threading support for the ListFormatter in our version as that is also quite slow and we don't use multithreading.

Unfortunately, I don't have the results from running the perf tests now but I did see similar results to what you describe.

You can see the reflection source has a huge perf improvement :)

[axunonb]

Really great PR, thanks!