From 3a2438e74ffd399989caf16bb7d4a72bbd144fbd Mon Sep 17 00:00:00 2001 From: peter Date: Mon, 11 Apr 2011 14:18:44 +0200 Subject: [PATCH] a parameter should always have a name --- .../plugins/groovy/lang/psi/util/GrClassImplUtil.java | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/plugins/groovy/src/org/jetbrains/plugins/groovy/lang/psi/util/GrClassImplUtil.java b/plugins/groovy/src/org/jetbrains/plugins/groovy/lang/psi/util/GrClassImplUtil.java index c3a7f45d28f1..dd759f17358f 100644 --- a/plugins/groovy/src/org/jetbrains/plugins/groovy/lang/psi/util/GrClassImplUtil.java +++ b/plugins/groovy/src/org/jetbrains/plugins/groovy/lang/psi/util/GrClassImplUtil.java @@ -17,6 +17,7 @@ package org.jetbrains.plugins.groovy.lang.psi.util; import com.intellij.openapi.util.*; +import com.intellij.openapi.util.text.StringUtil; import com.intellij.psi.*; import com.intellij.psi.impl.PsiClassImplUtil; import com.intellij.psi.impl.light.LightMethodBuilder; @@ -555,9 +556,13 @@ public class GrClassImplUtil { builder.setNavigationElement(method); builder.addModifier(PsiModifier.PUBLIC); final PsiParameter[] originalParameters = method.getParameterList().getParameters(); - for (PsiParameter originalParameter : originalParameters) { + for (int i = 0, originalParametersLength = originalParameters.length; i < originalParametersLength; i++) { + PsiParameter originalParameter = originalParameters[i]; PsiType type = substitutor.substitute(originalParameter.getType()); - builder.addParameter(originalParameter.getName(), type == null ? PsiType.getJavaLangObject(clazz.getManager(), clazz.getResolveScope()) : type); + if (type == null) { + type = PsiType.getJavaLangObject(clazz.getManager(), clazz.getResolveScope()); + } + builder.addParameter(StringUtil.notNullize(originalParameter.getName(), "p" + i), type); } builder.setBaseIcon(GroovyIcons.METHOD); return builder;