Delete ///CLOVER comments

Issue #823
This commit is contained in:
Damian Szczepanik 2015-03-20 20:55:36 +01:00
parent 59d1b6ad1f
commit 9f2bf96b20
7 changed files with 3 additions and 31 deletions

View File

@ -24,14 +24,12 @@ package com.puppycrawl.tools.checkstyle;
**/
public final class Defn
{
///CLOVER:OFF
/**
* Do no allow <code>Defn</code> instances to be created.
**/
private Defn()
{
}
///CLOVER:ON
/** Name of resource bundle for Checkstyle. */
public static final String CHECKSTYLE_BUNDLE =

View File

@ -108,25 +108,11 @@ class PackageObjectFactory implements ModuleFactory
throws CheckstyleException
{
try {
final Class<?> clazz = Class.forName(className, true,
moduleClassLoader);
final Class<?> clazz = Class.forName(className, true, moduleClassLoader);
return clazz.newInstance();
}
catch (final ClassNotFoundException e) {
throw new CheckstyleException(
"Unable to find class for " + className, e);
}
catch (final InstantiationException e) {
///CLOVER:OFF
throw new CheckstyleException(
"Unable to instantiate " + className, e);
///CLOVER:ON
}
catch (final IllegalAccessException e) {
///CLOVER:OFF
throw new CheckstyleException(
"Unable to instantiate " + className, e);
///CLOVER:ON
catch (final ClassNotFoundException | InstantiationException | IllegalAccessException e) {
throw new CheckstyleException("Unable to find class for " + className, e);
}
}

View File

@ -27,12 +27,10 @@ import antlr.collections.AST;
*/
public final class ScopeUtils
{
///CLOVER:OFF
/** prevent instantiation */
private ScopeUtils()
{
}
///CLOVER:ON
/**
* Returns the Scope specified by the modifier set.

View File

@ -63,7 +63,6 @@ public class HandlerFactory
});
typeHandlers.put(type, ctor);
}
///CLOVER:OFF
catch (final NoSuchMethodException e) {
throw new RuntimeException("couldn't find ctor for "
+ handlerClass);
@ -73,7 +72,6 @@ public class HandlerFactory
throw new RuntimeException("couldn't find ctor for "
+ handlerClass);
}
///CLOVER:ON
}
/** Creates a HandlerFactory. */
@ -169,7 +167,6 @@ public class HandlerFactory
indentCheck, ast, parent);
}
}
///CLOVER:OFF
catch (final InstantiationException e) {
LOG.debug("couldn't instantiate constructor for " + ast, e);
throw new RuntimeException("couldn't instantiate constructor for "
@ -188,7 +185,6 @@ public class HandlerFactory
if (expHandler == null) {
throw new RuntimeException("no handler for type " + ast.getType());
}
///CLOVER:ON
return expHandler;
}

View File

@ -89,13 +89,11 @@ public final class JavadocUtils
TOKEN_VALUE_TO_NAME = tempTokenValueToName;
}
///CLOVER:OFF
/** prevent instantiation */
private JavadocUtils()
{
}
///CLOVER:ON
/**
* Gets validTags from a given piece of Javadoc.

View File

@ -232,9 +232,7 @@ public class SuppressWithNearbyCommentFilter
final Matcher matcher = regexp.matcher(comment);
// Match primarily for effect.
if (!matcher.find()) {
///CLOVER:OFF
return string;
///CLOVER:ON
}
String result = string;
for (int i = 0; i <= matcher.groupCount(); i++) {

View File

@ -229,9 +229,7 @@ public class SuppressionCommentFilter
final Matcher matcher = regexp.matcher(comment);
// Match primarily for effect.
if (!matcher.find()) {
///CLOVER:OFF
return string;
///CLOVER:ON
}
String result = string;
for (int i = 0; i <= matcher.groupCount(); i++) {