Merge #134

134: Implement strict flag r=Emilgardis a=brainstorm

Fixes #127 among others

Co-authored-by: Roman Valls Guimera <brainstorm@nopcode.org>

Author: bors[bot]

Cargo.toml

@@ -15,6 +15,7 @@ readme = "README.md"
 
 [features]
 derive-from = []
+strict = []
 
 [dependencies]
 xmltree = "0.8"

src/error.rs

@@ -91,8 +91,10 @@ pub(crate) fn check_name(name: &str, tag: &str) -> Result<()> {
     static PATTERN: Lazy<Regex> = Lazy::new(|| Regex::new("^[_A-Za-z0-9]*$").unwrap());
     if PATTERN.is_match(name) {
         Ok(())
-    } else {
+    } else if cfg!(feature = "strict") {
         Err(NameError::Invalid(name.to_string(), tag.to_string()).into())
+    } else {
+        Ok(())
     }
 }
 

src/svd/clusterinfo.rs

@@ -111,6 +111,7 @@ impl ClusterInfo {
         if let Some(name) = self.derived_from.as_ref() {
             check_derived_name(name, "derivedFrom")?;
         } else if self.children.is_empty() {
+            #[cfg(feature = "strict")]
             return Err(SVDError::EmptyCluster)?;
         }
         Ok(self)

src/svd/peripheral.rs

@@ -162,6 +162,7 @@ impl Peripheral {
             check_dimable_name(name, "derivedFrom")?;
         } else if let Some(registers) = self.registers.as_ref() {
             if registers.is_empty() {
+                #[cfg(feature = "strict")]
                 return Err(SVDError::EmptyRegisters)?;
             }
         }

src/svd/registerinfo.rs

@@ -191,6 +191,7 @@ impl RegisterInfo {
             check_derived_name(name, "derivedFrom")?;
         } else if let Some(fields) = self.fields.as_ref() {
             if fields.is_empty() {
+                #[cfg(feature = "strict")]
                 return Err(SVDError::EmptyFields)?;
             }
         }