Skip to content

Replace use of `RawFd` with `File` for `RefinedJob`

Michael Aaron Murphy requested to merge huntergoldstein:replace_rawfd_with_file into master

Created by: huntergoldstein

Changes introduced by this pull request:

  • Standard streams are now represented as Files instead of RawFds for RefinedJob::Builtins
  • Removed custom drop implementation for RefinedJob

Drawbacks: Not having a custom Drop implementation is nice, but now there's a lot of unsafe { File::from_raw_fd(...) } running about which seems like a lateral move from before.

Merge request reports