Comment From: sbrannen

This has been merged into main in 490b5c77fc7bc504a03792f5690189c3429b4ff0 and revised in d71853f1054ff6b5e86e7c43db54ad955853bac6.

Thanks


FYI: I unfortunately had to revert two sets of changes due to bugs in the Eclipse compiler (see b69e5acfe3e16f282f2f5a7bbb04c624cf9e6df9). Once those bugs have been fixed we can consider reintroducing those changes.

Comment From: quaff

FYI: I unfortunately had to revert two sets of changes due to bugs in the Eclipse compiler (see b69e5ac). Once those bugs have been fixed we can consider reintroducing those changes.

  • See https://github.com/eclipse-jdt/eclipse.jdt.core/issues/1394

Comment From: sbrannen

Thanks for the link, @quaff. 👍

Unfortunately that seems slightly different from what I encountered (or at least one of the use cases that failed for me), so I reported our use case separately.

  • https://github.com/eclipse-jdt/eclipse.jdt.core/issues/1686

Note that the example I submitted for that Eclipse issue was based on an extremely simplified version of SharedConnectionInvocationHandler#invoke(Object, Method, Object[]).

Comment From: quaff

Thanks for the link, @quaff. 👍

Unfortunately that seems slightly different from what I encountered (or at least one of the use cases that failed for me), so I reported our use case separately.

Note that the example I submitted for that Eclipse issue was based on an extremely simplified version of SharedConnectionInvocationHandler#invoke(Object, Method, Object[]).

FYI: I verified that ControllerMethodInvocationInterceptor caused that issue I linked.

Comment From: sbrannen

FYI: I verified that ControllerMethodInvocationInterceptor caused that issue I linked.

Thanks for confirming that! 👍

Now we know that we're waiting on both of those issues to be resolved.