diff --git a/pmd-apex/src/main/java/net/sourceforge/pmd/lang/apex/rule/security/ApexOpenRedirectRule.java b/pmd-apex/src/main/java/net/sourceforge/pmd/lang/apex/rule/security/ApexOpenRedirectRule.java
index 554b492554..22cedeaf22 100644
--- a/pmd-apex/src/main/java/net/sourceforge/pmd/lang/apex/rule/security/ApexOpenRedirectRule.java
+++ b/pmd-apex/src/main/java/net/sourceforge/pmd/lang/apex/rule/security/ApexOpenRedirectRule.java
@@ -101,9 +101,11 @@ public class ApexOpenRedirectRule extends AbstractApexRule {
             }
         } else {
             if (node instanceof ASTField) {
-                /* sergey.gorbaty: 
-                 * Apex Jorje parser is returning a null from Field.getFieldInfo(), but the info is available from an inner field. 
-                 * DO NOT attempt to optimize this block without checking that Jorje parser actually fixed its bug.
+                /*
+                 * sergey.gorbaty: Apex Jorje parser is returning a null from
+                 * Field.getFieldInfo(), but the info is available from an inner
+                 * field. DO NOT attempt to optimize this block without checking
+                 * that Jorje parser actually fixed its bug.
                  * 
                  */
                 try {
@@ -143,13 +145,6 @@ public class ApexOpenRedirectRule extends AbstractApexRule {
         }
     }
 
-    private void addVariable(AbstractApexNode<?> node) {
-        ASTVariableExpression variable = node.getFirstChildOfType(ASTVariableExpression.class);
-        if (variable != null) {
-            listOfStringLiteralVariables.add(Helper.getFQVariableName(variable));
-        }
-    }
-
     /**
      * Traverses all new declarations to find PageReferences
      *