Closed magicant closed 3 weeks ago
The recent changes enhance the clarity and consistency of signal management within the codebase. Key modifications include renaming terms from "signal handling" to "signal disposition" across multiple files, updating function names and parameters, and refining documentation. This shift aims to improve semantic alignment and reduce confusion in signal management functionality while maintaining existing behavior.
Files | Change Summary |
---|---|
yash-builtin/src/set.rs, trap.rs, wait/core.rs, yash-cli/src/lib.rs, startup.rs | Renamed signal handling functions and types from "SignalHandling" to "Disposition," improving terminology clarity and consistency across signal management. |
yash-env/* | Unified signal management terminology, replacing "SignalHandling" with "Disposition," and updated method signatures and logic in the System trait and related files. |
yash-env/src/system/* | Renamed methods and parameters to reflect the new "Disposition" terminology, modifying internal logic while maintaining functionality. |
yash-env/src/trap/* | Replaced "signal handling" with "signal disposition" across various functions and types, updating documentation and ensuring test cases align with new terms. |
yash-semantics/src/command/simple_command/external.rs | Updated documentation and function calls for clarity, aligning language with the new terminology while preserving existing functionality. |
sequenceDiagram
participant User
participant Shell
participant System
User->>Shell: Execute command
Shell->>System: Set signal disposition
System-->>Shell: Confirm disposition set
Shell-->>User: Command executed
π "In the realm of signals, we take a new flight,
Dispositions now guide us, shining so bright.
With clarity blooming, confusion takes flight,
Hopping through changes, our code feels just right!
So hereβs to the journey, our code in delight,
A rabbitβs cheer echoes, coding day and night!" π
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
This pull request renames the
SignalHandling
enum toDisposition
and also renames related functions and variables. The word "disposition" is used in the POSIX standard to exclusively describe the action taken when a signal is received. The previous nameSignalHandling
was misleading because it could be interpreted as handling the signal itself, which is not the case. This change makes the code more readable and consistent with the POSIX standard.Summary by CodeRabbit
New Features
Bug Fixes
Documentation
Tests
Refactor