Make clear that previous JSON export/import was for metadata.

CQCL · Jan 6, 2025 · 2294908 · 2294908
1 parent 7fd9c47
commit 2294908
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/hugr-core/src/export.rs b/hugr-core/src/export.rs
@@ -563,7 +563,7 @@ impl<'a> Context<'a> {
 
             for (name, value) in opdef.iter_misc() {
                 let name = self.bump.alloc_str(name);
-                let value = self.export_json(value);
+                let value = self.export_json_meta(value);
                 meta.push(model::MetaItem { name, value });
             }
 
@@ -1054,14 +1054,14 @@ impl<'a> Context<'a> {
 
         for (name, value) in metadata_map {
             let name = self.bump.alloc_str(name);
-            let value = self.export_json(value);
+            let value = self.export_json_meta(value);
             meta.push(model::MetaItem { name, value });
         }
 
         meta.into_bump_slice()
     }
 
-    pub fn export_json(&mut self, value: &serde_json::Value) -> model::TermId {
+    pub fn export_json_meta(&mut self, value: &serde_json::Value) -> model::TermId {
         let value = serde_json::to_string(value).expect("json values are always serializable");
         let value = self.make_term(model::Term::Str(self.bump.alloc_str(&value)));
         let value = self.bump.alloc_slice_copy(&[value]);

diff --git a/hugr-core/src/import.rs b/hugr-core/src/import.rs
@@ -174,7 +174,7 @@ impl<'a> Context<'a> {
         for meta_item in node_data.meta {
             // TODO: For now we expect all metadata to be JSON since this is how
             // it is handled in `hugr-core`.
-            let value = self.import_json_value(meta_item.value)?;
+            let value = self.import_json_meta(meta_item.value)?;
             self.hugr.set_metadata(node, meta_item.name, value);
         }
 
@@ -1262,7 +1262,7 @@ impl<'a> Context<'a> {
         }
     }
 
-    fn import_json_value(
+    fn import_json_meta(
         &mut self,
         term_id: model::TermId,
     ) -> Result<serde_json::Value, ImportError> {