remove deny lints

Author: burrbull

.github/workflows/ci.yml

@@ -105,6 +105,7 @@ jobs:
           VENDOR: ${{ matrix.vendor }}
           OPTIONS: ${{ matrix.options }}
           COMMAND: check
+          RUST_TOOLCHAIN: ${{ matrix.rust }}
         run: bash ci/script.sh
 
   ci-clippy:

CHANGELOG.md

@@ -7,6 +7,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/).
 
 ## [Unreleased]
 
+- Remove all deny lints from generated crate
 - Add `reexport_core_peripherals` and `reexport_interrupt` features disabled by default
 - rename `const-generic` feature to `array_proxy`
 - `FieldWriter` takes offset as struct field instead of const generic.

ci/script.sh

@@ -40,14 +40,33 @@ main() {
 
     # test crate
     cargo init --name foo $td
-    echo 'cortex-m = "0.7.7"' >> $td/Cargo.toml
+    echo 'cortex-m = "0.7.7"'    >> $td/Cargo.toml
     echo 'cortex-m-rt = "0.7.3"' >> $td/Cargo.toml
-    echo 'vcell = "0.1.3"' >> $td/Cargo.toml
+    echo 'vcell = "0.1.3"'       >> $td/Cargo.toml
     if [[ "$options" == *"--atomics"* ]]; then
         echo 'portable-atomic = { version = "1.4", default-features = false }' >> $td/Cargo.toml
     fi
-    echo '[profile.dev]' >> $td/Cargo.toml
-    echo 'incremental = false' >> $td/Cargo.toml
+    echo '[profile.dev]'         >> $td/Cargo.toml
+    echo 'incremental = false'   >> $td/Cargo.toml
+
+    echo '[lints.rust]'          >> $td/Cargo.toml
+    echo 'dead_code = "deny"'                    >> $td/Cargo.toml
+    echo 'improper_ctypes = "deny"'              >> $td/Cargo.toml
+    echo 'missing_docs = "deny"'                 >> $td/Cargo.toml
+    echo 'no_mangle_generic_items = "deny"'      >> $td/Cargo.toml
+    echo 'non_shorthand_field_patterns = "deny"' >> $td/Cargo.toml
+    echo 'overflowing_literals = "deny"'         >> $td/Cargo.toml
+    echo 'path_statements = "deny"'              >> $td/Cargo.toml
+    echo 'patterns_in_fns_without_body = "deny"' >> $td/Cargo.toml
+    echo 'unconditional_recursion = "deny"'      >> $td/Cargo.toml
+    echo 'unused_allocation = "deny"'            >> $td/Cargo.toml
+    echo 'unused_comparisons = "deny"'           >> $td/Cargo.toml
+    echo 'unused_parens = "deny"'                >> $td/Cargo.toml
+    echo 'while_true = "deny"'                   >> $td/Cargo.toml
+    if [[ "$RUST_TOOLCHAIN" == *"nightly"* ]]; then
+        echo 'private_bounds = "deny"'           >> $td/Cargo.toml
+        echo 'private_interfaces = "deny"'       >> $td/Cargo.toml
+    fi
 
     case $VENDOR in
         Atmel)

src/generate/device.rs

@@ -51,20 +51,6 @@ pub fn render(d: &Device, config: &Config, device_x: &mut String) -> Result<Toke
     out.extend(quote! { #![doc = #doc] });
     if !config.make_mod {
         out.extend(quote! {
-            // Deny a subset of warnings
-            #![deny(dead_code)]
-            #![deny(improper_ctypes)]
-            #![deny(missing_docs)]
-            #![deny(no_mangle_generic_items)]
-            #![deny(non_shorthand_field_patterns)]
-            #![deny(overflowing_literals)]
-            #![deny(path_statements)]
-            #![deny(patterns_in_fns_without_body)]
-            #![deny(unconditional_recursion)]
-            #![deny(unused_allocation)]
-            #![deny(unused_comparisons)]
-            #![deny(unused_parens)]
-            #![deny(while_true)]
             // Explicitly allow a few warnings that may be verbose
             #![allow(non_camel_case_types)]
             #![allow(non_snake_case)]