fix: correct Linux PENDIN termios constant

[gnodet]

Summary

• Fix PENDIN local flag constant: 0x0002000 (8192) → 0x0004000 (16384) per Linux asm-generic/termbits.h (octal 0040000)
• Applied to both JNI (LinuxNativePty) and FFM (CLibrary) terminal providers

Impact

Currently low — JLine uses this constant symmetrically in toTermios() and toAttributes(), so the wrong value roundtrips correctly. However, interoperability with termios structs set by other programs (e.g., reading terminal state set by the shell) would silently lose this flag.

Closes #1833

Summary by CodeRabbit

• Bug Fixes
  • Fixed incorrect terminal attribute flag configuration on Linux platforms, improving terminal behavior and compatibility.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 07bd8131-9275-4fb7-ad3b-189496779827

📥 Commits

Reviewing files that changed from the base of the PR and between da637d3 and 6178700.

📒 Files selected for processing (2)

• terminal-ffm/src/main/java/org/jline/terminal/impl/ffm/CLibrary.java
• terminal-jni/src/main/java/org/jline/terminal/impl/jni/linux/LinuxNativePty.java

---

📝 Walkthrough

Walkthrough

Two files are updated to correct the Linux PENDIN termios constant from an incorrect value (0x0002000) to the correct Linux kernel definition (0x0004000), fixing interoperability with externally-set terminal state.

Changes

Cohort / File(s)	Summary
FFM Implementation terminal-ffm/src/main/java/org/jline/terminal/impl/ffm/CLibrary.java	Updated Linux PENDIN local flag constant from 0x0002000 to 0x0004000 to align with actual kernel definition.
JNI Implementation terminal-jni/src/main/java/org/jline/terminal/impl/jni/linux/LinuxNativePty.java	Updated Linux PENDIN local flag constant from 0x0002000 to 0x0004000 to align with actual kernel definition.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A bit was amiss, now it's right and precise,
The PENDIN flag hops to the kernel's device,
From 0x2000 to 0x4000 we leap,
Terminal states now their secrets can keep! ✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'fix: correct Linux PENDIN termios constant' clearly and concisely describes the main change: correcting an incorrect Linux constant value from 0x0002000 to 0x0004000.
Linked Issues check	✅ Passed	The PR successfully corrects the PENDIN constant from 0x0002000 to 0x0004000 in both JNI and FFM implementations, directly addressing all requirements in issue #1833.
Out of Scope Changes check	✅ Passed	All changes are directly scoped to correcting the PENDIN constant across the two terminal provider implementations, with no unrelated modifications present.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch ci-issue-1833

---

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.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
100.0% Duplication on New Code

See analysis details on SonarQube Cloud