mstange / samply

Command-line sampling profiler for macOS and Linux
Apache License 2.0
1.98k stars 48 forks source link

Consolidate state and remove refcells #206

Closed mstange closed 1 month ago

mstange commented 1 month ago

This PR makes it so that all ProfileContext state mutation happens through ProfileContext methods. This lets us remove many RefCells.

Not super well-tested but I figured I'd rather land it early so that we can find and fix the regressions together.