Merge #1189

1189: Derive `Ord`, `PartialOrd` for `Pid` r=posborne a=ranweiler Closes #1188. Co-authored-by: Joe Ranweiler <joe@lemma.co>
nix-rust · Mar 3, 2020 · 837fb44 · 837fb44
2 parents 2a40283 + 2ff30d8
commit 837fb44
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -14,6 +14,8 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 - Added `unistd::setfsuid` and `unistd::setfsgid` to set the user or group
   identity for filesystem checks per-thread.
   (#[1163](https://github.com/nix-rust/nix/pull/1163))
+- Derived `Ord`, `PartialOrd` for `unistd::Pid` (#[1189](https://github.com/nix-rust/nix/pull/1189))
+
 ### Changed
 ### Fixed
 ### Removed

diff --git a/src/unistd.rs b/src/unistd.rs
@@ -115,7 +115,7 @@ impl fmt::Display for Gid {
 ///
 /// Newtype pattern around `pid_t` (which is just alias). It prevents bugs caused by accidentally
 /// passing wrong value.
-#[derive(Debug, Copy, Clone, Eq, PartialEq, Hash)]
+#[derive(Debug, Copy, Clone, Eq, PartialEq, Ord, PartialOrd, Hash)]
 pub struct Pid(pid_t);
 
 impl Pid {