stty: Improve option handling and align behavior with GNU coreutils #9255

mattsu2020 · 2025-11-13T01:33:32Z

Summary:

Improve option parsing and error handling in the stty subcommand.
Align behavior and messages more closely with GNU coreutils stty.
Refactor duplicated logic and add tests to improve readability and maintainability.

Details:

Adjusted error handling for invalid/unsupported options to better match GNU’s exit codes and error messages.
Refactored branching logic for combined options to prevent unintended behavior in edge cases.
Clarified responsibilities of internal helper functions and added comments to better document their behavior.
Added regression tests covering representative TTY configuration patterns to prevent future behavioral drift.

Rationale:

There were behavioral differences between this implementation and GNU coreutils that could cause unexpected results in existing scripts or tooling relying on GNU-compatible stty behavior.
This change reduces migration friction and makes stty usage more predictable across environments.

Scope / Impact:

The change is scoped to the stty utility only; no other utilities are affected.
The primary goal is compatibility and robustness; no breaking changes are intended for valid existing usages.

Testing:

cargo test --package uutils-stty --all-features
Added and executed new tests for relevant option combinations and error cases.
Performed manual verification for common TTY settings, display, and error scenarios.

#9056

- Add GNU-compatible parsing of colon-separated hex -g output - Restore termios from saved state before applying new settings - Preserve platform-dependent fields while validating input robustly

- Replace Box<dyn UError> with USimpleError for stty helpers and save-format parsing to reduce indirection and simplify error handling - Use as_str/copied for argument iteration and convert flag bits via into() for clearer, more idiomatic Rust and improved type safety

- Update missing/invalid argument helpers to return Box<dyn UError> for consistent error typing - Propagate errors with map_err(Into::into) at call sites to match new helper signatures - Simplify string handling (String::as_str, &**s) to reduce verbosity and clarify intent

- Return USimpleError directly from missing_arg/invalid_* helpers to avoid Box indirection - Replace redundant map_err(Into::into) uses with direct missing_arg calls for clarity - Aligns error handling with UResult expectations and improves readability

- Fix mis-indented else branch to ensure missing_arg is called correctly - Improve readability and maintain consistent control flow structure

- Cast parsed hex flag values to u64 to match bitflag expectations - Remove redundant conversions when constructing termios flag sets - Ensures proper handling of saved formats on platforms with 64-bit flags

- Use parse_hex_u32 return type directly for flag bitfields - Simplifies code by eliminating unnecessary u32-to-u64 conversions

- Cast parsed flag bit values into appropriate integer types - Prevent potential type mismatch or incorrect flag interpretation during restore

- Use tcflag_t-based parser for hex flag values to match termios types - Avoid extra casts when reconstructing Input/Output/Control/Local flags for correctness and clarity

- Add tcflag, bitflag, and lflags to spell-checker ignore list - Prevent false positives when working with recently added flag names

- Add "cflags" to spell-checker ignore hints in stty.rs - Prevent false positives during spell-checking for terminal flag names

- Treat unknown dash-prefixed arguments as invalid options to match GNU stty behavior - Treat unrecognized bare words as missing arguments for clearer error reporting

- Treat unrecognized options starting with '-' as invalid arguments - Treat unrecognized bare words as invalid arguments instead of missing - Match project test expectations and clarify the inline documentation

github-actions · 2025-11-13T01:55:33Z

GNU testsuite comparison:

Skip an intermittent issue tests/timeout/timeout (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)

codspeed-hq · 2025-11-13T02:54:24Z

CodSpeed Performance Report

Merging #9255 will degrade performances by 2.22%

_{Comparing mattsu2020:stty_fix (4f4ec2c) with main (d432131)}

Summary

❌ 1 regression
✅ 122 untouched
⏩ 5 skipped¹

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

	Benchmark	`BASE`	`HEAD`	Change
❌	`du_max_depth_balanced_tree[(6, 4, 10)]`	32.3 ms	33.1 ms	-2.22%

5 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

github-actions · 2025-11-13T03:02:28Z

GNU testsuite comparison:

Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)

mattsu2020 added 13 commits November 12, 2025 20:37

feat(stty): support restoring settings from -g save format

0dd8afb

- Add GNU-compatible parsing of colon-separated hex -g output - Restore termios from saved state before applying new settings - Preserve platform-dependent fields while validating input robustly

fix(stty): correct block indentation for missing_arg handling

8ccd314

- Fix mis-indented else branch to ensure missing_arg is called correctly - Improve readability and maintain consistent control flow structure

fix(stty): correctly parse and apply 64-bit termios flag bits

17abac4

- Cast parsed hex flag values to u64 to match bitflag expectations - Remove redundant conversions when constructing termios flag sets - Ensures proper handling of saved formats on platforms with 64-bit flags

refactor(stty): remove redundant casts in parse_save_format

8c84686

- Use parse_hex_u32 return type directly for flag bitfields - Simplifies code by eliminating unnecessary u32-to-u64 conversions

fix(stty): ensure flag bit masks are converted before truncation

7fd9246

- Cast parsed flag bit values into appropriate integer types - Prevent potential type mismatch or incorrect flag interpretation during restore

fix(stty): parse bitflags using tcflag_t for save format

767a74f

- Use tcflag_t-based parser for hex flag values to match termios types - Avoid extra casts when reconstructing Input/Output/Control/Local flags for correctness and clarity

chore(stty): update spell-checker ignores for new flag terms

1754286

- Add tcflag, bitflag, and lflags to spell-checker ignore list - Prevent false positives when working with recently added flag names

chore(stty): extend spell-check ignore list for cflags

582470a

- Add "cflags" to spell-checker ignore hints in stty.rs - Prevent false positives during spell-checking for terminal flag names

fix(stty): distinguish invalid options from missing args

6d1f375

- Treat unknown dash-prefixed arguments as invalid options to match GNU stty behavior - Treat unrecognized bare words as missing arguments for clearer error reporting

fix(stty): align unknown arg handling with project expectations

0afdf35

- Treat unrecognized options starting with '-' as invalid arguments - Treat unrecognized bare words as invalid arguments instead of missing - Match project test expectations and clarify the inline documentation

Merge branch 'uutils:main' into stty_fix

4f4ec2c

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

stty: Improve option handling and align behavior with GNU coreutils #9255

stty: Improve option handling and align behavior with GNU coreutils #9255

Uh oh!

mattsu2020 commented Nov 13, 2025

Uh oh!

github-actions bot commented Nov 13, 2025

Uh oh!

codspeed-hq bot commented Nov 13, 2025

Uh oh!

github-actions bot commented Nov 13, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

stty: Improve option handling and align behavior with GNU coreutils #9255

Are you sure you want to change the base?

stty: Improve option handling and align behavior with GNU coreutils #9255

Uh oh!

Conversation

mattsu2020 commented Nov 13, 2025

Summary:

Details:

Rationale:

Scope / Impact:

Testing:

Uh oh!

github-actions bot commented Nov 13, 2025

Uh oh!

codspeed-hq bot commented Nov 13, 2025

CodSpeed Performance Report

Merging #9255 will degrade performances by 2.22%

Summary

Benchmarks breakdown

Footnotes

Uh oh!

github-actions bot commented Nov 13, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant