From bc9eae330a4fc5c37eb368c85753ae11f2f2c322 Mon Sep 17 00:00:00 2001
From: Roman Ivanov <ivanov-jr@mail.ru>
Date: Wed, 22 Apr 2015 17:05:31 -0400
Subject: [PATCH] PMD Rule:OneDeclarationPerLine Priority:4 Use one line for
 each declaration, it enhances code readability. Issue #744

---
 config/pmd.xml                                                 | 1 -
 .../tools/checkstyle/checks/SuppressWarningsHolder.java        | 3 ++-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/config/pmd.xml b/config/pmd.xml
index fb4b42c09..4b15adeb4 100644
--- a/config/pmd.xml
+++ b/config/pmd.xml
@@ -73,7 +73,6 @@
     <exclude name="DataflowAnomalyAnalysis"/>
     <!-- turning local varibles to fields create design problems and extend scope of variable -->
     <exclude name="AvoidFinalLocalVariable"/>
-    <exclude name="OneDeclarationPerLine"/>
     <exclude name="AvoidPrefixingMethodParameters"/>
     <exclude name="AvoidLiteralsInIfCondition"/>
     <!-- Checkstyle is not thread safe -->
diff --git a/src/main/java/com/puppycrawl/tools/checkstyle/checks/SuppressWarningsHolder.java b/src/main/java/com/puppycrawl/tools/checkstyle/checks/SuppressWarningsHolder.java
index 980131756..15542cae4 100644
--- a/src/main/java/com/puppycrawl/tools/checkstyle/checks/SuppressWarningsHolder.java
+++ b/src/main/java/com/puppycrawl/tools/checkstyle/checks/SuppressWarningsHolder.java
@@ -289,7 +289,8 @@ public class SuppressWarningsHolder
             final int firstLine = targetAST.getLineNo();
             final int firstColumn = targetAST.getColumnNo();
             final DetailAST nextAST = targetAST.getNextSibling();
-            final int lastLine, lastColumn;
+            final int lastLine;
+            final int lastColumn;
             if (nextAST != null) {
                 lastLine = nextAST.getLineNo();
                 lastColumn = nextAST.getColumnNo() - 1;