ASTNode: improve IllegalArgumentException for more information

jukzi · jukzi · commit 1d61578b4701 · 2024-11-14T09:48:38.000+01:00
#3309
diff --git a/org.eclipse.jdt.core/dom/org/eclipse/jdt/core/dom/ASTNode.java b/org.eclipse.jdt.core/dom/org/eclipse/jdt/core/dom/ASTNode.java
@@ -3443,10 +3443,10 @@ public final int getLength() {
 	 */
 	public final void setSourceRange(int startPosition, int length) {
 		if (startPosition >= 0 && length < 0) {
-			throw new IllegalArgumentException();
+			throw new IllegalArgumentException("negative length=" + length + " startPosition= " + startPosition); //$NON-NLS-1$ //$NON-NLS-2$
 		}
 		if (startPosition < 0 && length != 0) {
-			throw new IllegalArgumentException();
+			throw new IllegalArgumentException("negative startPosition=" + startPosition + " length=" + length); //$NON-NLS-1$ //$NON-NLS-2$
 		}
 		// source positions are not considered a structural property
 		// but we protect them nevertheless

Original file line number	Diff line number	Diff line change
`@@ -3443,10 +3443,10 @@ public final int getLength() {`
`3443`	`3443`	`*/`
`3444`	`3444`	`public final void setSourceRange(int startPosition, int length) {`
`3445`	`3445`	`if (startPosition >= 0 && length < 0) {`
`3446`		`- throw new IllegalArgumentException();`
	`3446`	`+ throw new IllegalArgumentException("negative length=" + length + " startPosition= " + startPosition); //$NON-NLS-1$ //$NON-NLS-2$`
`3447`	`3447`	`}`
`3448`	`3448`	`if (startPosition < 0 && length != 0) {`
`3449`		`- throw new IllegalArgumentException();`
	`3449`	`+ throw new IllegalArgumentException("negative startPosition=" + startPosition + " length=" + length); //$NON-NLS-1$ //$NON-NLS-2$`
`3450`	`3450`	`}`
`3451`	`3451`	`// source positions are not considered a structural property`
`3452`	`3452`	`// but we protect them nevertheless`