coderabbitai[bot]
commented
7 months ago Walkthrough
The overarching change across multiple files in the yash-env and yash-semantics modules involves transitioning from using nix::Error to a more specific Errno type for error handling. This move standardizes error handling across the system and semantics layers, improving the clarity and specificity of errors. Functions and methods have been updated to return a custom Result type leveraging Errno, and error messages have been refined to use to_string() for more consistent and readable output.
Changes
| Files | Summary |
|---|---|
src/input.rs, src/system/fd_set.rs, src/system/virtual/io.rs |
No detailed summary provided. |
src/lib.rs, src/pwd.rs, src/semantics.rs, src/subshell.rs, src/system.rs, src/system/real.rs, src/system/virtual.rs, src/system/virtual/file_system.rs, src/command/compound_command/subshell.rs, src/command/item.rs, src/command/pipeline.rs, src/command/simple_command/absent.rs, src/command/simple_command/external.rs, src/expansion.rs, src/redir.rs |
Transitioned from nix::Error to Errno for error handling, updated function return types to use crate::system::Result and Errno, and refined error messages using to_string(). |
🐇✨
In the realm of code where errors intertwine,
A rabbit hopped through, making things align.
nix::Errorfades,Errnotakes its place,
With clearer messages, errors we now face.
Through files and functions, changes bound,
A tale of improvement, inyashfound.
🌟🐾
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 PR is part of #353.
This PR introduces the
Errnotype that wraps a rawerrnovalue. This new type replaces all uses ofnix::errno::Errnoexcept in the following implementations:Errnotype,RealSystem.