From 4fade39e72675b776ef27c5fd84531bcb850da37 Mon Sep 17 00:00:00 2001 From: Oliver Burn Date: Mon, 3 Dec 2001 00:42:42 +0000 Subject: [PATCH] patch for ignoring whitespace between Javadoc and declarations --- ChangeLog | 6 ++++++ .../tools/checkstyle/VerifierImpl.java | 21 +++++++++++++++++-- .../tools/checkstyle/InputWhitespace.java | 9 ++++++++ 3 files changed, 34 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 186f278ab..35f9d2421 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2001-12-03 Oliver Burn + + * src/checkstyle/com/puppycrawl/tools/checkstyle/VerifierImpl.java: + Included patch to ignore blank lines between Javadoc and the + declaration. From Lars kuehne [ lars _DOT_ kuehne _AT_ ppi _DOT_ de ]. + 2001-11-26 Oliver Burn * src/checkstyle/com/puppycrawl/tools/checkstyle/java.g diff --git a/src/checkstyle/com/puppycrawl/tools/checkstyle/VerifierImpl.java b/src/checkstyle/com/puppycrawl/tools/checkstyle/VerifierImpl.java index 8cc4b4d6e..34e341e10 100644 --- a/src/checkstyle/com/puppycrawl/tools/checkstyle/VerifierImpl.java +++ b/src/checkstyle/com/puppycrawl/tools/checkstyle/VerifierImpl.java @@ -598,8 +598,25 @@ class VerifierImpl **/ private String[] getJavadocBefore(int aLineNo) { - // #HACK#: should be improved to skip blank lines. - return (String[]) mComments.get(new Integer(aLineNo - 1)); + int lineNo = aLineNo - 1; + + // skip blank lines + while ((lineNo > 0) && lineIsBlank(lineNo)) { + lineNo--; + } + + return (String[]) mComments.get(new Integer(lineNo)); + } + + /** + * Checks if the specified line is blank. + * @param aLineNo the line number to check + * @return if the specified line consists only of tabs and spaces. + **/ + private boolean lineIsBlank(int aLineNo) + { + // possible improvement: avoid garbage creation in trim() + return "".equals(mLines[aLineNo].trim()); } /** diff --git a/src/tests/com/puppycrawl/tools/checkstyle/InputWhitespace.java b/src/tests/com/puppycrawl/tools/checkstyle/InputWhitespace.java index 498d0eac7..c2a63db7b 100644 --- a/src/tests/com/puppycrawl/tools/checkstyle/InputWhitespace.java +++ b/src/tests/com/puppycrawl/tools/checkstyle/InputWhitespace.java @@ -39,4 +39,13 @@ class InputWhitespace catch(RuntimeException e){ } } + + /** + skip blank lines between comment and code, + should be ok + **/ + + + private int mVar4 = 1; + }