Documentation Preview

Compared to main:
This changeset changes 0 violations,
introduces 0 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-06-04 15:45:58+00:00 for 605a7a2)

What do you think about adding other examples of shadowing to this rule? e.g. a block-scoped variable shadowing one defined at method scope?

ok, an example would be:

fun process(items: List<String>) {
    val result = "default"   // method-scope variable

    for (item in items) {
        val result = item.trim()  // shadows method-scope 'result' -- block-scope shadowing
        println(result)
    }
}

Yes, we can add this to this rule.

• Or make it a separate rule for better (prio/active) control and reporting lines, or too fine grained?
• Or optionally make a property to enable/disable this type (e.g. "checkLocalShadowing").

Shall I add it to this rule?
If so, maybe name should then also change to "LocalVariableShadowing".

I'll leave that up to you. I just wanted to mention the possibility. Both versions (one rule for all shadowing cases or separate rules) are fine.