You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@maven.apache.org by GitBox <gi...@apache.org> on 2021/06/04 10:02:11 UTC

[GitHub] [maven-checkstyle-plugin] stevenschenk commented on pull request #47: [MCHECKSTYLE-401] - NullPointerException in Violation class

stevenschenk commented 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. If it 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:
users@infra.apache.org