From 3e54b8a5ba3eca2b9c5315323cca25db6501d9ac Mon Sep 17 00:00:00 2001 From: Stephan Schroevers Date: Sun, 11 Feb 2024 12:28:13 +0100 Subject: [PATCH] Tweak --- .../bugpatterns/LexicographicalAnnotationListing.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/error-prone-contrib/src/main/java/tech/picnic/errorprone/bugpatterns/LexicographicalAnnotationListing.java b/error-prone-contrib/src/main/java/tech/picnic/errorprone/bugpatterns/LexicographicalAnnotationListing.java index 9ef9e8e34e..d608cd79f0 100644 --- a/error-prone-contrib/src/main/java/tech/picnic/errorprone/bugpatterns/LexicographicalAnnotationListing.java +++ b/error-prone-contrib/src/main/java/tech/picnic/errorprone/bugpatterns/LexicographicalAnnotationListing.java @@ -36,6 +36,8 @@ *

The idea behind this checker is that maintaining a sorted sequence simplifies conflict * resolution, and can even avoid it if two branches add the same annotation. */ +// XXX: Currently this checker only flags method-level annotations. It should likely also flag +// type-, field- and parameter-level annotations. // XXX: Duplicate entries are often a mistake. Consider introducing a similar `BugChecker` that // flags duplicates. @AutoService(BugChecker.class)