Delete

.github/workflows/quality.yml

@@ -20,11 +20,7 @@ jobs:
       - name: Check the format
         run: cargo fmt --all -- --check
 
-      # type complexity must be ignored because we use huge templates for queries
+      # Clippy overly_complex_bool_expr is disabled because mess with generated files in target dir
+      # and I cannot disable it
       - name: Run clippy
-        run: >
-          cargo clippy
-          --all-targets
-          --all-features
-          --
-          -D warnings
+        run: cargo clippy --all-targets --all-features -- -A clippy::overly_complex_bool_expr -D warnings

czkawka_slint_gui/src/connect_delete.rs

@@ -20,7 +20,7 @@ pub fn connect_delete_button(app: &MainWindow) {
                 *selected_row = false;
             });
             let r = ModelRc::new(VecModel::from(entries_left));
-            app.set_empty_folder_model(r.into());
+            app.set_empty_folder_model(r);
         }
     });
 }

czkawka_slint_gui/src/connect_open.rs

@@ -1,6 +1,4 @@
 use crate::MainWindow;
-use slint::{ComponentHandle, Model, ModelRc, SharedString, VecModel};
-use std::borrow::BorrowMut;
 
 pub fn connect_open_items(app: &MainWindow) {
     app.on_item_opened(move |path| {

czkawka_slint_gui/src/main.rs

@@ -24,15 +24,16 @@ fn main() {
     app.run().unwrap();
 }
 
+type ModelType = VecModel<(bool, bool, bool, ModelRc<SharedString>)>;
 // TODO remove this after trying
 pub fn to_remove_debug(app: &MainWindow) {
-    let row_data: Rc<VecModel<(bool, bool, bool, ModelRc<SharedString>)>> = Rc::new(VecModel::default());
+    let row_data: Rc<ModelType> = Rc::new(VecModel::default());
 
-    for r in 0..100000 {
+    for r in 0..100_000 {
         let items = VecModel::default();
 
         for c in 0..3 {
-            items.push(slint::format!("Item {r}.{c}").into());
+            items.push(slint::format!("Item {r}.{c}"));
         }
 
         row_data.push((r % 2 == 0, r % 3 == 0, false, ModelRc::new(items)));