more lints (#648)

Author: Stephan Dilly

Cargo.lock

@@ -20,6 +20,7 @@ log = "0.4"
 thiserror = "1.0"
 url = "2.2"
 unicode-truncate = "0.2.0"
+easy-cast = "0.4"
 
 [dev-dependencies]
 tempfile = "3.2"

asyncgit/Cargo.toml

@@ -32,6 +32,9 @@ pub enum Error {
 
     #[error("TryFromInt error:{0}")]
     IntConversion(#[from] TryFromIntError),
+
+    #[error("EasyCast error:{0}")]
+    EasyCast(#[from] easy_cast::Error),
 }
 
 pub type Result<T> = std::result::Result<T, Error>;

asyncgit/src/error.rs

@@ -16,15 +16,9 @@
 #![deny(clippy::needless_update)]
 #![allow(clippy::module_name_repetitions)]
 #![allow(clippy::must_use_candidate)]
+#![allow(clippy::missing_errors_doc)]
 //TODO: get this in someday since expect still leads us to crashes sometimes
 // #![deny(clippy::expect_used)]
-//TODO:
-#![allow(clippy::missing_errors_doc)]
-#![allow(clippy::too_many_lines)]
-#![allow(clippy::cast_possible_wrap)]
-#![allow(clippy::cast_sign_loss)]
-#![allow(clippy::cast_possible_truncation)]
-#![allow(clippy::cast_precision_loss)]
 
 pub mod cached;
 mod commit_files;

asyncgit/src/lib.rs

@@ -1,5 +1,6 @@
 //!
 
+use easy_cast::{Conv, ConvFloat};
 use std::cmp;
 
 ///
@@ -12,9 +13,9 @@ pub struct ProgressPercent {
 impl ProgressPercent {
     ///
     pub fn new(current: usize, total: usize) -> Self {
-        let total = cmp::max(current, total) as f32;
-        let progress = current as f32 / total * 100.0;
-        let progress = progress as u8;
+        let total = f64::conv(cmp::max(current, total));
+        let progress = f64::conv(current) / total * 100.0;
+        let progress = u8::conv_nearest(progress);
         Self { progress }
     }
     ///

asyncgit/src/progress.rs

@@ -14,11 +14,11 @@ use std::{
     time::{SystemTime, UNIX_EPOCH},
 };
 
-fn current_tick() -> u64 {
+fn current_tick() -> u128 {
     SystemTime::now()
         .duration_since(UNIX_EPOCH)
         .expect("time before unix epoch!")
-        .as_millis() as u64
+        .as_millis()
 }
 
 #[derive(Default, Hash, Clone)]
@@ -29,7 +29,7 @@ pub struct Status {
 ///
 #[derive(Default, Hash, Copy, Clone, PartialEq)]
 pub struct StatusParams {
-    tick: u64,
+    tick: u128,
     status_type: StatusType,
     include_untracked: bool,
 }

asyncgit/src/status.rs

@@ -6,6 +6,7 @@ use super::{
     CommitId,
 };
 use crate::{error::Error, error::Result, hash};
+use easy_cast::Conv;
 use git2::{
     Delta, Diff, DiffDelta, DiffFormat, DiffHunk, DiffOptions, Patch,
     Repository,
@@ -186,6 +187,8 @@ pub fn get_diff_commit(
 }
 
 ///
+//TODO: refactor into helper type with the inline closures as dedicated functions
+#[allow(clippy::too_many_lines)]
 fn raw_diff_to_file_diff<'a>(
     diff: &'a Diff,
     work_dir: &Path,
@@ -216,8 +219,9 @@ fn raw_diff_to_file_diff<'a>(
                     delta.old_file().size(),
                     delta.new_file().size(),
                 );
-                res.size_delta = (res.sizes.1 as i64)
-                    .saturating_sub(res.sizes.0 as i64);
+                //TODO: use try_conv
+                res.size_delta = (i64::conv(res.sizes.1))
+                    .saturating_sub(i64::conv(res.sizes.0));
             }
             if let Some(hunk) = hunk {
                 let hunk_header = HunkHeader::from(hunk);

asyncgit/src/sync/diff.rs

@@ -6,6 +6,7 @@ use crate::{
         patches::get_file_diff_patch_and_hunklines, utils::repo,
     },
 };
+use easy_cast::Conv;
 use scopetime::scope_time;
 use std::path::Path;
 
@@ -49,7 +50,7 @@ pub fn stage_lines(
     let blob_id = repo.blob(new_content.as_bytes())?;
 
     idx.id = blob_id;
-    idx.file_size = new_content.as_bytes().len() as u32;
+    idx.file_size = u32::try_conv(new_content.as_bytes().len())?;
     //TODO: can we simply use add_frombuffer?
     index.add(&idx)?;