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 <[email protected]> Authored: Fri Jan 31 14:07:54 2014 +0100 Committer: Martin Tzvetanov Grigorov <[email protected]> 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) {
