fix clippy issues

Author: Paulo Remoli

src/ui/tab_state/mod.rs

@@ -1547,7 +1547,10 @@ impl TabState {
         // Extend the continuation map for the newly-appended lines.
         if let (Some(cmap), Some(parser)) = (
             self.continuation_map.as_mut(),
-            self.display.format.as_deref().filter(|_| !self.display.raw_mode),
+            self.display
+                .format
+                .as_deref()
+                .filter(|_| !self.display.raw_mode),
         ) {
             let map = Arc::make_mut(cmap);
             let mut last_parent = map.last().copied().unwrap_or(0);

src/ui/widgets/log_panel.rs

@@ -625,12 +625,12 @@ pub fn prepare_log_panel(
                             }
                             // Parent not cached (outside viewport) — parse just
                             // the level from its raw bytes without full layout.
-                            if let Some(parser) = tab.display.format.as_deref() {
-                                if let Some(parts) = parser.parse_line(tab.file_reader.get_line(parent)) {
-                                    if let Some(lvl) = parts.level {
-                                        return LogLevel::parse_level(lvl);
-                                    }
-                                }
+                            if let Some(parser) = tab.display.format.as_deref()
+                                && let Some(parts) =
+                                    parser.parse_line(tab.file_reader.get_line(parent))
+                                && let Some(lvl) = parts.level
+                            {
+                                return LogLevel::parse_level(lvl);
                             }
                         }
                     }

src/ui/widgets/source_select_popup.rs

@@ -33,7 +33,7 @@ impl<'a> Widget for DockerSelectPopup<'a> {
                     if name_w == 0 || c.name.is_empty() {
                         1u16
                     } else {
-                        ((c.name.chars().count() + name_w - 1) / name_w) as u16
+                        c.name.chars().count().div_ceil(name_w) as u16
                     }
                 })
                 .sum::<u16>()
@@ -100,7 +100,7 @@ impl<'a> Widget for DockerSelectPopup<'a> {
                     if name_w == 0 || c.name.is_empty() {
                         1
                     } else {
-                        (c.name.chars().count() + name_w - 1) / name_w
+                        c.name.chars().count().div_ceil(name_w)
                     }
                 })
                 .collect();
@@ -161,11 +161,7 @@ impl<'a> Widget for DockerSelectPopup<'a> {
                 let image: String = c.image.chars().take(image_w).collect();
                 let status: String = c.status.chars().take(status_w).collect();
 
-                let first_visible_chunk = if container_start < scroll {
-                    scroll - container_start
-                } else {
-                    0
-                };
+                let first_visible_chunk = scroll.saturating_sub(container_start);
 
                 for (chunk_idx, name_chunk) in
                     name_chunks.iter().enumerate().skip(first_visible_chunk)
@@ -204,9 +200,8 @@ impl<'a> Widget for DockerSelectPopup<'a> {
                 .render(vsplit[0], buf);
 
             if total_visual_rows > content_h {
-                let mut sb_state =
-                    ScrollbarState::new(total_visual_rows.saturating_sub(content_h))
-                        .position(scroll);
+                let mut sb_state = ScrollbarState::new(total_visual_rows.saturating_sub(content_h))
+                    .position(scroll);
                 StatefulWidget::render(
                     Scrollbar::new(ScrollbarOrientation::VerticalRight)
                         .style(Style::default().fg(self.theme.border)),

tests/integration.rs

@@ -641,7 +641,9 @@ fn test_build_continuation_map_basic() {
     use logana::ui::build_continuation_map;
 
     let reader = make_multiline_log();
-    let sample: Vec<&[u8]> = (0..reader.line_count()).map(|i| reader.get_line(i)).collect();
+    let sample: Vec<&[u8]> = (0..reader.line_count())
+        .map(|i| reader.get_line(i))
+        .collect();
     let parser = detect_format(&sample).expect("format should be detected");
 
     let cmap = build_continuation_map(&reader, parser.as_ref());
@@ -699,7 +701,9 @@ async fn test_exclude_filter_hides_continuation_lines() {
 
     let (_db, mut manager) = setup().await;
     let reader = make_multiline_log();
-    let sample: Vec<&[u8]> = (0..reader.line_count()).map(|i| reader.get_line(i)).collect();
+    let sample: Vec<&[u8]> = (0..reader.line_count())
+        .map(|i| reader.get_line(i))
+        .collect();
     let parser = detect_format(&sample).expect("format detected");
     let cmap = build_continuation_map(&reader, parser.as_ref());
 
@@ -714,8 +718,14 @@ async fn test_exclude_filter_hides_continuation_lines() {
     // Line 0 (ERROR) excluded; lines 1 & 2 are its continuations → also excluded.
     // Line 3 (INFO) should be visible.
     assert!(!visible.contains(0), "ERROR entry should be hidden");
-    assert!(!visible.contains(1), "continuation 1 should be hidden with parent");
-    assert!(!visible.contains(2), "continuation 2 should be hidden with parent");
+    assert!(
+        !visible.contains(1),
+        "continuation 1 should be hidden with parent"
+    );
+    assert!(
+        !visible.contains(2),
+        "continuation 2 should be hidden with parent"
+    );
     assert!(visible.contains(3), "INFO entry should be visible");
 }
 
@@ -727,7 +737,9 @@ async fn test_include_filter_shows_continuations_with_parent() {
 
     let (_db, mut manager) = setup().await;
     let reader = make_multiline_log();
-    let sample: Vec<&[u8]> = (0..reader.line_count()).map(|i| reader.get_line(i)).collect();
+    let sample: Vec<&[u8]> = (0..reader.line_count())
+        .map(|i| reader.get_line(i))
+        .collect();
     let parser = detect_format(&sample).expect("format detected");
     let cmap = build_continuation_map(&reader, parser.as_ref());
 
@@ -742,7 +754,16 @@ async fn test_include_filter_shows_continuations_with_parent() {
     // Line 0 matches; its continuations (1, 2) should be shown.
     // Line 3 (INFO) does not match include → hidden.
     assert!(visible.contains(0), "ERROR entry should be visible");
-    assert!(visible.contains(1), "continuation 1 should follow its visible parent");
-    assert!(visible.contains(2), "continuation 2 should follow its visible parent");
-    assert!(!visible.contains(3), "INFO entry should be hidden (no match)");
+    assert!(
+        visible.contains(1),
+        "continuation 1 should follow its visible parent"
+    );
+    assert!(
+        visible.contains(2),
+        "continuation 2 should follow its visible parent"
+    );
+    assert!(
+        !visible.contains(3),
+        "INFO entry should be hidden (no match)"
+    );
 }