View | Details | Raw Unified | Return to bug 331120
Collapse All | Expand All

(-)model/org/eclipse/jdt/core/Signature.java (-2 / +6 lines)
Lines 974-980 Link Here
974
 *   correct
974
 *   correct
975
 */
975
 */
976
public static String getElementType(String typeSignature) throws IllegalArgumentException {
976
public static String getElementType(String typeSignature) throws IllegalArgumentException {
977
	return new String(getElementType(typeSignature.toCharArray()));
977
	char[] signature = typeSignature.toCharArray();
978
	char[] elementType = getElementType(signature);
979
	return signature == elementType ? typeSignature : new String(elementType);
978
}
980
}
979
/**
981
/**
980
 * Returns the number of parameter types in the given method signature.
982
 * Returns the number of parameter types in the given method signature.
Lines 1387-1393 Link Here
1387
 * @since 3.1
1389
 * @since 3.1
1388
 */
1390
 */
1389
public static String getTypeErasure(String parameterizedTypeSignature) throws IllegalArgumentException {
1391
public static String getTypeErasure(String parameterizedTypeSignature) throws IllegalArgumentException {
1390
	return new String(getTypeErasure(parameterizedTypeSignature.toCharArray()));
1392
	char[] signature = parameterizedTypeSignature.toCharArray();
1393
	char[] erasure = getTypeErasure(signature);
1394
	return signature == erasure ? parameterizedTypeSignature : new String(erasure);
1391
}
1395
}
1392
1396
1393
/**
1397
/**
(-)src/org/eclipse/jdt/core/tests/model/SignatureTests.java (+16 lines)
Lines 173-178 Link Here
173
		"Signature#getElementType not correct", "Qlist<Qstring;>;",
173
		"Signature#getElementType not correct", "Qlist<Qstring;>;",
174
		Signature.getElementType("[[[[Qlist<Qstring;>;"));
174
		Signature.getElementType("[[[[Qlist<Qstring;>;"));
175
}
175
}
176
public void testGetElementType2() {
177
	String typeSign = "Ljava.util.List;";
178
	assertTrue(Signature.getElementType(typeSign) == typeSign);
179
}
176
/**
180
/**
177
 * @see Signature
181
 * @see Signature
178
 */
182
 */
Lines 771-776 Link Here
771
	);
775
	);
772
}
776
}
773
777
778
public void testGetTypeErasure9() {
779
	String sign = "Ljava.util.List;";
780
	assertTrue(Signature.getTypeErasure(sign) == sign);
781
}
782
783
public void testGetTypeErasure10() {
784
	assertEquals(
785
			"Ljava.util.List;",
786
			Signature.getTypeErasure("Ljava.util.List<-[Ljava.lang.Number;>;")
787
	);
788
}
789
774
/**
790
/**
775
 * @see Signature
791
 * @see Signature
776
 */
792
 */

Return to bug 331120