plasma-umass / coz

Coz: Causal Profiling
Other
4.03k stars 160 forks source link

[Rust] Remove thread_init #214

Closed lopukhov closed 1 year ago

lopukhov commented 1 year ago

After fixing #110 the function thread_init is not needed anymore.

This PR removes the function, its usage in the crate and the out of date caveat. This is a breaking change to the API, but in because the crate has not reached version 1.x this is considered ok in the Rust ecosystem (which tries to use SemVer). Before publishing to crates.io the crate version should be bumped to 0.2.x to signal the a breaking change. If the breaking change is not desired another option is to leave a dummy thread_init function that has no effect.

The changes have been tested with one of my programs, the toy example and with cargo test.