diff --git a/generic/fmp4/src/fmp4mux/boxes.rs b/generic/fmp4/src/fmp4mux/boxes.rs
index 40b0567b..6efbf729 100644
--- a/generic/fmp4/src/fmp4mux/boxes.rs
+++ b/generic/fmp4/src/fmp4mux/boxes.rs
@@ -1727,8 +1727,7 @@ fn write_traf(
 
     let mut current_data_offset = 0;
 
-    let mut iter = GroupBy::new(cfg.buffers, |a: &Buffer, b: &Buffer| a.idx == b.idx);
-    while let Some(run) = iter.next() {
+    for run in GroupBy::new(cfg.buffers, |a: &Buffer, b: &Buffer| a.idx == b.idx) {
         if run[0].idx != idx {
             // FIXME: What to do with >4GB offsets?
             current_data_offset = (current_data_offset as u64
diff --git a/generic/fmp4/src/fmp4mux/imp.rs b/generic/fmp4/src/fmp4mux/imp.rs
index b4a41dfd..2dfa6eef 100644
--- a/generic/fmp4/src/fmp4mux/imp.rs
+++ b/generic/fmp4/src/fmp4mux/imp.rs
@@ -643,7 +643,7 @@ impl FMP4Mux {
                         };
 
                         let end_timestamp = match gop_buffers.peek() {
-                            Some(ref buffer) => {
+                            Some(buffer) => {
                                 if stream.intra_only {
                                     buffer.pts
                                 } else {
@@ -702,7 +702,7 @@ impl FMP4Mux {
         let mut caps = None;
         if state.stream_header.is_none() {
             let (_, new_caps) = self
-                .update_header(element, state, &settings, false)?
+                .update_header(element, state, settings, false)?
                 .unwrap();
             caps = Some(new_caps);
         }