diff --git a/findbugs/src/java/edu/umd/cs/findbugs/detect/FindRefComparison.java b/findbugs/src/java/edu/umd/cs/findbugs/detect/FindRefComparison.java
index <HASH>..<HASH> 100644
--- a/findbugs/src/java/edu/umd/cs/findbugs/detect/FindRefComparison.java
+++ b/findbugs/src/java/edu/umd/cs/findbugs/detect/FindRefComparison.java
@@ -569,7 +569,7 @@ public class FindRefComparison implements Detector, ExtendedTypes {
                 if (field != null) {
                     // If the field is final, we'll assume that the String value
                     // is static.
-                    if (field.isFinal() && field.isFinal()) {
+                    if (field.isFinal()) {
                         pushValue(staticStringTypeInstance);
                     } else {
                         pushValue(type);
Update FindRefComparison.java

This expression looks redundant; is it sensible to remove?