Fixed error message to quote parameter name

Author: valentinkip

compiler/frontend/src/org/jetbrains/kotlin/diagnostics/rendering/DefaultErrorMessages.java

@@ -721,7 +721,7 @@ public static DiagnosticRenderer getRendererForDiagnostic(@NotNull Diagnostic di
         MAP.put(CANNOT_COMPLETE_RESOLVE, "Cannot choose among the following candidates without completing type inference: {0}", AMBIGUOUS_CALLS);
         MAP.put(UNRESOLVED_REFERENCE_WRONG_RECEIVER, "Unresolved reference. None of the following candidates is applicable because of receiver type mismatch: {0}", AMBIGUOUS_CALLS);
 
-        MAP.put(NO_VALUE_FOR_PARAMETER, "No value passed for parameter {0}", NAME);
+        MAP.put(NO_VALUE_FOR_PARAMETER, "No value passed for parameter ''{0}''", NAME);
         MAP.put(MISSING_RECEIVER, "A receiver of type {0} is required", RENDER_TYPE);
         MAP.put(NO_RECEIVER_ALLOWED, "No receiver can be passed to this function or property");
 

idea/testData/copyPaste/imports/KT10433.expected.kt

@@ -1,5 +1,5 @@
 // ERROR: Unresolved reference: vfff
-// ERROR: No value passed for parameter text
+// ERROR: No value passed for parameter 'text'
 package p
 
 fun g() {

idea/testData/copyPaste/imports/NoImportForSamePackage.expected.kt

@@ -1,4 +1,4 @@
-// ERROR: No value passed for parameter i
+// ERROR: No value passed for parameter 'i'
 package a
 
 private fun f(p: A, t: T) {

idea/testData/intentions/addNameToArgument/incompleteCall.kt

@@ -1,4 +1,4 @@
-// ERROR: No value passed for parameter p
+// ERROR: No value passed for parameter 'p'
 
 fun foo(s: String, b: Boolean, p: Int){}
 

idea/testData/intentions/addNameToArgument/incompleteCall.kt.after

@@ -1,4 +1,4 @@
-// ERROR: No value passed for parameter p
+// ERROR: No value passed for parameter 'p'
 
 fun foo(s: String, b: Boolean, p: Int){}
 

idea/testData/intentions/addNamesToCallArguments/incompleteCall.kt

@@ -1,4 +1,4 @@
-// ERROR: No value passed for parameter p
+// ERROR: No value passed for parameter 'p'
 
 fun foo(s: String, b: Boolean, p: Int){}
 

idea/testData/intentions/addNamesToCallArguments/incompleteCall.kt.after

@@ -1,4 +1,4 @@
-// ERROR: No value passed for parameter p
+// ERROR: No value passed for parameter 'p'
 
 fun foo(s: String, b: Boolean, p: Int){}
 

idea/testData/intentions/conventionNameCalls/replaceContains/missingArgument.kt

@@ -1,5 +1,5 @@
 // IS_APPLICABLE: false
-// ERROR: No value passed for parameter b
+// ERROR: No value passed for parameter 'b'
 // ERROR: 'operator' modifier is inapplicable on this function: must have a single value parameter
 fun test() {
     class Test{

idea/testData/intentions/conventionNameCalls/replaceGetOrSet/duplicateArguments.kt

@@ -1,6 +1,6 @@
 // IS_APPLICABLE: false
 // ERROR: An argument is already passed for this parameter
-// ERROR: No value passed for parameter b
+// ERROR: No value passed for parameter 'b'
 fun test() {
     class Test{
         operator fun get(a: Int, b: Int) : Int = 0

idea/testData/intentions/moveLambdaOutsideParentheses/moveLambda3.kt

@@ -1,6 +1,6 @@
 // IS_APPLICABLE: true
 // ERROR: Type mismatch: inferred type is () -> ??? but Int was expected
-// ERROR: No value passed for parameter b
+// ERROR: No value passed for parameter 'b'
 // ERROR: Unresolved reference: it
 // SKIP_ERRORS_AFTER
 fun foo() {

idea/testData/intentions/moveLambdaOutsideParentheses/moveLambda3.kt.after

@@ -1,6 +1,6 @@
 // IS_APPLICABLE: true
 // ERROR: Type mismatch: inferred type is () -> ??? but Int was expected
-// ERROR: No value passed for parameter b
+// ERROR: No value passed for parameter 'b'
 // ERROR: Unresolved reference: it
 // SKIP_ERRORS_AFTER
 fun foo() {

idea/testData/intentions/removeExplicitSuperQualifier/IncompleteCall.kt

@@ -1,4 +1,4 @@
-// ERROR: No value passed for parameter p
+// ERROR: No value passed for parameter 'p'
 
 open class B {
     open fun foo(p: String){}

idea/testData/intentions/removeExplicitSuperQualifier/IncompleteCall.kt.after

@@ -1,4 +1,4 @@
-// ERROR: No value passed for parameter p
+// ERROR: No value passed for parameter 'p'
 
 open class B {
     open fun foo(p: String){}

idea/testData/intentions/toInfixCall/zeroArguments.kt

@@ -1,5 +1,5 @@
 // IS_APPLICABLE: false
-// ERROR: No value passed for parameter p
+// ERROR: No value passed for parameter 'p'
 infix fun Int.xxx(p: Int) = 1
 
 fun foo(x: Int) {

idea/testData/quickfix/autoImports/mismatchingArgs/expectedTypeRequired.test

@@ -1,6 +1,6 @@
 // FILE: first.before.kt
 // "Import" "true"
-// ERROR: No value passed for parameter p
+// ERROR: No value passed for parameter 'p'
 // ERROR: Type mismatch: inferred type is Unit but String was expected
 
 package main
@@ -23,7 +23,7 @@ fun <T> X.foo(): T = TODO()
 
 // FILE: first.after.kt
 // "Import" "true"
-// ERROR: No value passed for parameter p
+// ERROR: No value passed for parameter 'p'
 // ERROR: Type mismatch: inferred type is Unit but String was expected
 
 package main

idea/testData/quickfix/changeSignature/notEditable.kt

@@ -1,5 +1,5 @@
 // "class org.jetbrains.kotlin.idea.quickfix.ChangeFunctionSignatureFix" "false"
-//ERROR: No value passed for parameter other
+//ERROR: No value passed for parameter 'other'
 
 abstract class StringComparable {
     public fun compareTo(other: String): Int = 0

idea/testData/quickfix/changeSignature/removeParameterNotAvailableForBuiltins.kt

@@ -1,5 +1,5 @@
 // "class org.jetbrains.kotlin.idea.quickfix.ChangeFunctionSignatureFix" "false"
-// ERROR: No value passed for parameter other
+// ERROR: No value passed for parameter 'other'
 
 fun f(d: Boolean) {
     d.or(<caret>)

idea/testData/quickfix/createFromUsage/createClass/callExpression/callWithMissingArgs.kt

@@ -3,7 +3,7 @@
 // ACTION: Remove parameter 's'
 // ACTION: Create secondary constructor
 // ACTION: Add 'i =' to argument
-// ERROR: No value passed for parameter s
+// ERROR: No value passed for parameter 's'
 
 class Foo(i: Int, s: String)
 

idea/testData/quickfix/createFromUsage/createClass/callExpression/callWithSuperclassWithConstructorParams.kt

@@ -1,5 +1,5 @@
 // "Create class 'Foo'" "true"
-// ERROR: No value passed for parameter n
+// ERROR: No value passed for parameter 'n'
 
 open class A(n: Int)
 

idea/testData/quickfix/createFromUsage/createClass/callExpression/callWithSuperclassWithConstructorParams.kt.after

@@ -1,5 +1,5 @@
 // "Create class 'Foo'" "true"
-// ERROR: No value passed for parameter n
+// ERROR: No value passed for parameter 'n'
 
 open class A(n: Int)
 

idea/testData/quickfix/createFromUsage/createClass/referenceExpression/enumEntryWithEnumQualifierAndParams.kt

@@ -1,5 +1,5 @@
 // "Create enum constant 'A'" "true"
-// ERROR: No value passed for parameter n
+// ERROR: No value passed for parameter 'n'
 package p
 
 fun foo() = X.<caret>A

idea/testData/quickfix/createFromUsage/createClass/referenceExpression/enumEntryWithEnumQualifierAndParams.kt.after

@@ -1,5 +1,5 @@
 // "Create enum constant 'A'" "true"
-// ERROR: No value passed for parameter n
+// ERROR: No value passed for parameter 'n'
 package p
 
 fun foo() = X.A

idea/testData/quickfix/createFromUsage/createSecondaryConstructor/noParametersSuperCall.kt

@@ -1,6 +1,6 @@
 // "Create secondary constructor" "true"
 // SHOULD_BE_AVAILABLE_AFTER_EXECUTION
-// ERROR: No value passed for parameter f
+// ERROR: No value passed for parameter 'f'
 open class Base(val f: Int)
 
 class Creation: Base {

idea/testData/quickfix/createFromUsage/createSecondaryConstructor/noParametersSuperCall.kt.after

@@ -1,6 +1,6 @@
 // "Create secondary constructor" "true"
 // SHOULD_BE_AVAILABLE_AFTER_EXECUTION
-// ERROR: No value passed for parameter f
+// ERROR: No value passed for parameter 'f'
 open class Base(val f: Int)
 
 class Creation: Base {

idea/testData/quickfix/createFromUsage/createVariable/parameter/inPropertyInitializerInEnumClass.kt

@@ -1,7 +1,7 @@
 // "Create parameter 'foo'" "true"
-// ERROR: No value passed for parameter foo
-// ERROR: No value passed for parameter foo
-// ERROR: No value passed for parameter foo
+// ERROR: No value passed for parameter 'foo'
+// ERROR: No value passed for parameter 'foo'
+// ERROR: No value passed for parameter 'foo'
 
 enum class E {
     A,

idea/testData/quickfix/createFromUsage/createVariable/parameter/inPropertyInitializerInEnumClass.kt.after

@@ -1,7 +1,7 @@
 // "Create parameter 'foo'" "true"
-// ERROR: No value passed for parameter foo
-// ERROR: No value passed for parameter foo
-// ERROR: No value passed for parameter foo
+// ERROR: No value passed for parameter 'foo'
+// ERROR: No value passed for parameter 'foo'
+// ERROR: No value passed for parameter 'foo'
 
 enum class E(foo: Int) {
     A(),

idea/testData/quickfix/createFromUsage/createVariable/parameter/inPropertyInitializerInEnumEntry.kt

@@ -1,7 +1,7 @@
 // "Create property 'foo' as constructor parameter" "true"
-// ERROR: No value passed for parameter foo
-// ERROR: No value passed for parameter foo
-// ERROR: No value passed for parameter foo
+// ERROR: No value passed for parameter 'foo'
+// ERROR: No value passed for parameter 'foo'
+// ERROR: No value passed for parameter 'foo'
 
 enum class E {
     A,

idea/testData/quickfix/createFromUsage/createVariable/parameter/inPropertyInitializerInEnumEntry.kt.after

@@ -1,7 +1,7 @@
 // "Create property 'foo' as constructor parameter" "true"
-// ERROR: No value passed for parameter foo
-// ERROR: No value passed for parameter foo
-// ERROR: No value passed for parameter foo
+// ERROR: No value passed for parameter 'foo'
+// ERROR: No value passed for parameter 'foo'
+// ERROR: No value passed for parameter 'foo'
 
 enum class E(val foo: Int) {
     A(),

idea/testData/quickfix/createFromUsage/createVariable/primaryParameter/valOnUserType.kt

@@ -1,5 +1,5 @@
 // "Create property 'foo' as constructor parameter" "true"
-// ERROR: No value passed for parameter foo
+// ERROR: No value passed for parameter 'foo'
 
 class A<T>(val n: T)
 

idea/testData/quickfix/createFromUsage/createVariable/primaryParameter/valOnUserType.kt.after

@@ -1,5 +1,5 @@
 // "Create property 'foo' as constructor parameter" "true"
-// ERROR: No value passed for parameter foo
+// ERROR: No value passed for parameter 'foo'
 
 class A<T>(val n: T, val foo: A<Int>)
 

idea/testData/quickfix/createFromUsage/createVariable/primaryParameter/varOnUserType.kt

@@ -1,5 +1,5 @@
 // "Create property 'foo' as constructor parameter" "true"
-// ERROR: No value passed for parameter foo
+// ERROR: No value passed for parameter 'foo'
 
 class A<T>(val n: T)
 

idea/testData/quickfix/createFromUsage/createVariable/primaryParameter/varOnUserType.kt.after

@@ -1,5 +1,5 @@
 // "Create property 'foo' as constructor parameter" "true"
-// ERROR: No value passed for parameter foo
+// ERROR: No value passed for parameter 'foo'
 
 class A<T>(val n: T, var foo: String)
 

idea/testData/quickfix/insertDelegationCall/nonApplicableInsertSuper.kt

@@ -1,5 +1,5 @@
 // "Insert 'super()' call" "true"
-// ERROR: No value passed for parameter x
+// ERROR: No value passed for parameter 'x'
 
 open class B(val x: Int)
 

idea/testData/quickfix/insertDelegationCall/nonApplicableInsertSuper.kt.after

@@ -1,5 +1,5 @@
 // "Insert 'super()' call" "true"
-// ERROR: No value passed for parameter x
+// ERROR: No value passed for parameter 'x'
 
 open class B(val x: Int)
 

idea/testData/quickfix/migration/javaAnnotationPositionedArguments/noValueForArgumentMultiple.after.kt

@@ -1,6 +1,6 @@
 // "Replace invalid positioned arguments for annotation" "true"
 // WITH_RUNTIME
 // ERROR: Only named arguments are available for Java annotations
-// ERROR: No value passed for parameter arg2
+// ERROR: No value passed for parameter 'arg2'
 
 @Ann(1, arg1 = "abc") class A

idea/testData/quickfix/migration/javaAnnotationPositionedArguments/noValueForArgumentMultiple.before.Main.kt

@@ -1,6 +1,6 @@
 // "Replace invalid positioned arguments for annotation" "true"
 // WITH_RUNTIME
 // ERROR: Only named arguments are available for Java annotations
-// ERROR: No value passed for parameter arg2
+// ERROR: No value passed for parameter 'arg2'
 
 @Ann(1, "abc"<caret>) class A

idea/testData/quickfix/supertypeInitialization/addParenthesis.kt

@@ -1,4 +1,4 @@
 // "Change to constructor invocation" "true"
-// ERROR: No value passed for parameter x
+// ERROR: No value passed for parameter 'x'
 open class A(x : Int) {}
 class B : A<caret> {}

idea/testData/quickfix/supertypeInitialization/addParenthesis.kt.after

@@ -1,4 +1,4 @@
 // "Change to constructor invocation" "true"
-// ERROR: No value passed for parameter x
+// ERROR: No value passed for parameter 'x'
 open class A(x : Int) {}
 class B : A(<caret>) {}

j2k/testData/fileOrElement/kotlinApiAccess/InheritedProperty.kt

@@ -1,4 +1,4 @@
-// ERROR: No value passed for parameter field
+// ERROR: No value passed for parameter 'field'
 import kotlinApi.*
 
 internal class C : KotlinClass() {

jps-plugin/testData/incremental/classHierarchyAffected/multiModuleDefaultValueInConstructorRemoved/build.log

@@ -27,7 +27,7 @@ End of files
 Exit code: ABORT
 ------------------------------------------
 COMPILATION FAILED
-No value passed for parameter x
+No value passed for parameter 'x'
 
 ================ Step #2 =================
 

jps-plugin/testData/incremental/pureKotlin/defaultValueInConstructorRemoved/build.log

@@ -30,8 +30,8 @@ End of files
 Exit code: ABORT
 ------------------------------------------
 COMPILATION FAILED
-No value passed for parameter x
-No value passed for parameter x
+No value passed for parameter 'x'
+No value passed for parameter 'x'
 
 ================ Step #2 =================
 

jps-plugin/testData/incremental/pureKotlin/defaultValueInConstructorRemoved/experimental-ic-build.log

@@ -29,8 +29,8 @@ End of files
 Exit code: ABORT
 ------------------------------------------
 COMPILATION FAILED
-No value passed for parameter x
-No value passed for parameter x
+No value passed for parameter 'x'
+No value passed for parameter 'x'
 
 ================ Step #2 =================
 

jps-plugin/testData/incremental/pureKotlin/defaultValueRemoved1/build.log

@@ -23,7 +23,7 @@ End of files
 Exit code: ABORT
 ------------------------------------------
 COMPILATION FAILED
-No value passed for parameter x
+No value passed for parameter 'x'
 
 ================ Step #2 =================
 

jps-plugin/testData/incremental/pureKotlin/defaultValueRemoved1/experimental-ic-build.log

@@ -24,7 +24,7 @@ End of files
 Exit code: ABORT
 ------------------------------------------
 COMPILATION FAILED
-No value passed for parameter x
+No value passed for parameter 'x'
 
 ================ Step #2 =================
 

jps-plugin/testData/incremental/pureKotlin/defaultValueRemoved2/build.log

@@ -25,8 +25,8 @@ End of files
 Exit code: ABORT
 ------------------------------------------
 COMPILATION FAILED
-No value passed for parameter y
-No value passed for parameter y
+No value passed for parameter 'y'
+No value passed for parameter 'y'
 
 ================ Step #2 =================
 

jps-plugin/testData/incremental/pureKotlin/defaultValueRemoved2/experimental-ic-build.log

@@ -27,8 +27,8 @@ End of files
 Exit code: ABORT
 ------------------------------------------
 COMPILATION FAILED
-No value passed for parameter y
-No value passed for parameter y
+No value passed for parameter 'y'
+No value passed for parameter 'y'
 
 ================ Step #2 =================