Skip to content

[4.0.x] Replace deprecated Terminal.getWidth/getHeight with getColumns/getRows - #12969

Merged
gnodet merged 1 commit into
maven-4.0.xfrom
backport/terminal-getcolumns-getrows-4.0.x
Aug 31, 2026
Merged

[4.0.x] Replace deprecated Terminal.getWidth/getHeight with getColumns/getRows#12969
gnodet merged 1 commit into
maven-4.0.xfrom
backport/terminal-getcolumns-getrows-4.0.x

Conversation

@gnodet

@gnodet gnodet commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Summary

Backport of #12968 to maven-4.0.x.

  • JLine 4.4.0 replaced Sized.getWidth()/getHeight() with getColumns()/getRows()
  • Updates FastTerminal overrides, MessageUtils.getTerminalWidth() delegate call, and a Javadoc reference in FastTerminalReentrancyTest

Test plan

  • Existing tests pass (FastTerminalReentrancyTest exercises the affected code paths)
  • CI green

🤖 Generated with Claude Code

JLine 4.4.0 replaced the Sized interface methods getWidth()/getHeight()
with getColumns()/getRows(). Update all call sites and overrides.

Co-Authored-By: Claude Opus 4.6 <[email protected]>

@gnodet gnodet left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Looks Good

Clean backport of #12968 to maven-4.0.x. The diff is identical to the original — all 4 references (3 code call sites + 1 Javadoc) across 3 files updated. The target branch already uses JLine 4.4.0, so the new methods are available.


🤖 This review was generated by ForgeBot.

@gnodet
gnodet merged commit 0ad2f71 into maven-4.0.x Aug 31, 2026
43 of 44 checks passed
@gnodet
gnodet deleted the backport/terminal-getcolumns-getrows-4.0.x branch August 31, 2026 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants