Do not perform superfluous bounds checks on drop

crossbeam-rs · Oct 27, 2019 · ebb6b5f · ebb6b5f
1 parent d2ab7a5
commit ebb6b5f
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/crossbeam-epoch/src/internal.rs b/crossbeam-epoch/src/internal.rs
@@ -133,10 +133,10 @@ impl Default for Bag {
 impl Drop for Bag {
     fn drop(&mut self) {
         // Call all deferred functions.
-        for i in 0..self.len {
+        for deferred in &mut self.deferreds[..self.len] {
             let no_op = Deferred::new(no_op_func);
-            let deferred = mem::replace(&mut self.deferreds[i], no_op);
-            deferred.call();
+            let owned_deferred = mem::replace(deferred, no_op);
+            owned_deferred.call();
         }
     }
 }