You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Ah yes. This is not pure, but there is an issue with the notify sometimes taking too long, so I tried to split up the work. Chrome complains sometimes about setTimeout handler taking too long to complete. This happens when it hovers around the 50ms mark.
Ok so I did not merge this. Sorry.
The reason is that I noticed that there is a real perf issue, only when you change config and have many loggers.
With a thousand loggers it takes 4 - 8 seconds! That's way too long.
It's better to fix the real perf issue than to hide it behind setTimeout I think.
I have some ideas on how to tackle it but it might make the API for mods (even) more messy.... Then I got a new job and got totally swamped with work and stopped working on this for a while.
But I will get back to this. Eventually. You are not forgotten.
Ah yes. This is not pure, but there is an issue with the notify sometimes taking too long, so I tried to split up the work. Chrome complains sometimes about setTimeout handler taking too long to complete. This happens when it hovers around the 50ms mark.
Originally posted by @Download in #52 (comment)
The text was updated successfully, but these errors were encountered: