Merge #214

214: [0.2.x]: Fix warning, remove #[deny(warnings)] but keep check on CI r=ryankurte a=eldruin

This fixes #213. Additionally I removed #[deny(warnings)] but added a check for it on CI.
I think we should cherry pick #138 to the 0.2.x branch as well and release a 0.2.4 version, since 1.0 will still take a while.
I can do that in a separate PR.

Co-authored-by: Diego Barrios Romero <[email protected]>

Author: bors[bot]

.travis.yml

@@ -1,5 +1,9 @@
 language: rust
 
+env:
+  global:
+    RUSTFLAGS='-D warnings'
+
 matrix:
   include:
     - env: TARGET=x86_64-unknown-linux-gnu

src/fmt.rs

@@ -3,7 +3,7 @@
 //! TODO write example of usage
 use core::fmt::{Result, Write};
 
-impl<Word, Error> Write for ::serial::Write<Word, Error=Error>
+impl<Word, Error> Write for dyn (::serial::Write<Word, Error=Error>)
 where
     Word: From<u8>,
 {

src/lib.rs

@@ -683,7 +683,6 @@
 //! ```
 
 #![deny(missing_docs)]
-#![deny(warnings)]
 #![no_std]
 
 #[macro_use]