stevenschenk edited a comment on pull request #47: URL: https://github.com/apache/maven-checkstyle-plugin/pull/47#issuecomment-854548556
> @stevenschenk the checkstyle API states that file and line can never be null. So I expect you are fixing a problem which can never occur, only in tests. > I mean, how can a violation not be on a line in a file? > > That said, I have no objections merging it anyway. My bad. Was not listed in the ticket, but I did not verify it either. I checked the constructor and there was no null check on these fields, so I assumed they were nullable. If the API states they cannot be null, should we do a `requiresNonNull()` check in the constructor? -- 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]
