Hi Roger,
> I guess my main motivation for doing this was seeing a lot of common
> code, that is, this pattern:
> if (blah == null) {
> throw new IllegalArgumentException("blah is null.")
> }
> and wanted to make some common code for it. Because lots of times the
> exception had no messag
Hi all,
I guess my main motivation for doing this was seeing a lot of
common code, that is, this pattern:
if (blah == null) {
throw new IllegalArgumentException("blah is null.")
}
and wanted to make some common code for it. Because lots of times the
exception had no message, or it was
[
https://issues.apache.org/jira/browse/PIVOT-965?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14993469#comment-14993469
]
Karel Hübl commented on PIVOT-965:
--
Hi,
I posted related question to JAVA Webstart & JNLP
Hi all,
just look at the commit (from Roger) for doing some cleanup, so not
null checks for arguments are refactored in a utility method like
this:
Utils.checkNull(size, "size");
I'm not against this (refactor common code is always good :-) ), but
just I wonder if in trunk we could use a way for
