While writing integration tests for my software, I noticed that many calls to commons logging were terminating with NullPointerException. I checked out the code and added input validation to CSVFormat.printRecord. While adding my unit test, I found that I broke another unit test. It appears that someone had enshrined the NPE in a unit test as expected behavior. So, part of my patch is the removal of that unit test. I tried creating a pull request, but was unable to do so. So, I have attached the patch file here. Please review it. I have never created a pull request on GitHub before. So, if this is a case where I should, please instruct me and I will be glad to.
Thanks, Derek Bennett
--------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
