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:
[email protected]
