nix icon indicating copy to clipboard operation
nix copied to clipboard

impl TryFrom<i32> for `nix::mman::MmapAdvise`

Open acully-vmware opened this issue 2 years ago • 4 comments
trafficstars

We have a use-case where we are intercepting the madvise syscall, and wish to handle certain types of advice specially. Without TryFrom, we need to write boilerplate to safely translate the application-provided i32 into a form nix::mman::madvise will take, and I'm not sure why madvise can't simply implement TryFrom itself?

acully-vmware avatar Jan 10 '23 18:01 acully-vmware

(For what it's worth, I can't see how the failed check can possibly be related to this change?)

acully-vmware avatar Jan 10 '23 18:01 acully-vmware

The CI failure was spurious, and I've restarted it. The change looks reasonable, but could you please add an entry to the CHANGELOG?

asomers avatar Jan 11 '23 22:01 asomers

The CI failure was spurious, and I've restarted it. The change looks reasonable, but could you please add an entry to the CHANGELOG?

Done.

acully-vmware avatar Jan 18 '23 15:01 acully-vmware

Friendly ping @acully-vmware, sorry for such a slow response, would you like to finish this PR?

You need to rebase your branch and adjust the location of the CHANGELOG entry (Put it under the new [Unreleased] - ReleaseDate ### Added section)

SteveLauC avatar Oct 06 '23 07:10 SteveLauC