Fix new clippy warnings

Author: weiznich

clippy.toml

@@ -8,3 +8,4 @@ doc-valid-idents = [
   "JavaScript", "NaN", "OAuth",
   "SQLite", "PostgreSQL", "MySQL"
 ]
+avoid-breaking-exported-api = false

diesel/src/lib.rs

@@ -113,7 +113,6 @@
 #![warn(
     clippy::unwrap_used,
     clippy::print_stdout,
-    clippy::wrong_pub_self_convention,
     clippy::mut_mut,
     clippy::non_ascii_literal,
     clippy::similar_names,

diesel/src/mysql/connection/mod.rs

@@ -48,7 +48,7 @@ impl CommitErrorProcessor for MysqlConnection {
             }
             TransactionManagerStatus::Valid(ref v) => v,
         };
-        default_process_commit_error(&state, error)
+        default_process_commit_error(state, error)
     }
 }
 

diesel/src/pg/expression/expression_methods.rs

@@ -27,7 +27,7 @@ pub trait PgExpressionMethods: Expression + Sized {
     /// assert_eq!(Ok(1), not_distinct.first(connection));
     /// # }
     /// ```
-    #[allow(clippy::clippy::wrong_self_convention)] // This is named after the sql operator
+    #[allow(clippy::wrong_self_convention)] // This is named after the sql operator
     fn is_not_distinct_from<T>(self, other: T) -> dsl::IsNotDistinctFrom<Self, T>
     where
         Self::SqlType: SqlType,
@@ -55,7 +55,7 @@ pub trait PgExpressionMethods: Expression + Sized {
     /// assert_eq!(Ok(1), not_distinct.first(connection));
     /// # }
     /// ```
-    #[allow(clippy::clippy::wrong_self_convention)] // This is named after the sql operator
+    #[allow(clippy::wrong_self_convention)] // This is named after the sql operator
     fn is_distinct_from<T>(self, other: T) -> dsl::IsDistinctFrom<Self, T>
     where
         Self::SqlType: SqlType,
@@ -292,7 +292,7 @@ pub trait PgArrayExpressionMethods: Expression + Sized {
     /// #     Ok(())
     /// # }
     /// ```
-    #[allow(clippy::clippy::wrong_self_convention)] // This is named after the sql operator
+    #[allow(clippy::wrong_self_convention)] // This is named after the sql operator
     fn is_contained_by<T>(self, other: T) -> dsl::IsContainedBy<Self, T>
     where
         Self::SqlType: SqlType,
@@ -870,7 +870,7 @@ pub trait PgNetExpressionMethods: Expression + Sized {
     /// #     Ok(())
     /// # }
     /// ```
-    #[allow(clippy::clippy::wrong_self_convention)] // This is named after the sql operator
+    #[allow(clippy::wrong_self_convention)] // This is named after the sql operator
     fn is_contained_by<T>(self, other: T) -> dsl::IsContainedByNet<Self, T>
     where
         T: AsExpression<Inet>,
@@ -928,7 +928,7 @@ pub trait PgNetExpressionMethods: Expression + Sized {
     /// #     Ok(())
     /// # }
     /// ```
-    #[allow(clippy::clippy::wrong_self_convention)] // This is named after the sql operator
+    #[allow(clippy::wrong_self_convention)] // This is named after the sql operator
     fn is_contained_by_or_eq<T>(self, other: T) -> dsl::IsContainedByNetLoose<Self, T>
     where
         T: AsExpression<Inet>,

diesel/src/pg/metadata_lookup.rs

@@ -46,7 +46,7 @@ where
             }
         }
 
-        let r = lookup_type(&&cache_key, self);
+        let r = lookup_type(&cache_key, self);
 
         match r {
             Ok(type_metadata) => {

diesel/src/sqlite/connection/raw.rs

@@ -245,7 +245,7 @@ impl SqliteCallbackError {
                 s = e.to_string();
                 &s
             }
-            SqliteCallbackError::Panic(_, msg) => &msg,
+            SqliteCallbackError::Panic(_, msg) => msg,
         };
         unsafe {
             context_error_str(ctx, msg);

diesel/src/sqlite/expression/expression_methods.rs

@@ -70,7 +70,7 @@ pub trait SqliteExpressionMethods: Expression + Sized {
     /// #     Ok(())
     /// # }
     /// ```
-    #[allow(clippy::clippy::wrong_self_convention)] // This is named after the sql operator
+    #[allow(clippy::wrong_self_convention)] // This is named after the sql operator
     fn is_not<T>(self, other: T) -> dsl::IsNot<Self, T>
     where
         Self::SqlType: SqlType,

diesel/src/util/once_cell.rs

@@ -84,7 +84,7 @@ impl<T> OnceCell<T> {
         // `assert`, while keeping `set/get` would be sound, but it seems
         // better to panic, rather than to silently use an old value.
         assert!(self.set(val).is_ok(), "reentrant init");
-        self.get().unwrap()
+        self.get().expect("We set the value in the line above")
     }
 
     pub(crate) fn get(&self) -> Option<&T> {

diesel_cli/src/infer_schema_internals/inference.rs

@@ -204,7 +204,7 @@ pub fn load_table_data(
     let primary_key = get_primary_keys(&mut connection, &name)?;
     let primary_key = primary_key
         .iter()
-        .map(|k| rust_name_for_sql_name(&k))
+        .map(|k| rust_name_for_sql_name(k))
         .collect();
 
     let column_data = get_column_information(&mut connection, &name, column_sorting)?

diesel_cli/src/infer_schema_internals/sqlite.rs

@@ -109,7 +109,7 @@ impl SqliteVersion {
 
 fn get_sqlite_version(conn: &mut SqliteConnection) -> SqliteVersion {
     let query = "SELECT sqlite_version()";
-    let result = sql::<sql_types::Text>(&query).load::<String>(conn).unwrap();
+    let result = sql::<sql_types::Text>(query).load::<String>(conn).unwrap();
     let parts = result[0]
         .split('.')
         .map(|part| part.parse().unwrap())