Fixed rsx! for loops and if statements

examples/benchmark.rs

@@ -1,37 +0,0 @@
-use std::sync::Arc;
-
-use osui::prelude::*;
-
-pub fn main() {
-    let engine = Benchmark::new(Console::new());
-
-    let mut benchmark_result = Vec::new();
-
-    for flat in 0..10 {
-        // for nest in 0..10 {
-        benchmark_result.push(
-            engine
-                .run(move |cx| {
-                    // let sync_nest = use_state(nest);
-                    cx.scope().child(app, None);
-                    // use_sync_effect(cx, &sync_nest, |s| s.get_dl(), &[&sync_nest]);
-
-                    rsx! {
-                        app
-                    }
-                    .view(cx.clone())
-                })
-                .expect("Failed to run engine"),
-        );
-        // }
-    }
-}
-
-fn app(cx: &Arc<Context>) -> View {
-    // let nest = use_sync_state(cx, 0, |v| *v);
-
-    rsx! {
-        "hello_world"
-    }
-    .view(cx.clone())
-}

macros/src/emit.rs

@@ -3,7 +3,7 @@ use proc_macro2::TokenStream;
 use quote::quote;
 
 pub fn emit_rsx(root: RsxRoot) -> TokenStream {
-    let nodes = root.nodes.iter().map(emit_node);
+    let nodes = root.nodes.iter().map(emit_node_scope);
 
     quote! {{
         let mut r = osui::frontend::Rsx::new();
@@ -25,19 +25,42 @@ fn emit_deps(deps: &[Dep]) -> TokenStream {
         .collect()
 }
 
-fn emit_node(node: &RsxNode) -> TokenStream {
+fn emit_deps_vec(deps: &[Dep]) -> TokenStream {
-    match node {
+    let deps = deps.iter().map(|d| {
-        RsxNode::Text(text) => quote! {
+        let ident = &d.ident;
-            r.static_scope(|scope| scope.view(Arc::new(move |ctx| {
-                ctx.draw_text(Point { x: 0, y: 0 }, &format!(#text))
-            })));
-        },
 
-        RsxNode::Component(name) => quote! {
+        if let Some(pat) = &d.pat {
-            r.static_scope(|scope| scope.view(Arc::new(move |ctx| {
+            quote! {{
-                scope.child(#name, None)
+                let #pat = #ident.clone();
-            })));
+                Box::new(#pat) as Box<dyn HookDependency>
-        },
+            }}
+        } else {
+            quote! {
+                Box::new(#ident.clone()) as Box<dyn HookDependency>
+            }
+        }
+    });
+
+    quote! {
+        vec![ #(#deps),* ]
+    }
+}
+
+fn emit_node_scope(node: &RsxNode) -> TokenStream {
+    match node {
+        RsxNode::Text(_) => {
+            let emit = emit_node(node);
+            quote! {
+                r.static_scope(|scope| {#emit});
+            }
+        }
+
+        RsxNode::Component(_) => {
+            let emit = emit_node(node);
+            quote! {
+                r.static_scope(|scope| {#emit});
+            }
+        }
 
         RsxNode::Mount(m) => quote! {
             #m.mount();
@@ -49,6 +72,7 @@ fn emit_node(node: &RsxNode) -> TokenStream {
             children,
         } => {
             let deps_emit = emit_deps(deps);
+            let deps_vec_emit = emit_deps_vec(deps);
             let kids = children.iter().map(emit_node);
 
             quote! {
@@ -62,7 +86,7 @@ fn emit_node(node: &RsxNode) -> TokenStream {
                         } else {
                             scope.children.lock().unwrap().clear();
                         }
-                    });
+                    }, #deps_vec_emit);
                 }
             }
         }
@@ -74,6 +98,7 @@ fn emit_node(node: &RsxNode) -> TokenStream {
             children,
         } => {
             let deps_emit = emit_deps(deps);
+            let deps_vec_emit = emit_deps_vec(deps);
             let kids = children.iter().map(emit_node);
 
             quote! {
@@ -84,9 +109,31 @@ fn emit_node(node: &RsxNode) -> TokenStream {
                         for #pat in #expr {
                             #(#kids)*
                         }
-                    });
+                    }, #deps_vec_emit);
                 }
             }
         }
     }
 }
+
+fn emit_node(node: &RsxNode) -> TokenStream {
+    match node {
+        RsxNode::Text(text) => quote! {
+            scope.view(Arc::new(move |ctx| {
+                ctx.draw_text(Point { x: 0, y: 0 }, &format!(#text))
+            }));
+        },
+
+        RsxNode::Component(name) => quote! {
+            scope.child(#name, None);
+        },
+
+        RsxNode::Mount(m) => quote! {
+            #m.mount();
+        },
+
+        RsxNode::If { .. } => panic!("Invalid if statement"),
+
+        RsxNode::For { .. } => panic!("Invalid for loop"),
+    }
+}