Celtoys / Remotery

Single C file, Realtime CPU/GPU Profiler with Remote Web Viewer
Apache License 2.0
3.11k stars 264 forks source link

fix TSAN warnings on macOS #238

Closed conversy closed 2 years ago

dwilliamson commented 2 years ago

Will merge this tomorrow when I get back in the office. Thanks...

JCash commented 2 years ago

I've updated my draft PR with a similar change as the one suggested here. I've tested with asan, ubsan and tsan and it seems to behave correctly in the browser view. Currently only tested on macOS. My goal is to test it on Android as well (that's where I have some issues currently)

dwilliamson commented 2 years ago

Thanks for submitting this. Going to move to the more complete https://github.com/Celtoys/Remotery/pull/239 based on some of your ideas.

conversy commented 2 years ago

Sure, looking forward to it!