RedundantImport: remove redundant check

This commit is contained in:
Paul Campbell 2016-09-05 17:47:49 +01:00
parent 14462f9ea4
commit ee52569e22
2 changed files with 7 additions and 8 deletions

View file

@ -230,7 +230,7 @@ You need to include `checkstyle` as the version bundled with the `maven-checksty
* [ParameterNumber](#parameternumber) - checkstyle - enabled * [ParameterNumber](#parameternumber) - checkstyle - enabled
* [ParenPad](#parenpad) - checkstyle - enabled * [ParenPad](#parenpad) - checkstyle - enabled
* [PublicReferenceToPrivateType](#publicreferencetoprivatetype) - sevntu - enabled * [PublicReferenceToPrivateType](#publicreferencetoprivatetype) - sevntu - enabled
* [RedundantImport](#redundantimport) - checkstyle - enabled * [RedundantImport](#redundantimport) - checkstyle - disabled
* [RedundantModifier](#redundantmodifier) - checkstyle - enabled * [RedundantModifier](#redundantmodifier) - checkstyle - enabled
* [RedundantReturn](#redundantreturn) - sevntu - enabled * [RedundantReturn](#redundantreturn) - sevntu - enabled
* [Regexp](#regexp) - checkstyle - disabled * [Regexp](#regexp) - checkstyle - disabled
@ -1747,12 +1747,6 @@ doSomethingElse( 5);
doSomethingElse(5 ); doSomethingElse(5 );
```` ````
#### [RedundantImport](http://checkstyle.sourceforge.net/config_imports.html#RedundantImport)
Checks for redundant `import`s. Checks for duplicates, imports from the `java.lang` package or from the current package.
> TODO: remove - [UnusedImports](#unusedimports) performs all the same checks and more
#### [RedundantModifier](http://checkstyle.sourceforge.net/config_modifier.html#RedundantModifier) #### [RedundantModifier](http://checkstyle.sourceforge.net/config_modifier.html#RedundantModifier)
Checks for redundant modifiers. Checks for: Checks for redundant modifiers. Checks for:
@ -2709,6 +2703,12 @@ Already covered by the [OneTopLevelClass](#onetoplevelclass) check.
> TODO: enable > TODO: enable
#### [RedundantImport](http://checkstyle.sourceforge.net/config_imports.html#RedundantImport)
[UnusedImports](#unusedimports) performs all the same checks and more.
Checks for redundant `import`s. Checks for duplicates, imports from the `java.lang` package or from the current package.
#### [Regexp](http://checkstyle.sourceforge.net/config_regexp.html#Regexp) #### [Regexp](http://checkstyle.sourceforge.net/config_regexp.html#Regexp)
Generic rule; doesn't embody a 'quality' check. Generic rule; doesn't embody a 'quality' check.

View file

@ -174,7 +174,6 @@
</module> </module>
<module name="ParenPad"/> <!-- parentheses should have no padding spaces --> <module name="ParenPad"/> <!-- parentheses should have no padding spaces -->
<module name="RedundantImport"/> <!-- checks for redundant imports (i.e. in the same package) -->
<module name="RedundantModifier"/> <!-- checks for redundant modifies (e.g. public methods in an interface) --> <module name="RedundantModifier"/> <!-- checks for redundant modifies (e.g. public methods in an interface) -->
<module name="ReturnCount"/> <!-- Restricts return statements to 2 per method (1 if return type is void) --> <module name="ReturnCount"/> <!-- Restricts return statements to 2 per method (1 if return type is void) -->
<module name="RightCurly"/> <!-- placement of right curly braces ('}') for code blocks at end of line --> <module name="RightCurly"/> <!-- placement of right curly braces ('}') for code blocks at end of line -->