[GitHub] lbruun commented on a change in pull request #320: Add the JavaFixUtilities.isPrimary() utility API

[GitBox <gi...@apache.org>]

lbruun commented on a change in pull request #320: Add the JavaFixUtilities.isPrimary() utility API
URL: https://github.com/apache/incubator-netbeans/pull/320#discussion_r161393655
 
 

 ##########
 File path: spi.java.hints/src/org/netbeans/spi/java/hints/JavaFixUtilities.java
 ##########
 @@ -1337,6 +1336,7 @@ private void rewrite(Tree from, Tree to) {
 
         OPERATOR_PRIORITIES.put(Kind.ARRAY_ACCESS, 1);
         OPERATOR_PRIORITIES.put(Kind.METHOD_INVOCATION, 1);
+        OPERATOR_PRIORITIES.put(Kind.MEMBER_REFERENCE, 1);
 
 Review comment:
   Will this affect other stuff than the new utility method introduced in this PR? 
   I can see that `OPERATOR_PRIORITIES` is also used in method `requiresParenthesis()`.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

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

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