🐛 Remove children correctly

frontend/src/app/render_wasm/api.cljs

@@ -198,9 +198,8 @@
           (when-not (empty? entries)
             (let [id (first entries)]
               (sr/heapu32-set-uuid id heap (mem/ptr8->ptr32 current-offset))
-              (recur (rest entries) (+ current-offset CHILD-ENTRY-SIZE)))))
+              (recur (rest entries) (+ current-offset CHILD-ENTRY-SIZE)))))))
-
+    (h/call wasm/internal-module "_set_children")))
-        (h/call wasm/internal-module "_set_children")))))
 
 (defn- get-string-length [string] (+ (count string) 1))
 

render-wasm/src/main.rs

@@ -219,7 +219,8 @@ pub extern "C" fn add_shape_child(a: u32, b: u32, c: u32, d: u32) {
 
 #[no_mangle]
 pub extern "C" fn set_children() {
-    let bytes = mem::bytes();
+    let bytes = mem::bytes_or_empty();
+
     let entries: IndexSet<Uuid> = bytes
         .chunks(size_of::<<Uuid as SerializableResult>::BytesType>())
         .map(|data| Uuid::from_bytes(data.try_into().unwrap()))
@@ -237,13 +238,10 @@ pub extern "C" fn set_children() {
             state.delete_shape(id);
         }
     });
-}
 
-#[no_mangle]
+    if !bytes.is_empty() {
-pub extern "C" fn clear_shape_children() {
+        mem::free_bytes();
-    with_current_shape!(state, |shape: &mut Shape| {
+    }
-        shape.clear_children();
-    });
 }
 
 #[no_mangle]

render-wasm/src/mem.rs

@@ -56,6 +56,12 @@ pub fn bytes() -> Vec<u8> {
         .map_or_else(|| panic!("Buffer is not initialized"), |buffer| *buffer)
 }
 
+pub fn bytes_or_empty() -> Vec<u8> {
+    let mut guard = BUFFERU8.lock().unwrap();
+
+    guard.take().map_or_else(|| Vec::new(), |buffer| *buffer)
+}
+
 pub trait SerializableResult {
     type BytesType;
     fn from_bytes(bytes: Self::BytesType) -> Self;

render-wasm/src/shapes.rs

@@ -444,10 +444,6 @@ impl Shape {
         (added, removed)
     }
 
-    pub fn clear_children(&mut self) {
-        self.children.clear();
-    }
-
     pub fn fills(&self) -> std::slice::Iter<Fill> {
         self.fills.iter()
     }