lkishalmi commented on pull request #2553:
URL: https://github.com/apache/netbeans/pull/2553#issuecomment-751504093


   After a two days debug session, I've found that the problem is most probably 
in the Lookup API implementation.
   Whenever the 
[ProxyLookup.setLookups()](https://github.com/apache/netbeans/blob/e087135bddcea43cea64fba402aa0c156fc0cc1c/platform/openide.util.lookup/src/org/openide/util/lookup/ProxyLookup.java#L183)
 is called, some listeners on the LookupResult do not get called as the changes 
in LazyLookups do not get evaluated.
   That boils down to 
https://github.com/apache/netbeans/blob/e087135bddcea43cea64fba402aa0c156fc0cc1c/platform/openide.util.lookup/src/org/openide/util/lookup/ProxyLookup.java#L684
 ```allItems()``` is being called wit a ```callBeforeLookup``` ```false```, 
changing that to true is an actual solution to the SourceGroup change problem, 
but it could break other testcases.
   It seems the template evaluation would happen at 
https://github.com/apache/netbeans/blob/e087135bddcea43cea64fba402aa0c156fc0cc1c/platform/openide.util.lookup/src/org/openide/util/lookup/ProxyLookup.java#L734
   without that the next call of initResults() would end up with some 
unevaluated cached result at 
   
https://github.com/apache/netbeans/blob/e087135bddcea43cea64fba402aa0c156fc0cc1c/platform/openide.util.lookup/src/org/openide/util/lookup/ProxyLookup.java#L440


----------------------------------------------------------------
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.

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

Reply via email to