Attachment 203074 Details for Bug 357259 – Fix

[patch] Fix

patch.txt (text/plain), 3.43 KB, created by Markus Keller

on 2011-09-09 12:19:23 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Markus Keller

Created: 2011-09-09 12:19:23 EDT

Size: 3.43 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.jdt.core
>Index: dom/org/eclipse/jdt/core/dom/AST.java
>===================================================================
>RCS file: /cvsroot/eclipse/org.eclipse.jdt.core/dom/org/eclipse/jdt/core/dom/AST.java,v
>retrieving revision 1.168
>diff -u -r1.168 AST.java
>--- dom/org/eclipse/jdt/core/dom/AST.java	28 Jul 2011 17:07:33 -0000	1.168
>+++ dom/org/eclipse/jdt/core/dom/AST.java	9 Sep 2011 16:16:57 -0000
>@@ -1060,7 +1060,6 @@
> 	 * <ul>
> 	 * <li>the node belongs to a different AST</li>
> 	 * <li>the node already has a parent</li>
>-	 * <li>a cycle in would be created</li>
> 	 * </ul>
> 	 */
> 	public ArrayType newArrayType(Type componentType) {
>@@ -1071,28 +1070,26 @@
> 
> 	/**
> 	 * Creates and returns a new unparented array type node with the given
>-	 * element type and number of dimensions.
>+	 * element type and number of (additional) dimensions.
> 	 * <p>
> 	 * Note that if the element type passed in is an array type, the
> 	 * element type of the result will not be the same as what was passed in.
> 	 * </p>
> 	 *
>-	 * @param elementType the element type (never an array type)
>+	 * @param elementType the element type (can an array type)
> 	 * @param dimensions the number of dimensions, a positive number
> 	 * @return a new unparented array type node
> 	 * @exception IllegalArgumentException if:
> 	 * <ul>
> 	 * <li>the node belongs to a different AST</li>
> 	 * <li>the node already has a parent</li>
>-	 * <li>a cycle in would be created</li>
> 	 * <li>the element type is null</li>
>-	 * <li>the element type is an array type</li>
> 	 * <li>the number of dimensions is lower than 1</li>
> 	 * <li>the number of dimensions is greater than 1000</li>
> 	 * </ul>
> 	 */
> 	public ArrayType newArrayType(Type elementType, int dimensions) {
>-		if (elementType == null || elementType.isArrayType()) {
>+		if (elementType == null) {
> 			throw new IllegalArgumentException();
> 		}
> 		if (dimensions < 1 || dimensions > 1000) {
>@@ -1386,7 +1383,6 @@
> 	 * <ul>
> 	 * <li>the node belongs to a different AST</li>
> 	 * <li>the node already has a parent</li>
>-	 * <li>a cycle in would be created</li>
> 	 * </ul>
> 	 */
> 	public ExpressionStatement newExpressionStatement(Expression expression) {
>@@ -1426,7 +1422,6 @@
> 	 * <ul>
> 	 * <li>the node belongs to a different AST</li>
> 	 * <li>the node already has a parent</li>
>-	 * <li>a cycle in would be created</li>
> 	 * <li>the given fragment is null</li>
> 	 * </ul>
> 	 */
>@@ -2271,7 +2266,6 @@
> 	 * <ul>
> 	 * <li>the node belongs to a different AST</li>
> 	 * <li>the node already has a parent</li>
>-	 * <li>a cycle in would be created</li>
> 	 * </ul>
> 	 * @since 3.0
> 	 */
>@@ -2302,7 +2296,6 @@
> 	 * <ul>
> 	 * <li>the node belongs to a different AST</li>
> 	 * <li>the node already has a parent</li>
>-	 * <li>a cycle in would be created</li>
> 	 * </ul>
> 	 */
> 	public TypeDeclarationStatement
>@@ -2355,9 +2348,7 @@
> 	 * <ul>
> 	 * <li>the node belongs to a different AST</li>
> 	 * <li>the node already has a parent</li>
>-	 * <li>a cycle in would be created</li>
> 	 * <li>the given fragment is null</li>
>-	 * <li>a cycle in would be created</li>
> 	 * </ul>
> 	 */
> 	public VariableDeclarationExpression
>@@ -2402,7 +2393,6 @@
> 	 * <ul>
> 	 * <li>the node belongs to a different AST</li>
> 	 * <li>the node already has a parent</li>
>-	 * <li>a cycle in would be created</li>
> 	 * <li>the variable declaration fragment is null</li>
> 	 * </ul>
> 	 */

Actions: View | Diff

Attachments on bug 357259: 203074 | 203079