Review fixes

Cargo.toml

@@ -53,7 +53,6 @@ required-features = ["__v7"]
 [dependencies]
 cortex-m = "0.7.0"
 cortex-m-rtic-macros = { path = "macros", version = "0.6.0-alpha.0" }
-# rtic-core = { git = "https://github.com/rtic-rs/rtic-core", branch = "new_monotonic" }
 rtic-monotonic = { git = "https://github.com/rtic-rs/rtic-monotonic", branch = "master" }
 rtic-core = "0.3.1"
 heapless = "0.5.0"

macros/src/codegen.rs

@@ -127,6 +127,7 @@ pub fn app(app: &App, analysis: &Analysis, extra: &Extra) -> TokenStream2 {
                 #[doc(hidden)]
                 pub type #mangled_name = #ty;
 
+                /// This module holds the static implementation for `#name::now()`
                 #[allow(non_snake_case)]
                 pub mod #name {
                     /// Access the global `Monotonic` implementation, not that this will panic
@@ -154,7 +155,7 @@ pub fn app(app: &App, analysis: &Analysis, extra: &Extra) -> TokenStream2 {
     let rt_err = util::rt_err_ident();
 
     quote!(
-        /// Implementation details
+        /// The RTIC application module
         pub mod #name {
             /// Always include the device crate which contains the vector table
             use #device as #rt_err;

macros/src/codegen/module.rs

@@ -53,13 +53,9 @@ pub fn codegen(
 
         Context::Idle => {}
 
-        Context::HardwareTask(..) => {
+        Context::HardwareTask(_) => {}
-            // None for now.
-        }
 
-        Context::SoftwareTask(..) => {
+        Context::SoftwareTask(_) => {}
-            // None for now.
-        }
     }
 
     if ctxt.has_locals(app) {

src/lib.rs

@@ -32,7 +32,7 @@
 #![deny(missing_docs)]
 #![deny(rust_2018_compatibility)]
 #![deny(rust_2018_idioms)]
-// #![deny(warnings)]
+#![deny(warnings)]
 #![no_std]
 
 use cortex_m::{interrupt::InterruptNumber, peripheral::NVIC};