lahodaj opened a new pull request, #8242:
URL: https://github.com/apache/netbeans/pull/8242

   Consider code like:
   ```
   public class Test {
   |
   }
   ```
   
   And try to slowly type `Tes` inside VS Code. This will open code completion, 
and offer to create an initialize-all constructor. But, when the completion 
item is confirmed, it sometimes happens the action does not do anything (except 
removing the typed part of the identifier).
   
   The reason is, I think, that the completion is computed for `T`, and when 
typing `es`, the completion is filtered, but not recomputed. But, reparsing the 
source code (due to the typed characters) may change `Element` instances, and 
the lazy computation for `TextEdit` for inserting the constructor will try to 
compare original and current `Element` for equality, and that will fail.
   
   I am not quite clear why there is the `Element` check, but it would be 
better to compare e.g. `ElementHandle`s, not `Element`s.


-- 
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: notifications-unsubscr...@netbeans.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@netbeans.apache.org
For additional commands, e-mail: notifications-h...@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists

Reply via email to