Escape <> and & symbol in doc attributes.

Author: pellico

Cargo.toml

@@ -56,6 +56,7 @@ serde = { version = "1.0", optional = true }
 serde_json = { version = "1.0.85", optional = true }
 serde_yaml = { version = "0.9.11", optional = true }
 regex = "1.7.0"
+html-escape = "0.2"
 
 [dependencies.svd-parser]
 features = ["expand"]

src/generate/interrupt.rs

@@ -63,7 +63,7 @@ pub fn render(
                 .as_ref()
                 .map(|s| util::respace(s))
                 .as_ref()
-                .map(|s| util::escape_brackets(s))
+                .map(|s| util::escape_special_chars(s))
                 .unwrap_or_else(|| interrupt.0.name.clone())
         );
 

src/generate/peripheral.rs

@@ -184,7 +184,7 @@ pub fn render(p_original: &Peripheral, index: &Index, config: &Config) -> Result
         }
     }
 
-    let description = util::escape_brackets(
+    let description = util::escape_special_chars(
         util::respace(p.description.as_ref().unwrap_or(&name.as_ref().to_owned())).as_ref(),
     );
 
@@ -516,7 +516,7 @@ impl FieldRegions {
 }
 
 fn make_comment(size: u32, offset: u32, description: &str) -> String {
-    let desc = util::escape_brackets(&util::respace(description));
+    let desc = util::escape_special_chars(&util::respace(description));
     if size > 32 {
         let end = offset + size / 8;
         format!("0x{offset:02x}..0x{end:02x} - {desc}")
@@ -1306,7 +1306,7 @@ fn cluster_block(
     config: &Config,
 ) -> Result<TokenStream> {
     let description =
-        util::escape_brackets(&util::respace(c.description.as_ref().unwrap_or(&c.name)));
+        util::escape_special_chars(&util::respace(c.description.as_ref().unwrap_or(&c.name)));
     let mod_name = util::replace_suffix(&c.name, "");
 
     // name_snake_case needs to take into account array type.

src/generate/register.rs

@@ -39,7 +39,7 @@ pub fn render(
     let span = Span::call_site();
     let name_constant_case = name.to_constant_case_ident(span);
     let name_snake_case = name.to_snake_case_ident(span);
-    let description = util::escape_brackets(
+    let description = util::escape_special_chars(
         util::respace(&register.description.clone().unwrap_or_else(|| {
             warn!("Missing description for register {}", register.name);
             Default::default()
@@ -129,7 +129,7 @@ pub fn render_register_mod(
         rsize.next_power_of_two()
     };
     let rty = rsize.to_ty()?;
-    let description = util::escape_brackets(
+    let description = util::escape_special_chars(
         util::respace(&register.description.clone().unwrap_or_else(|| {
             warn!("Missing description for register {}", register.name);
             Default::default()
@@ -436,7 +436,7 @@ pub fn fields(
         let name_snake_case = name.to_snake_case_ident(span);
         let name_constant_case = name.to_sanitized_constant_case();
         let description_raw = f.description.as_deref().unwrap_or(""); // raw description, if absent using empty string
-        let description = util::respace(&util::escape_brackets(description_raw));
+        let description = util::respace(&util::escape_special_chars(description_raw));
 
         let can_read = can_read
             && (f.access != Some(Access::WriteOnly))
@@ -833,7 +833,7 @@ pub fn fields(
                 for v in &variants {
                     let pc = &v.pc;
                     let sc = &v.sc;
-                    let doc = util::escape_brackets(&util::respace(&v.doc));
+                    let doc = util::escape_special_chars(&util::respace(&v.doc));
                     proxy_items.extend(quote! {
                         #[doc = #doc]
                         #inline
@@ -1105,7 +1105,7 @@ fn add_from_variants(
 
     let mut vars = TokenStream::new();
     for v in variants.iter().map(|v| {
-        let desc = util::escape_brackets(&util::respace(&format!("{}: {}", v.value, v.doc)));
+        let desc = util::escape_special_chars(&util::respace(&format!("{}: {}", v.value, v.doc)));
         let pcv = &v.pc;
         let pcval = &util::unsuffixed(v.value);
         quote! {
@@ -1178,7 +1178,7 @@ fn description_with_bits(description: &str, offset: u64, width: u32) -> String {
     };
     if !description.is_empty() {
         res.push_str(" - ");
-        res.push_str(&util::respace(&util::escape_brackets(description)));
+        res.push_str(&util::respace(&util::escape_special_chars(description)));
     }
     res
 }

src/util.rs

@@ -7,6 +7,7 @@ use quote::quote;
 use std::collections::HashSet;
 use std::path::{Path, PathBuf};
 use svd_rs::{MaybeArray, Peripheral, PeripheralInfo};
+use html_escape::encode_text_minimal;
 
 use syn::{
     punctuated::Punctuated, token::Colon2, AngleBracketedGenericArguments, GenericArgument, Lit,
@@ -287,6 +288,12 @@ pub fn escape_brackets(s: &str) -> String {
         })
 }
 
+/// Escape basic html tags and brackets
+pub fn escape_special_chars(s: &str) -> String {
+    let html_escaped = encode_text_minimal(s);
+    escape_brackets(&html_escaped)
+}
+
 pub fn name_of<T: FullName>(maybe_array: &MaybeArray<T>, ignore_group: bool) -> Cow<str> {
     match maybe_array {
         MaybeArray::Single(info) => info.fullname(ignore_group),