use enumerate() instead of indexing

Fixes:

warning: the loop variable `i` is used to index `time_data`
   --> src/rrd_graph_new.rs:388:14
    |
388 |     for i in 0..time_data.len() {
    |              ^^^^^^^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_range_loop
    = note: `#[warn(clippy::needless_range_loop)]` on by default
help: consider using an iterator and enumerate()
    |
388 |     for (i, <item>) in time_data.iter().enumerate() {
    |         ~~~~~~~~~~~    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Signed-off-by: Maximiliano Sandoval <m.sandoval@proxmox.com>

diff --git a/src/rrd_graph_new.rs b/src/rrd_graph_new.rs
index 32b6126aa2faf8a5494ad19ee024f7390e8d7d70..f549c4fc09301f6f89ef356842bdfcb6057c7d1e 100644 (file)
--- a/src/rrd_graph_new.rs
+++ b/src/rrd_graph_new.rs
@@ -385,10 +385,9 @@ fn compute_outline_path(
 ) -> String {
     let mut path = String::new();
     let mut last_undefined = true;
-    for i in 0..time_data.len() {
-        let t = time_data[i];
+    for (i, t) in time_data.iter().enumerate() {
         let value = *values.get(i).unwrap_or(&f64::NAN);
-        let x = compute_x(t);
+        let x = compute_x(*t);
 
         if last_undefined {
             if value.is_nan() {