search

eclipse-cdt / cdt-lsp

Eclipse CDT™ LSP Extensions for CDT
Eclipse Public License 2.0
23 stars 11 forks source link

[#327] fix blocking jobs #328

Closed ghentschke closed 1 month ago

ghentschke commented 1 month ago

...by removing synchronized from refreshTreeContentFromLS. The method will be called from different jobs started by the DeferredTreeContentManager. The completable future returned by symbols.thenAcceptAsync cannot join, when the refreshTreeContentFromLS is synchronized.

fixes #327

ghentschke commented 1 month ago

This change should fix blocking, but the code around looks too relaxed for multi-thread

Thanks. I'll take a closer look on it.