Re: NPE in CSVFormat

2021-01-27 Thread Gary Gregory
CSV does not use LOGGING... Gary On Wed, Jan 27, 2021, 21:01 Gilles Sadowski wrote: > Le jeu. 28 janv. 2021 à 02:30, Gary Gregory a > écrit : > > > > You can't attach files to this mailing list. > > > > You'll have to figure out GitHub PRs so we can see what you're proposing > to > > do ;-) >

Re: NPE in CSVFormat

2021-01-27 Thread Gilles Sadowski
Le jeu. 28 janv. 2021 à 02:30, Gary Gregory a écrit : > > You can't attach files to this mailing list. > > You'll have to figure out GitHub PRs so we can see what you're proposing to > do ;-) Or use JIRA... But about the issue itself: Hasn't "Commons Logging" been superseded by the "Log4j" proje

Re: NPE in CSVFormat

2021-01-27 Thread Gary Gregory
You can't attach files to this mailing list. You'll have to figure out GitHub PRs so we can see what you're proposing to do ;-) Gary On Wed, Jan 27, 2021, 15:40 Derek Bennett wrote: > While writing integration tests for my software, I noticed that many calls > to commons logging were terminati

NPE in CSVFormat

2021-01-27 Thread Derek Bennett
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 so