Move lints into Cargo.toml

Author: uklotzde

Cargo.toml

@@ -19,3 +19,31 @@ smallvec = { version = "1.13.1", features = ["union"] }
 [features]
 default = ["std"]
 std = []
+
+[lints.rust]
+future_incompatible = "warn"
+let_underscore = "warn"
+missing_debug_implementations = "warn"
+rust_2018_idioms = "warn"
+rust_2021_compatibility = "warn"
+unreachable_pub = "warn"
+unsafe_code = "warn"
+unused = "warn"
+
+[lints.clippy]
+pedantic = "warn"
+clone_on_ref_ptr = "warn"
+missing_const_for_fn = "warn"
+self_named_module_files = "warn"
+
+# Repetitions of module/type names occur frequently when using many
+# modules for keeping the size of the source files handy. Often
+# types have the same name as their parent module.
+module_name_repetitions = "allow"
+
+# Repeating the type name in `Default::default()` expressions
+# is not needed as long as the context is obvious.
+default_trait_access = "allow"
+
+# The error types returned should be self-explanatory.
+missing_errors_doc = "allow"

examples/reservation.rs

@@ -46,7 +46,7 @@ impl PhoneNumber {
 }
 
 impl PhoneNumber {
-    pub fn len(&self) -> usize {
+    fn len(&self) -> usize {
         self.0.chars().filter(|c| !c.is_whitespace()).count()
     }
 }
@@ -137,7 +137,7 @@ impl Quantity {
         Self(1)
     }
 
-    fn new(value: usize) -> Self {
+    const fn new(value: usize) -> Self {
         Self(value)
     }
 }

src/lib.rs

@@ -1,26 +1,6 @@
 // SPDX-FileCopyrightText: slowtec GmbH
 // SPDX-License-Identifier: MPL-2.0
 
-// Opt-in for allowed-by-default lints (in alphabetical order)
-// See also: <https://doc.rust-lang.org/rustc/lints>
-#![warn(future_incompatible)]
-#![warn(let_underscore)]
-#![warn(missing_debug_implementations)]
-#![warn(rust_2018_idioms)]
-#![warn(rust_2021_compatibility)]
-#![warn(unreachable_pub)]
-#![warn(unsafe_code)]
-#![warn(unused)]
-// Clippy lints
-#![warn(clippy::pedantic)]
-// Additional restrictions
-#![warn(clippy::clone_on_ref_ptr)]
-#![warn(clippy::missing_const_for_fn)]
-#![warn(clippy::mod_module_files)]
-// Repeating the type name in `..Default::default()` expressions
-// is not needed since the context is obvious.
-#![allow(clippy::default_trait_access)]
-// Opt-out of feature "std"
 #![cfg_attr(not(feature = "std"), no_std)]
 
 //! # semval