Run fmt and clippy

Author: ghaith

compiler/plc_driver/src/lib.rs

@@ -19,8 +19,8 @@ use std::{
 
 use cli::{CompileParameters, ParameterError, SubCommands};
 use plc::{
-    codegen::CodegenContext, linker::LinkerType, output::FormatOption, DebugLevel, ErrorFormat,
-    OnlineChange, OptimizationLevel, Target, Threads,
+    codegen::CodegenContext, linker::LinkerType, output::FormatOption, DebugLevel, ErrorFormat, OnlineChange,
+    OptimizationLevel, Target, Threads,
 };
 
 use plc_diagnostics::{diagnostician::Diagnostician, diagnostics::Diagnostic};
@@ -179,7 +179,10 @@ pub fn get_compilation_context<T: AsRef<str> + AsRef<OsStr> + Debug>(
         debug_level: compile_parameters.debug_level(),
         single_module: compile_parameters.single_module,
         online_change: if compile_parameters.online_change {
-            OnlineChange::Enabled((compile_parameters.got_layout_file.clone(),compile_parameters.got_layout_format()))
+            OnlineChange::Enabled((
+                compile_parameters.got_layout_file.clone(),
+                compile_parameters.got_layout_format(),
+            ))
         } else {
             OnlineChange::Disabled
         },

src/codegen.rs

@@ -102,12 +102,7 @@ impl<'ink> CodeGen<'ink> {
         let module = context.create_module(module_location);
         module.set_source_file_name(module_location);
         let debug = debug::DebugBuilderEnum::new(context, &module, root, optimization_level, debug_level);
-        CodeGen {
-            module,
-            debug,
-            module_location: module_location.to_string(),
-            online_change,
-        }
+        CodeGen { module, debug, module_location: module_location.to_string(), online_change }
     }
 
     pub fn generate_llvm_index(
@@ -131,14 +126,8 @@ impl<'ink> CodeGen<'ink> {
         )?;
         index.merge(llvm_type_index);
 
-        let mut variable_generator = VariableGenerator::new(
-            &self.module,
-            &llvm,
-            global_index,
-            annotations,
-            &index,
-            &mut self.debug,
-        );
+        let mut variable_generator =
+            VariableGenerator::new(&self.module, &llvm, global_index, annotations, &index, &mut self.debug);
 
         //Generate global variables
         let llvm_gv_index =
@@ -177,8 +166,7 @@ impl<'ink> CodeGen<'ink> {
                 acc
             });
 
-
-        if let OnlineChange::Enabled((_,_)) = self.online_change {
+        if let OnlineChange::Enabled((_, _)) = self.online_change {
             let got_entries = &mut *got_layout.lock().unwrap();
 
             let mut new_symbols = Vec::new();

src/codegen/generators/pou_generator.rs

@@ -55,6 +55,8 @@ pub struct PouGenerator<'ink, 'cg> {
 
 /// Creates opaque implementations for all callable items in the index
 /// Returns a Typed index containing the associated implementations.
+/// FIXME: Ignoring the arguments warning for now, should refactor later to pass the options in a struct
+#[allow(clippy::too_many_arguments)]
 pub fn generate_implementation_stubs<'ink>(
     module: &Module<'ink>,
     llvm: Llvm<'ink>,

src/codegen/generators/variable_generator.rs

@@ -43,14 +43,7 @@ impl<'ctx, 'b> VariableGenerator<'ctx, 'b> {
         types_index: &'b LlvmTypedIndex<'ctx>,
         debug: &'b mut DebugBuilderEnum<'ctx>,
     ) -> Self {
-        VariableGenerator {
-            module,
-            llvm,
-            global_index,
-            annotations,
-            types_index,
-            debug,
-        }
+        VariableGenerator { module, llvm, global_index, annotations, types_index, debug }
     }
 
     pub fn generate_global_variables(