mernst opened a new pull request #154:
URL: https://github.com/apache/commons-io/pull/154
It is surprising that `FilenameUtils.equals` can throw
`NullPointerException`, because:
* such behavior is outside the ordinary contract of `equals()`, and
* `FilenameUtils.equals` throws NullPointerExceptioneven if the client did
not pass `null` as an argument. It throws NullPointerExceptioneven if one of
the arguments is not a valid, normalizable filename. However, this behavior is
not documented.
This pull request:
* documents the requirement that the arguments must be valid,
* makes the exception message more specific, indicating the filename that
was invalid, and
* throws `IllegalArgumentException` instead of `NullPointerException`.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
