Fixes some `SpellCheckingInspection` inspection violations.
Description:
>Spellchecker inspection helps locate typos and misspelling in your code, comments and literals.
While suppression works fine, it causes false positive in `RedundantSuppression` inspection reported as [IDEA-144818](https://youtrack.jetbrains.com/issue/IDEA-144818).
Tag `@noinspection` is self-descriptive enough on its own.
ParameterNameDiffersFromOverriddenParameter does not require parameters to match names anymore if method is from external library and parameter may be wrongly named there.
Fixes some `SpellCheckingInspection` inspection violations.
Description:
>Spellchecker inspection helps locate typos and misspelling in your code, comments and literals.
Fixes some `SpellCheckingInspection` inspection violations.
Description:
>Spellchecker inspection helps locate typos and misspelling in your code, comments and literals.
Fixes some `TestMethodWithoutAssertion` inspection violations.
Description:
>Reports test methods of JUnit test case classes that do not contain any assertions. Such methods indicate either incomplete or weak test cases.
Fixes `ConditionalExpression` inspection violations.
Description:
>Reports the ternary condition operator. Some coding standards prohibit the use of the condition operator, in favor of if-else statements.
Fixes `SpellCheckingInspection` inspection violations introduced in recent commits.
Description:
>Spellchecker inspection helps locate typos and misspelling in your code, comments and literals.
Fixes `ProblematicWhitespace` inspection violations introduced in recent commits.
Description:
>Reports tabs used for indentation when the code style is configured to use only spaces.
Fixes `LocalVariableNamingConvention` inspection violations introduced in recent commits.
Description:
>Reports local variables whose names are either too short, too long, or do not follow the specified regular expression pattern.
Fixes `ZeroLengthArrayInitialization` inspection violations introduced in recent commits.
Description:
>Reports on allocations of arrays with known lengths of zero. Since array lengths in Java are non-modifiable, it is almost always possible to share zero-length arrays, rather than repeatedly allocating new zero-length arrays. Such sharing may provide useful optimizations in program runtime or footprint. Note that this inspection does not report zero-length arrays allocated as static final fields, as it is assumed that those arrays are being used to implement array sharing.
Fixes `SimplifiableJUnitAssertion` inspection violations introduced in recent commits.
Description:
>Reports any JUnit assertTrue calls which can be replaced by equivalent assertEquals calls. assertEquals calls will normally give better error messages in case of test failure than assertTrue can.
Fixes `MisorderedAssertEqualsParameters` inspection violations introduced in recent commits.
Description:
>Reports any calls to JUnit assertEquals() which have a non-literal as the expected result argument and a literal as the actual result argument. Such calls will behave fine for assertions which pass, but may give confusing error reports if their expected and actual arguments differ.
Fixes `StringToUpperWithoutLocale` inspection violations introduced in recent commits.
Description:
>Reports any call of toUpperCase() or toLowerCase() on String objects which do not specify a java.util.Locale. Such calls are usually incorrect in an internationalized environment.
Fixes `NewExceptionWithoutArguments` inspection violations introduced in recent commits.
Description:
>Reports exception instance creation without any arguments specified. When an exception is constructed without arguments it contains no information about the fault that happened, which makes debugging needlessly hard.
Fixes `TooBroadScope` inspection violations introduced in recent commits.
Description:
>Reports any variable declarations of which the scope can be narrowed. Especially useful for "Pascal style" declarations at the start of a method, but variables with too broad a scope are also often left over after refactorings.
Fixes `UnnecessaryFullyQualifiedName` inspection violations introduced in recent commits.
Description:
>Reports on fully qualified class names which can be shortened. The quick fix for this inspection will shorten the fully qualified names, adding import statements as necessary.
Inspection in this case is right. Parameter `Properties` can be replaced with `Map<Object, Object>`. It works fine with Checkstyle, but it breaks binary compatiblity with Maven Checkstyle Plugin, which is not acceptable.
Michal Kordas
Roman Ivanov
liscju
Vladislav Lisetskiy
Andrei Selkin
Ilja Dubinin