From fcbd40f3ce04fe3011bf0e1b2642ac1fd049f5de Mon Sep 17 00:00:00 2001 From: Oleg Sukhodolsky Date: Mon, 13 Jun 2005 21:31:58 +0000 Subject: [PATCH] applied patch 1045127 --- .../checks/AbstractTypeAwareCheck.java | 19 ++++++++++++++++++- src/xdocs/releasenotes.xml | 2 ++ 2 files changed, 20 insertions(+), 1 deletion(-) diff --git a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/AbstractTypeAwareCheck.java b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/AbstractTypeAwareCheck.java index e87cb018f..cf9f3c097 100644 --- a/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/AbstractTypeAwareCheck.java +++ b/src/checkstyle/com/puppycrawl/tools/checkstyle/checks/AbstractTypeAwareCheck.java @@ -90,7 +90,9 @@ public abstract class AbstractTypeAwareCheck /** @see com.puppycrawl.tools.checkstyle.api.Check */ public final void leaveToken(DetailAST aAST) { - if (aAST.getType() == TokenTypes.CLASS_DEF) { + if (aAST.getType() == TokenTypes.CLASS_DEF + || aAST.getType() == TokenTypes.ENUM_DEF) + { // perhaps it was inner class int dotIdx = mCurrentClass.lastIndexOf("$"); if (dotIdx == -1) { @@ -105,6 +107,21 @@ public abstract class AbstractTypeAwareCheck mCurrentClass = mCurrentClass.substring(0, dotIdx); } } + else if (aAST.getType() != TokenTypes.PACKAGE_DEF + && aAST.getType() != TokenTypes.IMPORT) + { + leaveAST(aAST); + } + } + + /** + * Called when exiting an AST. A no-op by default, extending classes + * may choose to override this to augment their processing. + * @param aAST the AST we are departing. Guaranteed to not be PACKAGE_DEF, + * CLASS_DEF, or IMPORT + */ + protected void leaveAST(DetailAST aAST) + { } /** diff --git a/src/xdocs/releasenotes.xml b/src/xdocs/releasenotes.xml index f75daa886..8c2f2d190 100755 --- a/src/xdocs/releasenotes.xml +++ b/src/xdocs/releasenotes.xml @@ -23,6 +23,8 @@
  • One more bug fixed in type aware checks (RedundantThrows and JavadocMethod checks) (bug 1168408)
  • +
  • Applied patch 1045127 (thanks to Paul Constantinides)
  • +