Merge #63

63: Make the schema field optional r=Emilgardis a=jamesmunns

Some devices that have created fake SVDs, such as for the MSP430, may not have schemas listed.

This fixes the error seen with https://travis-ci.org/rust-embedded/svd2rust/jobs/456451847, See also rust-embedded/svd2rust#256 (comment)

Co-authored-by: James Munns <[email protected]>

Author: bors[bot]

src/svd/device.rs

@@ -18,7 +18,7 @@ use svd::peripheral::Peripheral;
 #[derive(Clone, Debug)]
 pub struct Device {
     pub name: String,
-    schema_version: String,
+    schema_version: Option<String>,
     pub version: Option<String>,
     pub description: Option<String>,
     pub address_unit_bits: Option<u32>,
@@ -40,8 +40,7 @@ impl Parse for Device {
             name: tree.get_child_text("name")?,
             schema_version: tree.attributes
                 .get("schemaVersion")
-                .unwrap()
-                .clone(),
+                .map(|s| s.clone()),
             cpu: parse::optional::<Cpu>("cpu", tree)?,
             version: tree.get_child_text_opt("version")?,
             description: tree.get_child_text_opt("description")?,