From ba325eb8a27c1ec04a3e9983678eeb1fd0a87628 Mon Sep 17 00:00:00 2001 From: Sergey Date: Mon, 16 Jan 2017 11:35:49 -0800 Subject: [PATCH] Dead code removal --- .../apex/rule/security/ApexOpenRedirectRule.java | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) 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 *