DonalEvans commented on a change in pull request #5473: URL: https://github.com/apache/geode/pull/5473#discussion_r476998382
########## File path: geode-core/src/main/java/org/apache/geode/cache/query/internal/NullToken.java ########## @@ -33,6 +33,7 @@ */ public class NullToken implements DataSerializableFixedID, Comparable { + @SuppressWarnings("lgtm[java/useless-null-check]") Review comment: Good call on explaining the suppression here and other places the annotation is used. As for the different approaches to suppressing, [LGTM requires that inline suppressions contain no line breaks](https://lgtm.com/help/lgtm/alert-suppression#where-in-code-do-i-add-suppression-comments), which would not be possible for some of the lines due to the spotless plugin automatically wrapping comments that extend too far to the right. In those cases, the @SuppressWarnings annotation is used, but is not used throughout because it would potentially mask other, legitimate alerts in methods that are currently present or that get introduced at a later date. ---------------------------------------------------------------- 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: us...@infra.apache.org