garydgregory commented on PR #1117: URL: https://github.com/apache/commons-lang/pull/1117#issuecomment-1745249840
-1 I agree with @aherbert Throwing an NPE for null input to a method makes perfect sense. In this case, this PR throws an NPE because internal processing happens to see a null value. The existing code correctly interprets that this null intermediary value is caused by bad input to the method and therefore properly throws `IllegalArgumentException`. Closing. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
