tmp-revert

Author: johnsoncodehk

packages/reactivity/src/system.ts

@@ -74,52 +74,43 @@ export function link(dep: Dependency, sub: Subscriber): Link | undefined {
   return linkNewDep(dep, sub, nextDep, currentDep)
 }
 
-export function propagate(current: Link): void {
-  let next = current.nextSub
-  let branchs: OneWayLink<Link | undefined> | undefined
-  let branchDepth = 0
+export function propagate(link: Link): void {
   let targetFlag = SubscriberFlags.Dirty
+  let subs = link
+  let stack = 0
 
   top: do {
-    const sub = current.sub
+    const sub = link.sub
     const subFlags = sub.flags
 
-    let shouldNotify = false
-
     if (
-      !(
+      (!(
         subFlags &
         (SubscriberFlags.Tracking |
           SubscriberFlags.Recursed |
           SubscriberFlags.Propagated)
-      )
-    ) {
-      sub.flags = subFlags | targetFlag
-      shouldNotify = true
-    } else if (
-      subFlags & SubscriberFlags.Recursed &&
-      !(subFlags & SubscriberFlags.Tracking)
+      ) &&
+        ((sub.flags = subFlags | targetFlag), true)) ||
+      (subFlags & SubscriberFlags.Recursed &&
+        !(subFlags & SubscriberFlags.Tracking) &&
+        ((sub.flags = (subFlags & ~SubscriberFlags.Recursed) | targetFlag),
+        true)) ||
+      (!(subFlags & SubscriberFlags.Propagated) &&
+        isValidLink(link, sub) &&
+        ((sub.flags = subFlags | SubscriberFlags.Recursed | targetFlag),
+        (sub as Dependency).subs !== undefined))
     ) {
-      sub.flags = (subFlags & ~SubscriberFlags.Recursed) | targetFlag
-      shouldNotify = true
-    } else if (
-      !(subFlags & SubscriberFlags.Propagated) &&
-      isValidLink(current, sub)
-    ) {
-      sub.flags = subFlags | SubscriberFlags.Recursed | targetFlag
-      shouldNotify = (sub as Dependency).subs !== undefined
-    }
-
-    if (shouldNotify) {
       const subSubs = (sub as Dependency).subs
       if (subSubs !== undefined) {
-        current = subSubs
         if (subSubs.nextSub !== undefined) {
-          branchs = { target: next, linked: branchs }
-          ++branchDepth
-          next = current.nextSub
+          subSubs.prevSub = subs
+          link = subs = subSubs
+          targetFlag = SubscriberFlags.PendingComputed
+          ++stack
+        } else {
+          link = subSubs
+          targetFlag = SubscriberFlags.PendingComputed
         }
-        targetFlag = SubscriberFlags.PendingComputed
         continue
       }
       if (subFlags & SubscriberFlags.Effect) {
@@ -130,25 +121,28 @@ export function propagate(current: Link): void {
     } else if (
       !(subFlags & targetFlag) &&
       subFlags & SubscriberFlags.Propagated &&
-      isValidLink(current, sub)
+      isValidLink(link, sub)
     ) {
       sub.flags = subFlags | targetFlag
     }
 
-    if ((current = next!) !== undefined) {
-      next = current.nextSub
-      targetFlag = branchDepth
+    if ((link = subs.nextSub!) !== undefined) {
+      subs = link
+      targetFlag = stack
         ? SubscriberFlags.PendingComputed
         : SubscriberFlags.Dirty
       continue
     }
 
-    while (branchDepth--) {
-      current = branchs!.target!
-      branchs = branchs!.linked
-      if (current !== undefined) {
-        next = current.nextSub
-        targetFlag = branchDepth
+    while (stack) {
+      --stack
+      const dep = subs.dep
+      const depSubs = dep.subs!
+      subs = depSubs.prevSub!
+      depSubs.prevSub = undefined
+      if ((link = subs.nextSub!) !== undefined) {
+        subs = link
+        targetFlag = stack
           ? SubscriberFlags.PendingComputed
           : SubscriberFlags.Dirty
         continue top