chore: Bump dependency versions, apply clippy and fmt (#1282)

Author: volsa

Cargo.lock

@@ -9,8 +9,13 @@ use crate::literals::AstLiteral;
 /// Macro that calls the visitor's `visit` method for every AstNode in the passed iterator `iter`.
 macro_rules! visit_all_nodes {
     ($visitor:expr, $iter:expr) => {
-        for node in $iter {
-            $visitor.visit(node);
+        // Note: The `allow` is needed to suppress warnings about `while let Some(...)` warnings
+        // because `visit_all_nodes!` is used for both Option and Non-Option types
+        #[allow(warnings)]
+        {
+            for node in $iter {
+                $visitor.visit(node);
+            }
         }
     };
 }

compiler/plc_ast/src/visitor.rs

@@ -1087,11 +1087,11 @@ impl<'ink, 'b> ExpressionCodeGenerator<'ink, 'b> {
         }
     }
 
+    // TODO: will be deleted once methods work properly (like functions)
     /// generates a new instance of a function called `function_name` and returns a PointerValue to it
     ///
     /// - `function_name` the name of the function as registered in the index
     /// - `context` the statement used to report a possible Diagnostic on
-    /// TODO: will be deleted once methods work properly (like functions)
     fn allocate_function_struct_instance(
         &self,
         function_name: &str,

src/codegen/generators/expression_generator.rs

@@ -868,8 +868,7 @@ impl Index {
     ///
     /// imports all global_variables, types and implementations
     /// # Arguments
-    /// - `other` the other index. The elements are drained from the given index and moved
-    /// into the current one
+    /// - `other` the other index. The elements are drained from the given index and moved into the current one
     pub fn import(&mut self, mut other: Index) {
         //global variables
         for (name, e) in other.global_variables.drain(..) {

src/index.rs

@@ -17,7 +17,7 @@ use crate::test_utils::tests::parse_and_preprocess;
 /// - Casting a value
 /// - Dereferencing a pointer
 /// - accessing the Address of a variable
-
+///
 /// A flat reference is treated as a Qualified-Reference with no qualifier.
 /// This means that a flat reference (a) and a qualified reference (a.b)
 /// are represented using the same AST-Structure. One has no qualifier (None),

src/tests/adr/reference_expressions_adr.rs

@@ -135,7 +135,7 @@ impl RecursiveValidator {
     /// Returns the data type of `entry` distinguishing between two cases:
     /// 1. If the entry is any type but an array its datatype is returned (as usual)
     /// 2. If the entry is an array their inner type name is returned. For example calling the
-    /// [`index::VariableIndexEntry::get_type_name`] method on the following code snippet
+    ///    [`index::VariableIndexEntry::get_type_name`] method on the following code snippet
     /// ```ST
     /// TYPE A : STRUCT
     ///     b : ARRAY[0..1] OF B;

src/validation/recursive.rs

@@ -315,12 +315,6 @@ fn initialization_of_string_arrays() {
     assert_eq!(main_type.z, "ten chars!\0".as_bytes());
 }
 
-#[repr(C, align(1))]
-#[derive(Debug)]
-struct Wrapper<T> {
-    inner: T,
-}
-
 /// .
 ///
 /// # Safety

tests/correctness/strings.rs

@@ -35,6 +35,7 @@ pub enum ReporterType {
 
 #[derive(Serialize)]
 pub struct BenchmarkReport {
+    #[allow(dead_code)]
     #[serde(skip_serializing)]
     pub id: u64,
     /// Host information, see [`Host`].
@@ -67,6 +68,7 @@ impl Serialize for DurationWrapper {
 
 #[derive(Serialize, Debug)]
 pub struct Host {
+    #[allow(dead_code)]
     #[serde(skip_serializing)]
     pub id: u64,
     pub os: String,