sdedic commented on a change in pull request #3906:
URL: https://github.com/apache/netbeans/pull/3906#discussion_r840324890
##########
File path:
java/java.lsp.server/src/org/netbeans/modules/java/lsp/server/protocol/Server.java
##########
@@ -195,6 +196,9 @@ synchronized void attachClient(NbCodeLanguageClient client)
{
public MessageConsumer attachLookup(MessageConsumer delegate) {
// PENDING: allow for message consumer wrappers to be registered
to add pre/post processing for
// the request plus build the request's default Lookup contents.
+ if (!(delegate instanceof Endpoint)) {
Review comment:
Thanks for catching this! I just didn't get right the idea that lsp4j
sends BOTH the incoming and outgoing messages go throught the same wrapper
through my head :(
--
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
