diff --git a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/DoubleCheckedLockingCheck.java b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/DoubleCheckedLockingCheck.java index 2050477fd..a4d97458f 100644 --- a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/DoubleCheckedLockingCheck.java +++ b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/DoubleCheckedLockingCheck.java @@ -30,8 +30,8 @@ import com.puppycrawl.tools.checkstyle.api.TokenTypes; * * See The "Double-Checked Locking is Broken" Declaration - * for a more in depth explanation. + * >The "Double-Checked Locking is Broken" Declaration for a + * more in depth explanation. * * @author Lars Kühne */ @@ -46,13 +46,13 @@ public class DoubleCheckedLockingCheck extends Check /** @see Check */ public void visitToken(DetailAST aAST) { - DetailAST synchronizedAST = + final DetailAST synchronizedAST = getLowestParent(aAST, TokenTypes.LITERAL_SYNCHRONIZED); if (synchronizedAST == null) { return; } - DetailAST ifAST = + final DetailAST ifAST = getLowestParent(synchronizedAST, TokenTypes.LITERAL_IF); if (ifAST == null) { return; @@ -83,8 +83,8 @@ public class DoubleCheckedLockingCheck extends Check private DetailAST getLowestParent(DetailAST aAST, int aTokenType) { DetailAST synchronizedParent = aAST; - while (synchronizedParent != null - && synchronizedParent.getType() != aTokenType) + while ((synchronizedParent != null) + && (synchronizedParent.getType() != aTokenType)) { synchronizedParent = synchronizedParent.getParent(); }