Liam Miller-Cushon
Liam Miller-Cushon
> My simplistic attempt at a reproducer did not prove anything, as I expected. Its at times like these when I'm really just giving you a hint that there's a...
Thanks for the repro. The check doesn't currently complain about methods that can be overridden, since overrides of those methods may throw checked exceptions and it isn't correct to remove...
It seems like the right thing to do eventually, I don't have specific plans on this front. A couple of notes: * I think there are some split package issues...
The relevant upstream bugs are: * https://bugs.openjdk.org/browse/JDK-8329951 (JDK 25), adds start position information to variable tree types for `var` * https://bugs.openjdk.org/browse/JDK-8358604 (pending), may add end positions
I think the status quo is that the CI for 17 is compiling on newer versions and then using 17 for maven-surefire-plugin. I haven't paged in all of the history...
Thanks for the investigation @commonquail Overall I think that the most transparent and least confusing way to enable checks is for projects to just list all of the checks that...
``` $ java -jar google-java-format-1.32.0-all-deps.jar TextBlocks.java public class TextBlocks { public static String textFun(String[] args) { return """ this is sample text """; } } ``` I agree that's a...
Thanks for the report. I think there are a few places where checks make assumptions that methods are idempotent in situations like this. Another example is [IdentityBinaryExpression](https://errorprone.info/bugpattern/IdentityBinaryExpression) where it discourages...
cc @cpovirk Thanks for the report. The use-case for MultipleNullnessAnnotations is primarily nullness annotations that have semantics similar to the https://github.com/jspecify/jspecify ones. We could look at having the check skip...
Thanks for the report, I had just noticed this, it is a regression after #27810. I have a fix in progress, and then I will make another turbine release. @hvadehra...