sdedic opened a new pull request, #4907: URL: https://github.com/apache/netbeans/pull/4907
The main issue fixed is the binding between LSP `UIContext` and the **first client that initializes it** and then is stuck with it. The original code gave the 1st client the priority over the client that may have been present in the computational context `Lookup.getDefault()`. The changed code will use the current Lookup.getDefault() that is transferred through various RP.Tasks - this is already implemented in UIContext.find() * fixes ADM report, so it uses fire-and-forge `notifyLater` instead `notify`. The LSP client may use modeless UI (vscode does) and the server thread is left blocked until the user dismisses the report (in vscode the user may just ignore it or hide it into status line without dismissing) * minor fix for tests: the structure must gson-deserialize on JDK11, so default ctor + setters are needed -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] For further information about the NetBeans mailing lists, visit: https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
