You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by mg...@apache.org on 2012/01/27 16:03:25 UTC

[2/2] git commit: Use Args to check the parameters

Use Args to check the parameters


Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/e0fb99d4
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/e0fb99d4
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/e0fb99d4

Branch: refs/heads/master
Commit: e0fb99d49a665b1c0c4863d5543ef2f59d8ce0a7
Parents: 715f8e2
Author: Martin Tzvetanov Grigorov <mg...@apache.org>
Authored: Fri Jan 27 16:31:13 2012 +0200
Committer: Martin Tzvetanov Grigorov <mg...@apache.org>
Committed: Fri Jan 27 16:31:13 2012 +0200

----------------------------------------------------------------------
 .../java/org/apache/wicket/MarkupContainer.java    |   10 ++--------
 1 files changed, 2 insertions(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/wicket/blob/e0fb99d4/wicket-core/src/main/java/org/apache/wicket/MarkupContainer.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/MarkupContainer.java b/wicket-core/src/main/java/org/apache/wicket/MarkupContainer.java
index 8b79831..ed80d33 100644
--- a/wicket-core/src/main/java/org/apache/wicket/MarkupContainer.java
+++ b/wicket-core/src/main/java/org/apache/wicket/MarkupContainer.java
@@ -608,10 +608,7 @@ public abstract class MarkupContainer extends Component implements Iterable<Comp
 	{
 		checkHierarchyChange(component);
 
-		if (component == null)
-		{
-			throw new IllegalArgumentException("argument component may not be null");
-		}
+		Args.notNull(component, "component");
 
 		children_remove(component);
 		removedComponent(component);
@@ -628,10 +625,7 @@ public abstract class MarkupContainer extends Component implements Iterable<Comp
 	 */
 	public MarkupContainer remove(final String id)
 	{
-		if (id == null)
-		{
-			throw new IllegalArgumentException("argument id may not be null");
-		}
+		Args.notNull(id, "id");
 
 		final Component component = get(id);
 		if (component != null)