review changes

Author: marmistrz

CHANGELOG.md

@@ -12,7 +12,7 @@ This project adheres to [Semantic Versioning](http://semver.org/).
   ([#672](https://github.com/nix-rust/nix/pull/672))
 - Added protocol families in `AddressFamily` enum.
   ([#647](https://github.com/nix-rust/nix/pull/647))
-- Added a convenience method `.pid()` to the `enum WaitStatus`
+- Added the `pid()` method to `WaitStatus` for extracting the PID.
   ([#722](https://github.com/nix-rust/nix/pull/722))
 
 ### Changed

src/sys/wait.rs

@@ -94,20 +94,20 @@ pub enum WaitStatus {
 }
 
 impl WaitStatus {
-    /// Extracts the PID from the WaitStatus if the status is not StillAlive.
+    /// Extracts the PID from the WaitStatus unless it equals StillAlive.
     pub fn pid(&self) -> Option<Pid> {
         use self::WaitStatus::*;
-        match self {
-            &Exited(p, _) => Some(p),
-            &Signaled(p, _, _) => Some(p),
-            &Stopped(p, _) => Some(p),
-            &Continued(p) => Some(p),
-            &StillAlive => None,
+        match *self {
+            Exited(p, _) => Some(p),
+            Signaled(p, _, _) => Some(p),
+            Stopped(p, _) => Some(p),
+            Continued(p) => Some(p),
+            StillAlive => None,
 
             #[cfg(any(target_os = "linux", target_os = "android"))]
-            &PtraceEvent(p, _, _) => Some(p),
+            PtraceEvent(p, _, _) => Some(p),
             #[cfg(any(target_os = "linux", target_os = "android"))]
-            &PtraceSyscall(p) => Some(p),
+            PtraceSyscall(p) => Some(p),
         }
     }
 }