- Removed unnecessary enumerations

- Optimized the deduplication implementation of `Effect`

Author: fxdmhtt

cache/src/effect.rs

@@ -9,10 +9,10 @@ impl Effect {
         let e = Rc::new(Effect { f: Box::new(f) });
 
         crate::current_effect_push(e.clone());
-
         e.run();
+        crate::current_effect_pop();
 
-        crate::current_effect_pop()
+        e
     }
 
     pub(crate) fn run(&self) {

cache/src/lib.rs

@@ -1,13 +1,3 @@
-pub enum Trace {
-    Push,
-    Pop,
-}
-
-pub enum MemoOperator {
-    Memo(Trace),
-    Pop,
-}
-
 mod cache;
 mod call_stack;
 

cache/src/signal.rs

@@ -61,14 +61,12 @@ where
         {
             self.dependents.borrow_mut().push(*last);
         }
-        if let Some(e) = call_stack::current_effect_peak()
-            && !self
-                .effects
-                .borrow()
-                .iter()
-                .any(|w| w.as_ptr() == Rc::as_ptr(&e))
-        {
-            self.effects.borrow_mut().push(Rc::downgrade(&e));
+
+        if let Some(e) = call_stack::current_effect_peak() {
+            let e = Rc::downgrade(&e);
+            if !self.effects.borrow().iter().any(|w| Weak::ptr_eq(w, &e)) {
+                self.effects.borrow_mut().push(e);
+            }
         }
 
         self.value.borrow()