SyncReader next() behavior synced

.travis.yml

@@ -1,3 +1,9 @@
+# Build only pushed (merged) master or any pull request. This avoids the
+# pull request to be build twice.
+branches:
+  only:
+    - master
+
 language: rust
 
 rust:

CHANGELOG.md

@@ -3,7 +3,11 @@
 - Internal refactoring (no breaking changes)
 - Removed unsafe `static mut`
 - Documentation update
-- Fixed the `KeyEvent` `BackTab` vs `Tab` bug on the Windows platform 
+
+## Windows only
+
+- Fixed the `KeyEvent` `BackTab` vs `Tab` bug
+- `SyncReader` `Iterator` implementation returns `None` from the `next` method in case of error 
 
 # Version 0.4.1
 

src/input/windows.rs

@@ -207,8 +207,9 @@ impl Iterator for SyncReader {
     /// `None` doesn't mean that the iteration is finished. See the
     /// [`SyncReader`](struct.SyncReader.html) documentation for more information.
     fn next(&mut self) -> Option<Self::Item> {
-        // TODO Don't unwrap, consume and return `None` to sync behavior with the UNIX
-        read_single_event().unwrap()
+        // This synces the behaviour with the unix::SyncReader (& documentation) where
+        // None is returned in case of error.
+        read_single_event().unwrap_or(None)
     }
 }