Good question, but a lot of other Java code takes the style that a check for a null throws NPE rather than IAE.

I could just omit the null check altogether, in which case we'd probably crash during equalsIgnoreCase due to one of the arguments being null, and result in the same NPE message.

My preference is for what I wrote.  :-)