[ https://issues.apache.org/jira/browse/PIVOT-1012?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16248224#comment-16248224 ]
Roger Whitcomb commented on PIVOT-1012: --------------------------------------- Note: many places have already been changed, but not marked with this issue. New changes will be identified as belonging to this issue, however. > Many places throw IllegalArgumentException during parameter validation, but > some are inconsistent > ------------------------------------------------------------------------------------------------- > > Key: PIVOT-1012 > URL: https://issues.apache.org/jira/browse/PIVOT-1012 > Project: Pivot > Issue Type: Improvement > Components: core, wtk, wtk-terra > Environment: All > Reporter: Roger Whitcomb > Assignee: Roger Whitcomb > Priority: Minor > Fix For: 2.1 > > > Primarily the code looks like this currently: > {code:java} > if (param == null) > throw new IllegalArgumentException(param + " is null"); > {code} > But not all places have the message in the exception, and not all places > check the parameters as they should, and not all places have the same message. > So, regularize this checking everywhere by making common "core" methods to do > this null check (or other checks, such as <= 0, etc.) so that the checking > and messaging are common. This also simplifies the code, and with JIT > compiling shouldn't affect runtime speed either, as this common method should > get compiled and/or inlined as appropriate. -- This message was sent by Atlassian JIRA (v6.4.14#64029)