nix icon indicating copy to clipboard operation
nix copied to clipboard

Duplicate definition of clockid

Open maxbla opened this issue 4 years ago • 1 comments

Is there some reason that we have both nix::time::ClockId and nix::sys::timerfd::ClockId? It seems like nix::time::ClockId is more complete - defining more types of clocks for different operating systems. It looks to me like the timerfd implementation could be entirely replaced by the time one.

I would be happy to submit a PR removing nix::sys::timerfd::ClockId (if doing so makes sense). Obviously this would be a breaking change, so it may have to wait.

maxbla avatar Oct 13 '20 08:10 maxbla

It looks like nix::time::ClockId was added fairly recently in https://github.com/nix-rust/nix/pull/1281. I suggest removing nix::sys::timerfd::ClockId when we're next allowed a breaking change.

maxbla avatar Oct 21 '20 21:10 maxbla