Documentation Preview

Compared to main:
This changeset changes 0 violations,
introduces 120 new violations, 0 new errors and 0 new configuration errors,
removes 0 violations, 0 errors and 0 configuration errors.
There are 0 changed duplications, 0 new duplications and 0 removed duplications.
There are 0 changed CPD errors, 0 new CPD errors and 0 removed CPD errors.

Regression Tester Report

(comment created at 2026-05-08 14:03:19+00:00 for 8bdd308)

Sometimes copy-paste errors can create

if (foo) {{
   int i =1;
   System.exit(i);
}
}

which might make sense to also catch (i.e. if parent size is 1, report it without checking for variable definitions).

The Spring regressions show another usecase where blocks are used to limit scope of anonymous classes -- should that be acceptable?

Thanks. Adding checks for the {{ ... }} case is definitely a good idea. Although there might be overlap with DoubleBraceInitialization.

I did look at the spring framework regression reports... and completely missed the anonymous inner local classes. You're completely right that that shouldn't be a violation.