Roger Whitcomb created PIVOT-1012:
-------------------------------------

             Summary: 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)

Reply via email to