Fix TODO - false positive of UselessParenthesis (refs #709)

pmd-java/src/main/java/net/sourceforge/pmd/lang/java/ast/AbstractJavaAccessNode.java

@@ -120,8 +120,6 @@ public abstract class AbstractJavaAccessNode extends AbstractJavaNode implements
         setModifier(isDefault, DEFAULT);
     }
 
-    // TODO: fix the rule - around binary expressions the parentheses are needed...
-    @SuppressWarnings("PMD.UselessParentheses") 
     private boolean isModifier(int mask) {
         return (modifiers & mask) == mask;
     }

pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/unnecessary/xml/UselessParentheses.xml

@@ -460,4 +460,15 @@ public class Useless {
         ]]></code>
     </test-code>
 
+    <test-code>
+        <description>FP on binary operator</description>
+        <expected-problems>0</expected-problems>
+        <code><![CDATA[
+public class Foo {
+    private boolean isModifier(int mask) {
+        return (modifiers & mask) == mask;
+    }
+}
+        ]]></code>
+    </test-code>
 </test-data>