etternagame / etterna

Advanced cross-platform rhythm game focused on keyboard play
https://etternaonline.com/
MIT License
496 stars 135 forks source link

Fix duplicate log suppression pr :) #1305

Closed bluebandit21 closed 3 months ago

bluebandit21 commented 5 months ago

Should fix problems of #1304

Note -- in order to prevent acquiring a global mutex between multiple threads, I exploit the fact that a spammed log message would happen on the same thread, letting us use thread_local variables instead.

poco0317 commented 3 months ago

replaced by 82a5e44653545910536f6ec318340191bbbdff51 because I think the amount of effort going into consolidating log lines has gotten lost in the sauce. it's better for now to restrict log file sizes and be reasonable about the amount of logging we do. if a log file ends up hitting a 64mb cap, then theres obviously a problem and we can look at how that happened in the first place