Skip to content

refactor: Clean up compiler warnings, unused code and test idioms - #43

Merged
bobbylight merged 1 commit into
masterfrom
clean-up-warnings-and-title-bug
Oct 5, 2026
Merged

bobbylight merged 1 commit into
masterfrom
clean-up-warnings-and-title-bug

Conversation

@bobbylight

Copy link
Copy Markdown
Owner

Summary

Cleans up minor code-quality issues: an unused field, a method ignoring its parameter, and assorted test-code nits.

Details

  • TextFilePropertiesDialog.createTitle(String) now uses its fileName parameter instead of textArea.getFileName()
  • Removed unused infoLabel field from FindToolBar
  • Tests: use List.getFirst(), remove stray ;;, unused import, a 00 literal, and a redundant StringBuilder/block lambda

Test plan

  • ./gradlew clean build passes (checkstyle, spotbugs, tests)

🤖 Generated with Claude Code

Fix TextFilePropertiesDialog.createTitle() to use its fileName parameter,
remove an unused field in FindToolBar, and tidy tests (getFirst(), stray
semicolons, unused imports, simplified lambdas).

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@bobbylight
bobbylight marked this pull request as ready for review October 5, 2026 01:01
@bobbylight bobbylight self-assigned this Oct 5, 2026
@bobbylight bobbylight added this to the 4.0.0 milestone Oct 5, 2026
@bobbylight
bobbylight merged commit ddf3571 into master Oct 5, 2026
3 checks passed
@bobbylight
bobbylight deleted the clean-up-warnings-and-title-bug branch October 5, 2026 01:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant