bump deps and make clippy happy (#57)

- rand 0.8
- `#[allow(clippy::vtable_address_comparisons)]` for SyncCallback
- `compare_and_swap` -> `compare_exchange_weak`, because of `#[warn(deprecated)]`

Signed-off-by: Chojan Shang <[email protected]>

Author: Chojan Shang

Cargo.toml

@@ -16,7 +16,7 @@ exclude = ["/.github/*", "/.travis.yml", "/appveyor.yml"]
 [dependencies]
 log = { version = "0.4", features = ["std"] }
 lazy_static = "1.2"
-rand = "0.7"
+rand = "0.8"
 
 [features]
 failpoints = []

src/lib.rs

@@ -244,6 +244,7 @@ impl Debug for SyncCallback {
 }
 
 impl PartialEq for SyncCallback {
+    #[allow(clippy::vtable_address_comparisons)]
     fn eq(&self, other: &Self) -> bool {
         Arc::ptr_eq(&self.0, &other.0)
     }
@@ -336,14 +337,21 @@ impl Action {
         if self.freq < 1f32 && !rand::thread_rng().gen_bool(f64::from(self.freq)) {
             return None;
         }
-        if let Some(ref cnt) = self.count {
+        if let Some(ref ref_cnt) = self.count {
+            let mut cnt = ref_cnt.load(Ordering::Acquire);
             loop {
-                let c = cnt.load(Ordering::Acquire);
-                if c == 0 {
+                if cnt == 0 {
                     return None;
                 }
-                if c == cnt.compare_and_swap(c, c - 1, Ordering::AcqRel) {
-                    break;
+                let new_cnt = cnt - 1;
+                match ref_cnt.compare_exchange_weak(
+                    cnt,
+                    new_cnt,
+                    Ordering::AcqRel,
+                    Ordering::Acquire,
+                ) {
+                    Ok(_) => break,
+                    Err(c) => cnt = c,
                 }
             }
         }