Fix spaces in labeled expression (KT-14131)
https://github.com/JetBrains/kotlin/pull/978 (cherry picked from commit fdd5ba7)
This commit is contained in:
committed by
Nikolay Krasko
parent
bdecb661e3
commit
3cc3ad2b9e
@@ -156,6 +156,9 @@ fun createSpacingBuilder(settings: CodeStyleSettings, builderUtil: KotlinSpacing
|
|||||||
betweenInside(RETURN_KEYWORD, LABEL_QUALIFIER, RETURN).spaces(0)
|
betweenInside(RETURN_KEYWORD, LABEL_QUALIFIER, RETURN).spaces(0)
|
||||||
afterInside(RETURN_KEYWORD, RETURN).spaces(1)
|
afterInside(RETURN_KEYWORD, RETURN).spaces(1)
|
||||||
afterInside(LABEL_QUALIFIER, RETURN).spaces(1)
|
afterInside(LABEL_QUALIFIER, RETURN).spaces(1)
|
||||||
|
betweenInside(LABEL_QUALIFIER, EOL_COMMENT, LABELED_EXPRESSION).spacing(0, Int.MAX_VALUE, 0, true, codeStyleSettings.KEEP_BLANK_LINES_IN_CODE)
|
||||||
|
betweenInside(LABEL_QUALIFIER, BLOCK_COMMENT, LABELED_EXPRESSION).spacing(0, Int.MAX_VALUE, 0, true, codeStyleSettings.KEEP_BLANK_LINES_IN_CODE)
|
||||||
|
afterInside(LABEL_QUALIFIER, LABELED_EXPRESSION).spaces(1)
|
||||||
|
|
||||||
betweenInside(FUN_KEYWORD, VALUE_PARAMETER_LIST, FUN).spacing(0, 0, 0, false, 0)
|
betweenInside(FUN_KEYWORD, VALUE_PARAMETER_LIST, FUN).spacing(0, 0, 0, false, 0)
|
||||||
after(FUN_KEYWORD).spaces(1)
|
after(FUN_KEYWORD).spaces(1)
|
||||||
|
|||||||
@@ -0,0 +1,40 @@
|
|||||||
|
fun test1() {
|
||||||
|
loop@ while (true) {
|
||||||
|
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
fun test2() {
|
||||||
|
listOf(1).forEach lit@ {
|
||||||
|
if (it == 0) return@lit
|
||||||
|
print(it)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
fun test3() {
|
||||||
|
l1@
|
||||||
|
|
||||||
|
|
||||||
|
/*comment*/ {
|
||||||
|
|
||||||
|
}
|
||||||
|
|
||||||
|
l2@ /*comment*/ {
|
||||||
|
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
fun test4() {
|
||||||
|
l1@
|
||||||
|
|
||||||
|
|
||||||
|
//eol comment
|
||||||
|
{
|
||||||
|
|
||||||
|
}
|
||||||
|
|
||||||
|
l2@ //eol comment
|
||||||
|
{
|
||||||
|
|
||||||
|
}
|
||||||
|
}
|
||||||
+40
@@ -0,0 +1,40 @@
|
|||||||
|
fun test1() {
|
||||||
|
loop@ while(true) {
|
||||||
|
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
fun test2() {
|
||||||
|
listOf(1).forEach lit@ {
|
||||||
|
if (it == 0) return@lit
|
||||||
|
print(it)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
fun test3() {
|
||||||
|
l1@
|
||||||
|
|
||||||
|
|
||||||
|
/*comment*/ {
|
||||||
|
|
||||||
|
}
|
||||||
|
|
||||||
|
l2@ /*comment*/ {
|
||||||
|
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
fun test4() {
|
||||||
|
l1@
|
||||||
|
|
||||||
|
|
||||||
|
//eol comment
|
||||||
|
{
|
||||||
|
|
||||||
|
}
|
||||||
|
|
||||||
|
l2@ //eol comment
|
||||||
|
{
|
||||||
|
|
||||||
|
}
|
||||||
|
}
|
||||||
+1
-1
@@ -1,6 +1,6 @@
|
|||||||
// IS_APPLICABLE: true
|
// IS_APPLICABLE: true
|
||||||
fun foo() {
|
fun foo() {
|
||||||
bar(l@{ it * 3 })
|
bar(l@ { it * 3 })
|
||||||
}
|
}
|
||||||
|
|
||||||
fun bar(b: (Int) -> Int) {
|
fun bar(b: (Int) -> Int) {
|
||||||
|
|||||||
@@ -511,6 +511,12 @@ public class FormatterTestGenerated extends AbstractFormatterTest {
|
|||||||
doTest(fileName);
|
doTest(fileName);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@TestMetadata("LabeledExpression.after.kt")
|
||||||
|
public void testLabeledExpression() throws Exception {
|
||||||
|
String fileName = KotlinTestUtils.navigationMetadata("idea/testData/formatter/LabeledExpression.after.kt");
|
||||||
|
doTest(fileName);
|
||||||
|
}
|
||||||
|
|
||||||
@TestMetadata("RightBracketOnNewLine.after.kt")
|
@TestMetadata("RightBracketOnNewLine.after.kt")
|
||||||
public void testRightBracketOnNewLine() throws Exception {
|
public void testRightBracketOnNewLine() throws Exception {
|
||||||
String fileName = KotlinTestUtils.navigationMetadata("idea/testData/formatter/RightBracketOnNewLine.after.kt");
|
String fileName = KotlinTestUtils.navigationMetadata("idea/testData/formatter/RightBracketOnNewLine.after.kt");
|
||||||
|
|||||||
Reference in New Issue
Block a user