Remove revents from PollFd::new

revents is an output field so regardless of what value it is set to it
will be overwritten by many of the function calls that take a PollFd.
The only value that makes sense for the caller to pass in in
`EventFlags::empty()` so we just hardcode that instead of making the
caller do it.

Author: Susurrus

CHANGELOG.md

@@ -95,6 +95,9 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 - Exposed all fcntl(2) operations at the module level, so they can be
   imported direclty instead of via `FcntlArg` enum.
   ([#541](https://github.com/nix-rust/nix/pull/541))
+- Removed `revents` argument from `PollFd::new()` as it's an output argument and
+  will be overwritten regardless of value.
+  ([#542](https://github.com/nix-rust/nix/pull/542)
 
 ### Fixed
 - Fixed multiple issues with Unix domain sockets on non-Linux OSes

src/poll.rs

@@ -13,12 +13,12 @@ pub struct PollFd {
 }
 
 impl PollFd {
-    pub fn new(fd: libc::c_int, events: EventFlags, revents: EventFlags) -> PollFd {
+    pub fn new(fd: libc::c_int, events: EventFlags) -> PollFd {
         PollFd {
             pollfd: libc::pollfd {
                 fd: fd,
                 events: events.bits(),
-                revents: revents.bits(),
+                revents: EventFlags::empty().bits(),
             },
         }
     }