Standardize and improve descriptions of intentions/inspections
This commit is contained in:
Alexey Belkov
committed by
Mikhail Glukhikh
Mikhail Glukhikh
parent
bc7ccbc39b
commit
3b2bbee595
@@ -1,6 +1,6 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports type parameters that can have 'in' or 'out' variance added.
|
||||
Usage of 'in' and 'out' variances leads to more precise type inference in Kotlin.
|
||||
This inspection reports type parameters that can have an <b>in</b> or <b>out</b> variance.
|
||||
Usage of <b>in</b> and <b>out</b> variances leads to more precise type inference in Kotlin.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,6 +1,6 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports array properties in a data class without overridden <code>equals()</code> or <code>hashCode()</code> inside.
|
||||
Due to default <code>equals()</code> behaviour for arrays in JVM, it's strongly recommended to override <code>equals()</code> and <code>hashCode()</code> in such cases.
|
||||
This inspection reports array properties in a data class without overridden <b>equals()</b> or <b>hashCode()</b> inside.
|
||||
Due to default <b>equals()</b> behaviour for arrays in JVM, it is strongly recommended to override <b>equals()</b> and <b>hashCode()</b> in such cases.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,6 +1,6 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports primary constructor parameters that can have 'val' or 'var' removed.
|
||||
Unnecessary usage of 'val' and 'var' in primary constructor consumes unnecessary memory.
|
||||
This inspection reports primary constructor parameters that can have <b>val</b> or <b>var</b> removed.
|
||||
Unnecessary usage of <b>val</b> or <b>var</b> in the primary constructor consumes memory.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports properties which are explicitly assigned to constructor parameters and can be declared directly in constructor instead.
|
||||
This inspection reports properties which are explicitly assigned to constructor parameters and can be declared directly in the constructor instead.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports mutable local variables (declared with 'var' keyword) that can be made immutable.
|
||||
This inspection reports mutable local variables (declared with a <b>var</b> keyword) that can be made immutable.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports if statements with three or more branches that can be replaced with when.
|
||||
This inspection reports <b>if</b> statements with three or more branches that can be replaced with a <b>when</b> expression.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
Reports class names that do not follow the recommended naming conventions.
|
||||
This inspection reports class names that do not follow the recommended naming conventions.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports extension properties that conflict with synthetic ones automatically produced from Java get/set-methods. Those properties should be either removed or renamed to avoid breaking code by future changes in the compiler.
|
||||
This inspection reports extension properties that conflict with synthetic ones automatically produced from Java <b>get/set</b> methods. These properties should be either removed or renamed to avoid breaking code by future changes in the compiler.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
Reports <code>const</code> property names that do not follow the recommended naming conventions.
|
||||
This inspection reports <b>const</b> property names that do not follow the recommended naming conventions.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports 'if' expressions whose condition is constant and therefore the expression can be simplified.
|
||||
This inspection reports <b>if</b> expressions whose condition is constant and therefore the expression can be simplified.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports a lambda (function literal) expression that can be replaced with function reference expression
|
||||
This inspection reports a lambda (function literal) expression that can be replaced with a function reference expression.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports Pair constructor that can be converted to 'to' function.
|
||||
This inspection reports a <b>Pair</b> constructor that can be converted to a <b>to</b> function call.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports a function reference expression that can be replaced with lambda
|
||||
This inspection reports a function reference expression that can be replaced with a lambda.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports secondary constructor that can be replaced with more concise primary constructor
|
||||
This inspection reports a secondary constructor that can be replaced with a more concise primary constructor.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports string concatenation that can be converted to simple string template (one with no "${...}" entries).
|
||||
This inspection reports string concatenation that can be converted to a simple string template (the one with no <b>${...}</b> entries).
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports a try-finally block with resource.close() in finally which can be converted into the resource.use() call
|
||||
This inspection reports a <b>try-finally</b> block with <b>resource.close()</b> in <b>finally</b> which can be converted to a <b>resource.use()</b> call.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports two consecutive comparisons which can be converted to range check
|
||||
This inspection reports two consecutive comparisons which can be converted to a range check.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports calls to a data class' copy method without named arguments.
|
||||
This inspection reports calls to a data class' <b>copy</b> method without named arguments.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports private constructors of data classes because they are always exposed via the generated 'copy' method.
|
||||
This inspection reports private constructors of data classes because they are always exposed via the generated <b>copy</b> method.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,6 +1,6 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports deprecated functions and properties that do not have <i>kotlin.ReplaceWith</i> argument in its <i>kotlin.deprecated</i> annotation
|
||||
and there is a suggestion to add one basing on the body.
|
||||
This inspection reports deprecated functions and properties that do not have a <b>kotlin.ReplaceWith</b> argument in its <b>kotlin.deprecated</b> annotation
|
||||
and there is a suggestion to add one based on the body.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports any declarations that can be destructured
|
||||
This inspection reports declarations that can be destructured.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,6 +1,6 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports different IDE and Gradle plugin versions are used.
|
||||
This can cause inconsistencies between IDE and Gradle build in error reporting or code behaviour.
|
||||
This inspection reports that different IDE and Gradle plugin versions are used.
|
||||
This can cause inconsistencies between IDE and Gradle builds in error reporting or code behaviour.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
Reports different kotlin stdlib and compiler versions that could lead to unpredictable issues
|
||||
This inspection reports different kotlin stdlib and compiler versions that could lead to unpredictable issues.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports ranges that are empty because the 'start' value is greater than the 'endInclusive' value.
|
||||
This inspection reports ranges that are empty because the <b>start</b> value is greater than the <b>endInclusive</b> value.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
Reports enum entry names that do not follow the recommended naming conventions.
|
||||
This inspection reports enum entry names that do not follow the recommended naming conventions.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,6 +1,6 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports classes that override equals() but do not override hashCode(), or vice versa, which could potentially lead to problems when class is added to a Collection.
|
||||
It also reports object declarations which override either equals() or hashCode()
|
||||
This inspection reports classes that override <b>equals()</b> but do not override <b>hashCode()</b>, or vice versa, which could potentially lead to problems when a class is added to a <b>Collection</b>.
|
||||
It also reports object declarations which override either <b>equals()</b> or <b>hashCode()</b>.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports usages of explicit 'this' when it can be omitted.
|
||||
This inspection reports usages of explicit <b>this</b> when it can be omitted.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,16 +1,16 @@
|
||||
<html>
|
||||
<body>
|
||||
<p>
|
||||
This inspection reports non-const Kotlin properties used as Java annotation arguments.
|
||||
It's possible for property which is annotated by <code>@JvmField</code>,
|
||||
has initializer that can be evaluated in compile-time, has primitive or String type.
|
||||
Such properties have ConstantValue attribute in bytecode in Kotlin 1.1-1.2.
|
||||
This attribute allows javac to fold usages of the corresponding field, and use that field in annotations.
|
||||
This inspection reports non-<b>const</b> Kotlin properties used as Java annotation arguments.
|
||||
It is possible for a property which is annotated with <b>@JvmField</b>,
|
||||
has an initializer that can be evaluated at compile-time, has a primitive or <b>String</b> type.
|
||||
Such properties have a <b>ConstantValue</b> attribute in bytecode in Kotlin 1.1-1.2.
|
||||
This attribute allows javac to fold usages of the corresponding field and use that field in annotations.
|
||||
This can cause incorrect behavior in case of separate or incremental compilation in mixed Java/Kotlin code.
|
||||
</p>
|
||||
<p>
|
||||
This behaviour is subject to change in Kotlin 1.3 (no ConstantValue attribute more).
|
||||
Recommended action: replace <code>JvmField</code> annotation with <code>const</code> modifier on relevant Kotlin property or inline it.
|
||||
This behavior is subject to change in Kotlin 1.3 (no <b>ConstantValue</b> attribute any more).
|
||||
Recommended action: replace the <b>@JvmField</b> annotation with a <b>const</b> modifier on a relevant Kotlin property or inline it.
|
||||
</p>
|
||||
|
||||
</body>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports an if expression checking variable being null or not of a type right after initializing it that can be converted into an elvis operator in the initializer
|
||||
This inspection reports an <b>if</b> expression checking variable being null or not right after initializing it that can be converted into an elvis operator in the initializer.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
Reports function names that do not follow the recommended naming conventions.
|
||||
This inspection reports function names that do not follow the recommended naming conventions.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports functions and properties that have platform type. In order to prevent unexpected errors, the type should be declared explicitly.
|
||||
This inspection reports functions and properties that have a platform type. In order to prevent unexpected errors, the type should be declared explicitly.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports any if-then expressions that can be folded into elvis (?:) expressions
|
||||
This inspection reports <b>if-then</b> expressions that can be folded into elvis (<b>?:</b>) expressions.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports any if-then expressions that can be folded into safe-access (?.) expressions
|
||||
This inspection reports <b>if-then</b> expressions that can be folded into safe-access (<b>?.</b>) expressions.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports variables / functions with implicit `Nothing?` type.
|
||||
This inspection reports variables / functions with implicit <b>Nothing?</b> type.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports usages of implicit 'this'.
|
||||
This inspection reports usages of implicit <b>this</b>.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports any 'when' expression that can be simplified by introducing subject argument
|
||||
This inspection reports a <b>when</b> expression that can be simplified by introducing a subject argument.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports Java Collections static method call replaceable by Kotlin stdlib, e.g. Collections.sort(list).
|
||||
This inspection reports a Java <b>Collections</b> static method call replaceable by Kotlin stdlib, e.g. <b>Collections.sort(list)</b>.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
+2
-2
@@ -1,6 +1,6 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection report calls of Java mutator methods (like fill, reverse, shuffle, sort) on immutable Kotlin collection.
|
||||
This will likely produce UnsupportedOperationException at runtime.
|
||||
This inspection report calls of Java mutator methods (like <b>fill</b>, <b>reverse</b>, <b>shuffle</b>, <b>sort</b>) on an immutable Kotlin collection.
|
||||
This will likely produce an <b>UnsupportedOperationException</b> at runtime.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports any property declaration that can be joined with the following assignment
|
||||
This inspection reports property declarations that can be joined with the following assignment.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,6 +1,6 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection is used during the code cleanup operation (Analyze | Code Cleanup) to automatically
|
||||
This inspection is used during the code cleanup operation (<b>Analyze | Code Cleanup</b>) to automatically
|
||||
replace usages of obsolete language features or unnecessarily verbose code constructs with
|
||||
compact and up-to-date syntax.
|
||||
<p>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports redundant double negation usages, like <code>val truth = !!true</code>
|
||||
This inspection reports redundant double negation usages, like <b>val truth = !!true</b>.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports usages of Kotlin internal declarations from Java code in different module.
|
||||
This inspection reports usages of Kotlin internal declarations in Java code from a different module.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports redundant 'override' function which can be omitted.
|
||||
This inspection reports redundant <b>override</b> modifiers which can be omitted.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
Reports any <b>import</b> statements in Kotlin code that are unused.
|
||||
This inspection reports <b>import</b> statements in Kotlin code that are unused.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,24 +1,30 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports dangerous operations inside constructors including:
|
||||
|
||||
<ul>
|
||||
<li>Accessing non-final property in constructor</li>
|
||||
<li>Calling non-final function in constructor</li>
|
||||
<li>Using 'this' as function argument in constructor of non-final class</li>
|
||||
<li>Accessing a non-final property in constructor</li>
|
||||
<li>Calling a non-final function in constructor</li>
|
||||
<li>Using <b>this</b> as a function argument in a constructor of a non-final class</li>
|
||||
</ul>
|
||||
|
||||
These operations are dangerous because your class can be inherited,
|
||||
and derived class is not yet initialized at this moment. Typical example:
|
||||
<code><pre>
|
||||
and a derived class is not yet initialized at this moment. Typical example:
|
||||
<br /><br />
|
||||
|
||||
<pre>
|
||||
<b>abstract class</b> Base {
|
||||
<b>val</b> code = calculate()
|
||||
<b>abstract fun</b> calculate(): Int
|
||||
}
|
||||
|
||||
<b>class</b> Derived(<b>private val</b> x: Int) : Base() {
|
||||
<b>override fun</b> calculate() = x
|
||||
}
|
||||
|
||||
<b>fun</b> testIt() {
|
||||
println(Derived(42).code) <i>// Expected: 42, actual: 0</i>
|
||||
}
|
||||
</pre></code>
|
||||
</pre>
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,14 +1,16 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports if, when and try statements that can be converted to expressions
|
||||
by lifting return or assignment out. Typical example:
|
||||
<code><pre>
|
||||
This inspection reports <b>if</b>, <b>when</b> and <b>try</b> statements that can be converted to expressions
|
||||
by lifting a return or an assignment out. Typical example:
|
||||
<br /><br />
|
||||
|
||||
<pre>
|
||||
<b>fun</b> foo(arg: Boolean): String {
|
||||
<b>when</b> (arg) {
|
||||
<b>true</b> -> <b>return</b> "Truth"
|
||||
<b>false</b> -> <b>return</b> "Falsehood"
|
||||
}
|
||||
}
|
||||
</pre></code>
|
||||
</pre>
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
Reports local variable names that do not follow the recommended naming conventions.
|
||||
This inspection reports local variable names that do not follow the recommended naming conventions.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports for-loops that can be replaced with a sequence of stdlib-operations (like "map", "filter" etc)
|
||||
This inspection reports <b>for</b> loops that can be replaced with a sequence of stdlib operations (like <b>map</b>, <b>filter</b>, etc).
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,6 +1,6 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports object and top-level <code>val</code> that might be declared as <code>const</code>
|
||||
This inspection reports an object and a top-level <b>val</b> that might be declared as <b>const</b>
|
||||
for better performance and Java interoperability.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports class members which can be made private
|
||||
This inspection reports class members which can be made private.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,8 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports suppression with old diagnostic names, for example:
|
||||
<code><pre>
|
||||
@Suppress("HEADER_WITHOUT_IMPLEMENTATION")
|
||||
</pre></code>
|
||||
This inspection reports suppressions with old diagnostic names, for example <b>@Suppress("HEADER_WITHOUT_IMPLEMENTATION")</b>.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports when chained null-checks can be replaced with safe-calls
|
||||
This inspection reports when chained null-checks can be replaced with safe-calls.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports when equality check should be used instead of elvis
|
||||
This inspection reports when an equality check should be used instead of the elvis operator.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports an anonymous object literal implementing a Java interface with single abstract method that can be converted into call with lambda expression.
|
||||
This inspection reports an anonymous object literal implementing a Java interface with a single abstract method that can be converted into a call with a lambda expression.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
Reports names of properties in objects and companion objects, as well as top-level properties, that do not follow the recommended naming conventions.
|
||||
This inspection reports names of properties in objects and companion objects, as well as top-level properties, that do not follow the recommended naming conventions.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
Reports file's package directive not matching location of the file
|
||||
This inspection reports file's <b>package</b> directive not matching the location of the file.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
Reports package names that do not follow the recommended naming conventions.
|
||||
This inspection reports package names that do not follow the recommended naming conventions.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,9 +1,9 @@
|
||||
<html>
|
||||
<body>
|
||||
Reports potentially unsafe calls of inline functions with flexible nullable (platform type with unknown nullability) extension receivers.
|
||||
This inspection reports potentially unsafe calls of inline functions with flexible nullable (platform type with unknown nullability) extension receivers.
|
||||
In Kotlin 1.0 or 1.1, such calls do not include nullability check in bytecode,
|
||||
but in Kotlin 1.2, nullability check is included and can provoke NPE if actual receiver is null.
|
||||
It's recommended to add explicit '!!' if exception is what you want,
|
||||
or consider changing the function' receiver type to nullable if it should work without exceptions.
|
||||
It's recommended to add explicit <b>!!</b> if exception is what you want,
|
||||
or consider changing the function's receiver type to nullable if it should work without exceptions.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
Reports private property names that do not follow the recommended naming conventions.
|
||||
This inspection reports private property names that do not follow the recommended naming conventions.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
Reports property names that do not follow the recommended naming conventions.
|
||||
This inspection reports property names that do not follow the recommended naming conventions.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,6 +1,6 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports <code>protected</code> visibility used inside of a non-inheritable class.
|
||||
<code>protected</code> members is accessible only in the class itself in such a case, so they are effectively <code>private</code>.
|
||||
This inspection reports <b>protected</b> visibility used inside of a non-inheritable class.
|
||||
In such cases <b>protected</b> members are accessible only in the class itself, so they are effectively <b>private</b>.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,10 +1,8 @@
|
||||
<html>
|
||||
<body>
|
||||
Reports recursive equals calls.
|
||||
This inspection reports recursive <b>equals</b> calls. For example:
|
||||
<br /><br />
|
||||
|
||||
<p>
|
||||
For example:
|
||||
<code>
|
||||
<pre>
|
||||
class X {
|
||||
override fun equals(other: Any?): Boolean {
|
||||
@@ -13,8 +11,5 @@ class X {
|
||||
}
|
||||
}
|
||||
</pre>
|
||||
</code>
|
||||
</p>
|
||||
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,23 +1,22 @@
|
||||
<html>
|
||||
<body>
|
||||
Reports recursive property accessor calls which can end up with StackOverflowError
|
||||
<p>
|
||||
For example:
|
||||
<code><pre>
|
||||
<b>class</b> A {
|
||||
<b>var</b> x = 0
|
||||
<b>get</b>() {
|
||||
<b>return</b> x //recursive getter call
|
||||
}
|
||||
This inspection reports recursive property accessor calls which can end up with a <b>StackOverflowError</b>. For example:
|
||||
<br /><br />
|
||||
|
||||
<b>var</b> y = 0
|
||||
<b>set</b>(value) {
|
||||
<b>if</b> (value > 0) {
|
||||
y = value //recursive setter call
|
||||
}
|
||||
<pre>
|
||||
<b>class</b> A {
|
||||
<b>var</b> x = 0
|
||||
<b>get</b>() {
|
||||
<b>return</b> x //recursive getter call
|
||||
}
|
||||
|
||||
<b>var</b> y = 0
|
||||
<b>set</b>(value) {
|
||||
<b>if</b> (value > 0) {
|
||||
y = value //recursive setter call
|
||||
}
|
||||
}
|
||||
</pre></code>
|
||||
</p>
|
||||
}
|
||||
}
|
||||
</pre>
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,8 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports local variables' explicitly given types which are obvious and thus redundant, like
|
||||
<pre><code>
|
||||
val f: Foo = Foo()
|
||||
</code></pre>
|
||||
This inspection reports local variables' explicitly given types which are obvious and thus redundant, like <b>val f: Foo = Foo()</b>.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports redundant property getter.
|
||||
This inspection reports redundant property getters.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,21 +1,17 @@
|
||||
<html>
|
||||
<body>
|
||||
Reports <b>if</b> statements which can be simplified to single statements.
|
||||
<p>
|
||||
For example:
|
||||
<code><pre>
|
||||
<b>if</b> (foo()) {
|
||||
<b>return true</b>
|
||||
} <b>else</b> {
|
||||
<b>return false</b>
|
||||
}
|
||||
</pre></code>
|
||||
can be simplified to
|
||||
<code><pre>
|
||||
<b>return</b> foo()
|
||||
</pre></code>
|
||||
<!-- tooltip end -->
|
||||
<p>
|
||||
This inspection reports <b>if</b> statements which can be simplified to single statements. For example:
|
||||
<br /><br />
|
||||
|
||||
<pre>
|
||||
<b>if</b> (foo()) {
|
||||
<b>return true</b>
|
||||
} <b>else</b> {
|
||||
<b>return false</b>
|
||||
}
|
||||
</pre>
|
||||
|
||||
<p>can be simplified to <b>return foo()</b>.</p>
|
||||
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,6 +1,6 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports modality modifiers which match the default modality of an element
|
||||
(<code>final</code> for most elements, <code>open</code> for members with override).
|
||||
(<b>final</b> for most elements, <b>open</b> for members with an <b>override</b>).
|
||||
</body>
|
||||
</html>
|
||||
|
||||
+2
-2
@@ -1,7 +1,7 @@
|
||||
<html>
|
||||
<body>
|
||||
Reports inline functions with not-null extension receivers which does not use the fact that extension receiver is not null.
|
||||
This function are dangerous to call in Kotlin 1.2 on actual nullable flexible receiver type.
|
||||
This inspection reports inline functions with not-null extension receivers which does not use the fact that extension receiver is not null.
|
||||
Such functions are dangerous to call in Kotlin 1.2 on actual nullable flexible receiver type.
|
||||
Consider making receiver type nullable.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports redundant type checks for object.
|
||||
This inspection reports redundant type checks for an object.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports redundant semicolon (';') token which is not required in Kotlin and may be removed.
|
||||
This inspection reports redundant semicolon (<b>;</b>) tokens which are not required in Kotlin and may be removed.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports redundant property setter.
|
||||
This inspection reports redundant property setters.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports 'suspend' modifier as redundant if no other suspend functions are called inside
|
||||
This inspection reports a <b>suspend</b> modifier as redundant if no other suspend functions are called inside.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports redundant 'Unit' which can be omitted.
|
||||
This inspection reports a redundant <b>Unit</b> type specification which can be omitted.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports redundant 'Unit' return type which can be omitted.
|
||||
This inspection reports a redundant <b>Unit</b> return type which can be omitted.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,6 +1,6 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports visibility modifiers which match the default visibility of an element
|
||||
(<code>public</code> for most elements, <code>protected</code> for members that override a protected member).
|
||||
(<b>public</b> for most elements, <b>protected</b> for members that override a protected member).
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports and offers to remove an empty class body
|
||||
This inspection reports and offers to remove an empty class body.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection detects an empty primary constructor when it would be implicitly available anyway
|
||||
This inspection detects an empty primary constructor when it would be implicitly available anyway.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection detects empty bodies of secondary constructors
|
||||
This inspection detects empty bodies of secondary constructors.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports any super member call with redundant supertype qualification
|
||||
This inspection reports <b>super</b> member calls with redundant supertype qualification.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports any function call with type arguments that can be safely removed and inferred
|
||||
This inspection reports function calls with type arguments that can be safely removed and inferred.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports for loops iterating over a collection of values using "withIndex()" function with index variable not used in the loop body.
|
||||
This inspection reports <b>for</b> loops iterating over a collection of values using a <b>withIndex()</b> function with index variable not used in the loop body.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports the redundant backticks in references
|
||||
This inspection reports redundant backticks in references.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports redundant calls of the conversion method
|
||||
This inspection reports redundant calls of conversion methods (for example, <b>toString()</b> on a <b>String</b>).
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports the redundant spread operator for arrayOf call
|
||||
This inspection reports the use of a redundant spread operator for a family of <b>arrayOf</b> function calls.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,6 +1,6 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports explicitly given parameter types in property setters.
|
||||
Setter parameter type always matches property type, so it's not needed to be explicit.
|
||||
A setter parameter type always matches the property type, so it's not required to be explicit.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports single-expression String template that can be safely removed
|
||||
This inspection reports single-expression string templates that can be safely removed.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports calls to 'toString()' in String templates that can be safely removed
|
||||
This inspection reports calls to <b>toString()</b> in string templates that can be safely removed.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection detects '==' or '!=' operator for arrays that should be replaced with 'contentEquals'
|
||||
This inspection detects usages of <b>==</b> or <b>!=</b> operator for arrays that should be replaced with <b>contentEquals</b>.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports 'arrayOf' calls that can be replaced with array literals [...].
|
||||
This inspection reports <b>arrayOf</b> calls that can be replaced with array literals <b>[...]</b>.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,4 +1,6 @@
|
||||
<html><body>
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports function calls that can be replaced with binary operators, especially comparison-related.
|
||||
Example: <code>2.compareTo(1) > 0</code> can be replaced by <code>2 > 1</code>.
|
||||
</body></html>
|
||||
Example: <b>2.compareTo(1) > 0</b> can be replaced by <b>2 > 1</b>.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports any explicit calls of <code>get</code> or <code>set</code> functions which can be replaced by indexing operator <code>[]</code>
|
||||
This inspection reports explicit calls of <b>get</b> or <b>set</b> functions which can be replaced by an indexing operator <b>[]</b>.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports <code>map.put</code> function calls replaceable with the indexing operator (<code>[]</code>).
|
||||
This inspection reports <b>map.put</b> function calls replaceable with the indexing operator (<b>[]</b>).
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports calls to 'rangeTo' or the '..' operator instead of calls to 'until'.
|
||||
This inspection reports calls to <b>rangeTo</b> or the <b>..</b> operator instead of calls to <b>until</b>.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection detects redundant '.let', when it includes only one call with lambda parameter as receiver.
|
||||
This inspection detects a redundant <b>let</b> when it includes only one call with lambda parameter as receiver.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection detects size checks of Collections/Array/String that should be replaced with 'isNotEmpty()'
|
||||
This inspection detects size checks of <b>Collections/Array/String</b> that should be replaced with <b>isNotEmpty()</b>.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection detects size zero checks of Collections/Array/String that should be replaced with 'isEmpty()'
|
||||
This inspection detects <b>size == 0</b> checks of <b>Collections/Array/String</b> that should be replaced with <b>isEmpty()</b>.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports <code>to</code> function calls replaceable with the infix form
|
||||
This inspection reports <b>to</b> function calls replaceable with the infix form.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,4 +1,6 @@
|
||||
<html><body>
|
||||
This inspection reports modifications of variables with a simple assignment (such as <code>y = y + x</code>)
|
||||
<html>
|
||||
<body>
|
||||
This inspection reports modifications of variables with a simple assignment (such as <b>y = y + x</b>)
|
||||
that can be replaced with an operator assignment.
|
||||
</body></html>
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
Provides actions for converting scope functions ('let', 'run', 'apply', 'also') between each other.
|
||||
This inspection provides actions for converting scope functions (<b>let</b>, <b>run</b>, <b>apply</b>, <b>also</b>) between each other.
|
||||
</body>
|
||||
</html>
|
||||
@@ -1,5 +1,5 @@
|
||||
<html>
|
||||
<body>
|
||||
This inspection detects assignments of a variable to itself
|
||||
This inspection detects assignments of a variable to itself.
|
||||
</body>
|
||||
</html>
|
||||
|
||||
Some files were not shown because too many files have changed in this diff Show More
Reference in New Issue
Block a user