TrailingCommaInspection: should suggest fixes for call-site without warnings

#KT-39131 Fixed
This commit is contained in:
Dmitry Gridin
2020-05-22 19:39:55 +07:00
parent 10462c31d2
commit c78fd4aaca
28 changed files with 22396 additions and 6851 deletions
File diff suppressed because it is too large Load Diff
File diff suppressed because it is too large Load Diff
File diff suppressed because it is too large Load Diff
File diff suppressed because it is too large Load Diff
File diff suppressed because it is too large Load Diff
@@ -0,0 +1,8 @@
// COMPILER_ARGUMENTS: -XXLanguage:+TrailingCommas
// FIX: Add line break
// DISABLE-ERRORS
fun a() {
val (a,
b,<caret>) = 1 to 2
}
@@ -0,0 +1,10 @@
// COMPILER_ARGUMENTS: -XXLanguage:+TrailingCommas
// FIX: Add line break
// DISABLE-ERRORS
fun a() {
val (
a,
b,
) = 1 to 2
}
@@ -0,0 +1,8 @@
// COMPILER_ARGUMENTS: -XXLanguage:+TrailingCommas
// FIX: Fix comma position
// DISABLE-ERRORS
fun a() {
val a = { (a, b // awd
,<caret>/**/), c, -> }
}
@@ -0,0 +1,10 @@
// COMPILER_ARGUMENTS: -XXLanguage:+TrailingCommas
// FIX: Fix comma position
// DISABLE-ERRORS
fun a() {
val a = {
(
a, b, // awd/**/
), c, -> }
}
@@ -0,0 +1,8 @@
// COMPILER_ARGUMENTS: -XXLanguage:+TrailingCommas
// FIX: Add trailing comma
// DISABLE-ERRORS
fun a() {
b(1, 3, 2424,
awdawd<caret>)
}
@@ -0,0 +1,10 @@
// COMPILER_ARGUMENTS: -XXLanguage:+TrailingCommas
// FIX: Add trailing comma
// DISABLE-ERRORS
fun a() {
b(
1, 3, 2424,
awdawd,
)
}
@@ -0,0 +1,8 @@
// COMPILER_ARGUMENTS: -XXLanguage:+TrailingCommas
// FIX: Add line break
// DISABLE-ERRORS
fun a() {
g[<caret>1, 2, 3,/**/
]
}
@@ -0,0 +1,9 @@
// COMPILER_ARGUMENTS: -XXLanguage:+TrailingCommas
// FIX: Add line break
// DISABLE-ERRORS
fun a() {
g[
1, 2, 3,/**/
]
}
+7
View File
@@ -0,0 +1,7 @@
// COMPILER_ARGUMENTS: -XXLanguage:+TrailingCommas
// FIX: Add line break
fun a(
a: Int, b: Any = fun(a: Int,) {},<caret>) {
}
@@ -0,0 +1,8 @@
// COMPILER_ARGUMENTS: -XXLanguage:+TrailingCommas
// FIX: Add line break
fun a(
a: Int, b: Any = fun(a: Int) {},
) {
}
@@ -0,0 +1,9 @@
// COMPILER_ARGUMENTS: -XXLanguage:+TrailingCommas
// FIX: Add line break
fun a() {
val a = { a: Int,
b: Int, <caret>->
}
}
@@ -0,0 +1,11 @@
// COMPILER_ARGUMENTS: -XXLanguage:+TrailingCommas
// FIX: Add line break
fun a() {
val a = {
a: Int,
b: Int,
->
}
}
@@ -0,0 +1,8 @@
// COMPILER_ARGUMENTS: -XXLanguage:+TrailingCommas
// FIX: Add line break
// DISABLE-ERRORS
@Anno(
[<caret>1, 2, 3,
])
class A
@@ -0,0 +1,9 @@
// COMPILER_ARGUMENTS: -XXLanguage:+TrailingCommas
// FIX: Add line break
// DISABLE-ERRORS
@Anno(
[
1, 2, 3,
])
class A
@@ -0,0 +1,7 @@
// COMPILER_ARGUMENTS: -XXLanguage:+TrailingCommas
// PROBLEM: none
fun a(<caret>
a: Int,) {
}
@@ -0,0 +1,8 @@
// COMPILER_ARGUMENTS: -XXLanguage:+TrailingCommas
// FIX: Add line break
// DISABLE-ERRORS
fun a() {
b<<caret>Int,
>()
}
@@ -0,0 +1,9 @@
// COMPILER_ARGUMENTS: -XXLanguage:+TrailingCommas
// FIX: Add line break
// DISABLE-ERRORS
fun a() {
b<
Int,
>()
}
@@ -0,0 +1,5 @@
// COMPILER_ARGUMENTS: -XXLanguage:+TrailingCommas
// FIX: Add line break
fun <<caret>T,
B,> a() = Unit
@@ -0,0 +1,7 @@
// COMPILER_ARGUMENTS: -XXLanguage:+TrailingCommas
// FIX: Add line break
fun <
T,
B,
> a() = Unit
@@ -0,0 +1,9 @@
// COMPILER_ARGUMENTS: -XXLanguage:+TrailingCommas
// FIX: Add line break
fun a() {
when (val b = 5) {
1, 2,
3, <caret>->
}
}
@@ -0,0 +1,10 @@
// COMPILER_ARGUMENTS: -XXLanguage:+TrailingCommas
// FIX: Add line break
fun a() {
when (val b = 5) {
1, 2,
3,
->
}
}