search

StefanKert / BuildVision

A Visual Studio extension to visualize the building process.
MIT License
261 stars 43 forks source link

Avoid overlapping timers #107

Closed davkean closed 3 years ago

davkean commented 3 years ago

Fixes: #106

StefanKert commented 3 years ago

Hi @davkean,

again thank you very much for this fix. I was quickly checking out the code and I think that it is possible to restructure the code because IMO we can get rid of the call to the IVsStatusbar and the fact that we are resolving the service for this very often.

IMO we can merge this PR as soon as the Disposing issue is fixed to roll out a fix for this perf issue quickly and I will try to restructure the calls to improve the way we are calling the service. I also created a issue to keep track of that #108 so if you are aware of any other issues please let me know.