Bump com.google.errorprone:error_prone_core from 2.48.0 to 2.49.0

[dependabot]

⚠️ Dependabot is rebasing this PR ⚠️

Rebasing might not happen immediately, so don't worry if this takes some time.

Note: if you make any changes to this PR yourself, they will take precedence over the rebase.

---

Bumps com.google.errorprone:error_prone_core from 2.48.0 to 2.49.0.

Release notes

Sourced from com.google.errorprone:error_prone_core's releases.

Error Prone 2.49.0

This release includes several changes to Matcher APIs, and removed some deprecated or problematic APIs:

• Remove deprecated MethodMatchers.withSignature API, which relies on fragile toString behaviour. Alternatives for matching on method signatures with varargs and type parameters were added in google/error-prone@a98a1c5.
• Removed variableType(Matcher) API. Matchers.variableType(Matcher) uses VariableTree#getType to match variable types, which own't work for lambda parameters with inferred types after JDK-8268850. The recommended replacement is variableType(TypePredicate).
• Make enclosingPackage return an optional. Module elements are not enclosed by a package, checks using enclosingPackage shouldn't assume an enclosing package exists when processing arbitrary elements.
• New FieldMatchers API, similar to MethodMatchers (google/error-prone@1dd9c3a).

New checks:

• AssertThrowsBlockToExpression: Discourage unnecessary block lambdas in assertThrows.
• AssertThrowsMinimizer: Suggest minimizing the amount of logic in assertThrows.
• MemorySegmentReferenceEquality: Discourage using reference equality for MemorySegments.
• PreferThrowsTag: Recommends using @throws instead of the legacy @exception javadoc tag.
• RecordAccessorInCompactConstructor: detect record accessors inside the compact canonical ctors, which read uninitialized fields.

Closed issues: #2283, #3503, #5210, #5289, #5548, #5548, #5554, #5609, #5614, #5656

Full changelog: google/error-prone@v2.48.0...v2.49.0

Commits

• 89d75c1 Release Error Prone 2.49.0
• 0b7b03b Fix up some javadoc on `ModifySourceCollectionInStream.isStreamApiInvocationO...
• fe5a7b1 Remove old FieldMatchers API
• d54a1d1 Fix up some Finally javadocs.
• d93b319 [RefactorSwitch] bugfix comment handling
• ff59782 [IfChainToSwitch] cleanup redundant conditions in ternary. No functional cha...
• 43b6df6 Generalise DuplicateAssertion to handle check* methods.
• 2c4346f Fix a bug in BooleanLiteral: it currently suggests replacing `Boolean.FALSE...
• 559039b [IfChainToSwitch] doc-only change. fix typo in code comments.
• 393c61c [IfChainToSwitch] enhance code generation to emit unnamed variables, when sup...
• Additional commits viewable in compare view

[fabian-barney]

Skipping auto-merge: the PR tests are failing and the Error Prone 2.49.0 notes explicitly remove/replace APIs, so this bump is not low-risk enough for automatic merge.

[fabian-barney]

Refreshed this dependency bump onto current main in #49 because the original Dependabot branch was stale relative to the current workflow state.

Validation on #49:

• ./mvnw -B -ntp -Pnullaway test
• ./mvnw -B -ntp -P!quality-gates-all,quality-gate-crap verify
• ./mvnw -B -ntp -P!quality-gates-all,quality-gate-cognitive verify

I also checked the repository for use of the Error Prone APIs removed in 2.49.0 and did not find any usage.

[fabian-barney]

Superseded by #49, which has been merged into main with the required NullAway compatibility fix.

[dependabot]

OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting @dependabot ignore this major version or @dependabot ignore this minor version. You can also ignore all major, minor, or patch releases for a dependency by adding an ignore condition with the desired update_types to your config file.

If you change your mind, just re-open this PR and I'll resolve any conflicts on it.