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 2014/02/02 10:07:04 UTC
[3/4] git commit: Use Args#notNull()
Use Args#notNull()
Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/25a34cc0
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/25a34cc0
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/25a34cc0
Branch: refs/heads/master
Commit: 25a34cc03c7b5e86f964a97c9d3b043b8a8ff71e
Parents: 288eae0
Author: Martin Tzvetanov Grigorov <mg...@apache.org>
Authored: Fri Jan 31 14:07:54 2014 +0100
Committer: Martin Tzvetanov Grigorov <mg...@apache.org>
Committed: Sun Feb 2 10:06:42 2014 +0100
----------------------------------------------------------------------
.../src/main/java/org/apache/wicket/MarkupContainer.java | 10 ++--------
.../main/java/org/apache/wicket/markup/ComponentTag.java | 6 ++----
2 files changed, 4 insertions(+), 12 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/wicket/blob/25a34cc0/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 6e0c3c2..1855262 100644
--- a/wicket-core/src/main/java/org/apache/wicket/MarkupContainer.java
+++ b/wicket-core/src/main/java/org/apache/wicket/MarkupContainer.java
@@ -193,10 +193,7 @@ public abstract class MarkupContainer extends Component implements Iterable<Comp
{
for (Component child : childs)
{
- if (child == null)
- {
- throw new IllegalArgumentException("argument child must be not null");
- }
+ Args.notNull(child, "child");
checkHierarchyChange(child);
@@ -269,10 +266,7 @@ public abstract class MarkupContainer extends Component implements Iterable<Comp
*/
public boolean contains(final Component component, final boolean recurse)
{
- if (component == null)
- {
- throw new IllegalArgumentException("argument component may not be null");
- }
+ Args.notNull(component, "component");
if (recurse)
{
http://git-wip-us.apache.org/repos/asf/wicket/blob/25a34cc0/wicket-core/src/main/java/org/apache/wicket/markup/ComponentTag.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/markup/ComponentTag.java b/wicket-core/src/main/java/org/apache/wicket/markup/ComponentTag.java
index d387368..d9158f0 100644
--- a/wicket-core/src/main/java/org/apache/wicket/markup/ComponentTag.java
+++ b/wicket-core/src/main/java/org/apache/wicket/markup/ComponentTag.java
@@ -30,6 +30,7 @@ import org.apache.wicket.markup.parser.XmlTag;
import org.apache.wicket.markup.parser.XmlTag.TagType;
import org.apache.wicket.markup.parser.filter.HtmlHandler;
import org.apache.wicket.request.Response;
+import org.apache.wicket.util.lang.Args;
import org.apache.wicket.util.lang.Generics;
import org.apache.wicket.util.string.AppendingStringBuffer;
import org.apache.wicket.util.string.StringValue;
@@ -178,10 +179,7 @@ public class ComponentTag extends MarkupElement
*/
public final void addBehavior(final Behavior behavior)
{
- if (behavior == null)
- {
- throw new IllegalArgumentException("Argument [behavior] cannot be null");
- }
+ Args.notNull(behavior, "behavior");
if (behaviors == null)
{