Let StrictUnusedVariable ignore record fields

[fawind]

Before this PR

Record fields were flagged as unused and had to be manually suppressed.

class Test {
  // without @SuppressWarnings("StrictUnusedVariable")	
  record Foo(int bar) {}

  static void method() {
    new Foo(1).bar()
  }
}

throws: error: [StrictUnusedVariable] The field 'bar' is never read.

After this PR

Flup on #1412 and #1414 now that error prone has better support for records.

Ignore record fields when checking for unused variables as only the record's getter-methods might be used.

Note that checking for the actual unused record class is done with the UnusedNestedClass check.

==COMMIT_MSG==
Let StrictUnusedVariable ignore record fields
==COMMIT_MSG==

[changelog-app]

Generate changelog in changelog/@unreleased

Type

• Feature
• Improvement
• Fix
• Break
• Deprecation
• Manual task
• Migration

Description

Let StrictUnusedVariable ignore record fields

Check the box to generate changelog(s)

• Generate changelog entry