Address clippy

[rbdr/gema_texto] / src / html_renderer.rs

diff --git a/src/html_renderer.rs b/src/html_renderer.rs
index 503337bf069cdfdab2516c98dd37c61806d9dcb3..61d2d733eb3240b1bd80a4047bddc962dcbdd1c0 100644 (file)
--- a/src/html_renderer.rs
+++ b/src/html_renderer.rs
@@ -1,4 +1,5 @@
 use crate::gemini_parser::GeminiLine;
 use crate::gemini_parser::GeminiLine;

+use std::path::Path;

 
 /// Renders HTML from a vector of `GeminiLine` elements.
 ///
 
 /// Renders HTML from a vector of `GeminiLine` elements.
 ///


@@ -99,13 +100,22 @@ fn line_content(line: &GeminiLine) -> String {
     match line {
         GeminiLine::Text(content, false) => format!("<p>{content}</p>"),
         GeminiLine::Link(url, text) => {
     match line {
         GeminiLine::Text(content, false) => format!("<p>{content}</p>"),
         GeminiLine::Link(url, text) => {

-            let processed_url;
-            if !url.starts_with("gemini:") && url.ends_with(".gmi") {
-                processed_url = url.replace(".gmi", ".html");
+            let path = Path::new(url);
+            let processed_url = if !url.starts_with("gemini:")
+                && path
+                    .extension()
+                    .map_or(false, |ext| ext.eq_ignore_ascii_case("gmi"))
+            {
+                url.replace(".gmi", ".html")

             } else {
             } else {

-                processed_url = url.to_string();
-            }
-            let display = if text.is_empty() { &processed_url } else { text };
+                url.to_string()
+            };
+
+            let display = if text.is_empty() {
+                &processed_url
+            } else {
+                text
+            };

             format!("<p class=\"a\"><a href=\"{processed_url}\">{display}</a></p>")
         }
         GeminiLine::Heading(level, content) => format!("<h{level}>{content}</h{level}>"),
             format!("<p class=\"a\"><a href=\"{processed_url}\">{display}</a></p>")
         }
         GeminiLine::Heading(level, content) => format!("<h{level}>{content}</h{level}>"),