feat: qemu 8.2.0

Author: arcnmx

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "qapi-codegen"
-version = "0.11.1" # keep in sync with html_root_url
+version = "0.11.2" # keep in sync with html_root_url
 authors = ["arcnmx"]
 edition = "2018"
 
@@ -17,4 +17,4 @@ travis-ci = { repository = "arcnmx/qapi-rs" }
 maintenance = { status = "passively-maintained" }
 
 [dependencies]
-qapi-parser = { version = "0.10", path = "../parser" }
+qapi-parser = { version = "0.11", path = "../parser" }

codegen/Cargo.toml

@@ -1,4 +1,4 @@
-#![doc(html_root_url = "https://docs.rs/qapi-codegen/0.11.1")]
+#![doc(html_root_url = "https://docs.rs/qapi-codegen/0.11.2")]
 
 //! Generates Rust types for the [QAPI schema language](https://qemu-project.gitlab.io/qemu/devel/qapi-code-gen.html#the-qapi-schema-language)
 
@@ -127,7 +127,7 @@ struct Context<W> {
     includes: Vec<String>,
     included: HashSet<PathBuf>,
     events: Vec<spec::Event>,
-    unions: Vec<spec::CombinedUnion>,
+    unions: BTreeMap<String, spec::CombinedUnion>,
     enums: BTreeMap<String, spec::Enum>,
     types: BTreeMap<String, spec::Struct>,
     struct_discriminators: BTreeMap<String, String>,
@@ -274,10 +274,24 @@ unsafe impl ::qapi_spec::Enum for {} {{
             Spec::Event(v) => {
                 write!(self.out, "
 #[derive(Debug, Clone, Serialize, Deserialize{})]
-pub struct {} {{
-", if v.data.is_empty() { ", Default" } else { "" }, event_identifier(&v.id))?;
-                for item in &v.data.fields {
-                    writeln!(self.out, "{},", valuety(item, true, &v.id))?;
+", if v.data.is_empty() { ", Default" } else { "" })?;
+                if let spec::DataOrType::Type(..) = v.data {
+                    writeln!(self.out, "#[serde(transparent)]")?;
+                    writeln!(self.out, "#[repr(transparent)]")?;
+                }
+                writeln!(self.out, "pub struct {} {{", event_identifier(&v.id))?;
+                match v.data {
+                    spec::DataOrType::Type(ref ty) => {
+                        let data = spec::Value {
+                            name: "data".into(),
+                            ty: ty.clone(),
+                            optional: false,
+                        };
+                        writeln!(self.out, "{},", valuety(&data, true, &v.id))?
+                    },
+                    spec::DataOrType::Data(ref data) => for item in &data.fields {
+                        writeln!(self.out, "{},", valuety(item, true, &v.id))?;
+                    },
                 }
                 writeln!(self.out, "}}")?;
                 writeln!(self.out, "
@@ -298,7 +312,7 @@ pub enum {} {{
                 writeln!(self.out, "}}")?;
             },
             Spec::CombinedUnion(v) => {
-                self.unions.push(v);
+                self.unions.insert(v.id.clone(), v);
             },
             Spec::PragmaWhitelist { .. } => (),
             Spec::PragmaExceptions { .. } => (),
@@ -400,7 +414,7 @@ impl {} {{
     }
 
     fn process_unions(&mut self) -> io::Result<()> {
-        for u in &self.unions {
+        for u in self.unions.values() {
             let discrim = u.discriminator.as_ref().map(|s| &s[..]).unwrap_or("type");
             let type_id = type_identifier(&u.id);
             write!(self.out, "
@@ -540,8 +554,12 @@ impl From<{}> for {} {{
 }}
 ", variant_ty, type_id, variant_ty, variant_name)?;
                         let ty = self.types.get(&variant.ty.name)
+                            .map(|ty| ty.wrapper_type())
+                            .or_else(|| self.unions.get(&variant.ty.name)
+                                .map(|_e| None)
+                            )
                             .ok_or_else(|| io::Error::new(io::ErrorKind::NotFound, format!("could not find qapi type {}, needed by {}", variant.ty.name, u.id)))?;
-                        if let Some(newtype) = ty.wrapper_type() {
+                        if let Some(newtype) = ty {
                             let newtype_ty = typename(&newtype.ty);
                             write!(self.out, "
 impl From<{}> for {} {{

codegen/src/lib.rs

@@ -1,6 +1,6 @@
 [package]
 name = "qapi-parser"
-version = "0.10.0" # keep in sync with html_root_url
+version = "0.11.0" # keep in sync with html_root_url
 authors = ["arcnmx"]
 edition = "2018"
 

parser/Cargo.toml

@@ -1,4 +1,4 @@
-#![doc(html_root_url = "https://docs.rs/qapi-parser/0.10.0")]
+#![doc(html_root_url = "https://docs.rs/qapi-parser/0.11.0")]
 
 pub mod spec {
     use std::fmt;
@@ -235,6 +235,9 @@ pub mod spec {
     #[serde(untagged, rename_all = "kebab-case")]
     pub enum Conditional {
         Define(ConditionalDefinition),
+        Not {
+            not: ConditionalDefinition,
+        },
         All {
             all: Vec<ConditionalDefinition>,
         },
@@ -263,12 +266,21 @@ pub mod spec {
         #[serde(default)]
         pub allow_oob: bool,
         #[serde(default)]
+        pub coroutine: bool,
+        #[serde(default)]
+        pub boxed: bool,
+        #[serde(default = "Command::success_response_default")]
+        pub success_response: bool,
+        #[serde(default)]
+        pub allow_preconfig: bool,
+        #[serde(default)]
         pub features: Features,
         #[serde(default = "Command::gen_default")]
         pub gen: bool,
     }
 
     impl Command {
+        fn success_response_default() -> bool { true }
         fn gen_default() -> bool { true }
     }
 
@@ -326,6 +338,8 @@ pub mod spec {
         pub id: String,
         #[serde(default)]
         pub data: Vec<SpecName>,
+        #[serde(default)]
+        pub prefix: Option<String>,
         #[serde(default, rename = "if")]
         pub conditional: Option<Conditional>,
     }
@@ -390,9 +404,11 @@ pub mod spec {
         #[serde(rename = "event")]
         pub id: String,
         #[serde(default)]
-        pub data: Data,
+        pub data: DataOrType,
         #[serde(default, rename = "if")]
         pub conditional: Option<Conditional>,
+        #[serde(default)]
+        pub features: Features,
     }
 
     #[derive(Debug, Clone, Deserialize)]
@@ -425,23 +441,41 @@ pub mod spec {
             name: String,
             #[serde(rename = "if")]
             conditional: Conditional,
+            #[serde(default)]
+            features: Features,
         },
         Explicit {
             name: String,
+            #[serde(default)]
+            features: Features,
         },
     }
 
+    impl SpecName {
+        pub fn name(&self) -> &String {
+            match self {
+                SpecName::Name(name) | SpecName::Explicit { name, .. } => name,
+                SpecName::Conditional { name, .. } => name,
+            }
+        }
+
+        pub fn features(&self) -> Option<&Features> {
+            match self {
+                SpecName::Name(..) => None,
+                SpecName::Explicit { features, .. } => Some(features),
+                SpecName::Conditional { features, .. } => Some(features),
+            }
+        }
+    }
+
     impl fmt::Display for SpecName {
         fn fmt(&self, fmt: &mut fmt::Formatter) -> fmt::Result {
             fmt::Display::fmt(self.as_ref(), fmt)
         }
     }
     impl AsRef<str> for SpecName {
         fn as_ref(&self) -> &str {
-            match self {
-                SpecName::Name(name) | SpecName::Explicit { name } => &name[..],
-                SpecName::Conditional { name, .. } => &name[..],
-            }
+            &self.name()[..]
         }
     }
 }

parser/src/lib.rs

@@ -0,0 +1 @@
+../../../schema/qapi/machine-common.json