ArrayTrailingComma added to Checkstyle style.
All new failures fixed.
This commit is contained in:
parent
2cd7daccf2
commit
40de78da0f
|
|
@ -116,5 +116,6 @@
|
|||
<property name="braceAdjustment" value="0"/>
|
||||
<property name="caseIndent" value="0"/>
|
||||
</module>
|
||||
<module name="ArrayTrailingComma"/>
|
||||
</module>
|
||||
</module>
|
||||
|
|
|
|||
|
|
@ -190,7 +190,8 @@ final class PropertyCacheFile
|
|||
/** hex digits */
|
||||
private static char[] sHexChars = {
|
||||
'0', '1', '2', '3', '4', '5', '6', '7',
|
||||
'8', '9', 'A', 'B', 'C', 'D', 'E', 'F' };
|
||||
'8', '9', 'A', 'B', 'C', 'D', 'E', 'F',
|
||||
};
|
||||
|
||||
/** mask for last byte */
|
||||
private static final int MASK_0X0F = 0x0F;
|
||||
|
|
|
|||
|
|
@ -53,7 +53,7 @@ public class XMLLogger
|
|||
|
||||
/** some known entities to detect */
|
||||
private static final String[] ENTITIES = {"gt", "amp", "lt", "apos",
|
||||
"quot"};
|
||||
"quot", };
|
||||
|
||||
/**
|
||||
* Creates a new <code>XMLLogger</code> instance.
|
||||
|
|
|
|||
|
|
@ -250,7 +250,8 @@ public class JavadocMethodCheck
|
|||
TokenTypes.PACKAGE_DEF,
|
||||
TokenTypes.IMPORT,
|
||||
TokenTypes.METHOD_DEF,
|
||||
TokenTypes.CTOR_DEF};
|
||||
TokenTypes.CTOR_DEF,
|
||||
};
|
||||
}
|
||||
|
||||
/** @see com.puppycrawl.tools.checkstyle.api.Check */
|
||||
|
|
@ -258,7 +259,8 @@ public class JavadocMethodCheck
|
|||
{
|
||||
return new int[] {
|
||||
TokenTypes.METHOD_DEF,
|
||||
TokenTypes.CTOR_DEF};
|
||||
TokenTypes.CTOR_DEF,
|
||||
};
|
||||
}
|
||||
|
||||
/** @see com.puppycrawl.tools.checkstyle.api.Check */
|
||||
|
|
@ -266,7 +268,8 @@ public class JavadocMethodCheck
|
|||
{
|
||||
return new int[] {
|
||||
TokenTypes.PACKAGE_DEF,
|
||||
TokenTypes.IMPORT};
|
||||
TokenTypes.IMPORT,
|
||||
};
|
||||
}
|
||||
|
||||
/** @see com.puppycrawl.tools.checkstyle.api.Check */
|
||||
|
|
|
|||
|
|
@ -74,7 +74,7 @@ public class ModifierOrderCheck
|
|||
private static final String[] JLS_ORDER =
|
||||
{
|
||||
"public", "protected", "private", "abstract", "static", "final",
|
||||
"transient", "volatile", "synchronized", "native", "strictfp"
|
||||
"transient", "volatile", "synchronized", "native", "strictfp",
|
||||
};
|
||||
|
||||
/** @see com.puppycrawl.tools.checkstyle.api.Check */
|
||||
|
|
|
|||
|
|
@ -49,8 +49,10 @@ public class RedundantModifierCheck
|
|||
/** @see com.puppycrawl.tools.checkstyle.api.Check */
|
||||
public int[] getDefaultTokens()
|
||||
{
|
||||
return new int[] {TokenTypes.METHOD_DEF,
|
||||
TokenTypes.VARIABLE_DEF};
|
||||
return new int[] {
|
||||
TokenTypes.METHOD_DEF,
|
||||
TokenTypes.VARIABLE_DEF,
|
||||
};
|
||||
}
|
||||
|
||||
/** @see com.puppycrawl.tools.checkstyle.api.Check */
|
||||
|
|
|
|||
|
|
@ -93,7 +93,8 @@ public class RedundantThrowsCheck
|
|||
TokenTypes.PACKAGE_DEF,
|
||||
TokenTypes.IMPORT,
|
||||
TokenTypes.METHOD_DEF,
|
||||
TokenTypes.CTOR_DEF};
|
||||
TokenTypes.CTOR_DEF,
|
||||
};
|
||||
}
|
||||
|
||||
/** @see com.puppycrawl.tools.checkstyle.api.Check */
|
||||
|
|
|
|||
|
|
@ -66,9 +66,11 @@ public class RightCurlyCheck
|
|||
/** @see com.puppycrawl.tools.checkstyle.api.Check */
|
||||
public int[] getDefaultTokens()
|
||||
{
|
||||
return new int[] {TokenTypes.LITERAL_TRY,
|
||||
TokenTypes.LITERAL_CATCH,
|
||||
TokenTypes.LITERAL_ELSE};
|
||||
return new int[] {
|
||||
TokenTypes.LITERAL_TRY,
|
||||
TokenTypes.LITERAL_CATCH,
|
||||
TokenTypes.LITERAL_ELSE,
|
||||
};
|
||||
}
|
||||
|
||||
/** @see com.puppycrawl.tools.checkstyle.api.Check */
|
||||
|
|
|
|||
|
|
@ -62,6 +62,6 @@ public class TypeNameCheck
|
|||
public int[] getDefaultTokens()
|
||||
{
|
||||
return new int[] {TokenTypes.CLASS_DEF,
|
||||
TokenTypes.INTERFACE_DEF};
|
||||
TokenTypes.INTERFACE_DEF, };
|
||||
}
|
||||
}
|
||||
|
|
|
|||
|
|
@ -77,10 +77,12 @@ public class UnusedImportsCheck
|
|||
/** @see com.puppycrawl.tools.checkstyle.api.Check */
|
||||
public int[] getDefaultTokens()
|
||||
{
|
||||
return new int[] {TokenTypes.IMPORT,
|
||||
TokenTypes.CLASS_DEF,
|
||||
TokenTypes.INTERFACE_DEF,
|
||||
TokenTypes.IDENT};
|
||||
return new int[] {
|
||||
TokenTypes.IMPORT,
|
||||
TokenTypes.CLASS_DEF,
|
||||
TokenTypes.INTERFACE_DEF,
|
||||
TokenTypes.IDENT,
|
||||
};
|
||||
}
|
||||
|
||||
/** @see com.puppycrawl.tools.checkstyle.api.Check */
|
||||
|
|
|
|||
|
|
@ -147,7 +147,7 @@ public class WhitespaceAroundCheck
|
|||
TokenTypes.SR_ASSIGN,
|
||||
TokenTypes.STAR,
|
||||
TokenTypes.STAR_ASSIGN,
|
||||
TokenTypes.LITERAL_ASSERT
|
||||
TokenTypes.LITERAL_ASSERT,
|
||||
};
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -48,7 +48,7 @@ public class BlockParentHandler extends ExpressionHandler
|
|||
TokenTypes.VARIABLE_DEF,
|
||||
TokenTypes.EXPR,
|
||||
TokenTypes.OBJBLOCK,
|
||||
TokenTypes.LITERAL_BREAK
|
||||
TokenTypes.LITERAL_BREAK,
|
||||
};
|
||||
|
||||
/**
|
||||
|
|
|
|||
|
|
@ -34,7 +34,7 @@ public class CaseHandler extends ExpressionHandler
|
|||
*/
|
||||
private final int[] mCaseChildren = new int[] {
|
||||
TokenTypes.LITERAL_CASE,
|
||||
TokenTypes.LITERAL_DEFAULT
|
||||
TokenTypes.LITERAL_DEFAULT,
|
||||
};
|
||||
|
||||
/**
|
||||
|
|
|
|||
|
|
@ -52,8 +52,8 @@ public class HandlerFactory
|
|||
try {
|
||||
Constructor ctor = aHandlerClass.getConstructor(new Class[] {
|
||||
IndentationCheck.class,
|
||||
DetailAST.class, // current AST
|
||||
ExpressionHandler.class // parent
|
||||
DetailAST.class, // current AST
|
||||
ExpressionHandler.class, // parent
|
||||
});
|
||||
mTypeHandlers.put(new Integer(aType), ctor);
|
||||
}
|
||||
|
|
@ -149,7 +149,7 @@ public class HandlerFactory
|
|||
new Object[] {
|
||||
aIndentCheck,
|
||||
aAst,
|
||||
aParent
|
||||
aParent,
|
||||
}
|
||||
);
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in New Issue