Update comments in NullabilityUtil#hasAnyAnnotationMatching

[msridhar]

In #1452 we added a comment implying that change was needed due to a javac bug but that was incorrect; it was our bug that was being masked before. This PR fixes the comments (no behavior change).

Summary by CodeRabbit

• Documentation
  • Improved internal code comments for clarity and maintainability.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

This pull request updates comments in the hasAnyAnnotationMatching method of the NullabilityUtil class. A note is added before a loop that checks declaration annotations, and an existing comment is revised to clarify that the subsequent loop examines type-use annotations. The change involves two lines added and two lines removed, with no modifications to method logic, control flow, or signatures.

Possibly related PRs

• PR #1362: Modifies the same hasAnyAnnotationMatching method with implementation changes, while this PR focuses on updating documentation comments

Suggested reviewers

• yuxincs

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately reflects the main change—updating comments in NullabilityUtil#hasAnyAnnotationMatching—which aligns with the PR description and the file-level summary.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.41%. Comparing base (f3b9181) to head (b48d5b8).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff            @@
##             master    #1457   +/-   ##
=========================================
  Coverage     88.41%   88.41%           
  Complexity     2715     2715           
=========================================
  Files            99       99           
  Lines          9019     9019           
  Branches       1803     1803           
=========================================
  Hits           7974     7974           
  Misses          517      517           
  Partials        528      528           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.