Merge pull request #2292 from zhiqiangxu/avoid_unnecessary_is_synced

chore: avoid unnecessary call to `is_synced`
autonomys · Dec 6, 2023 · 25e9cc4 · 25e9cc4
2 parents 486b56f + b9a6ee0
commit 25e9cc4
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/crates/subspace-farmer/src/single_disk_farm/plotting.rs b/crates/subspace-farmer/src/single_disk_farm/plotting.rs
@@ -553,14 +553,14 @@ where
             info!("Node is not synced yet, pausing plotting until sync status changes");
 
             loop {
-                if node_sync_status.is_synced() {
-                    info!("Node is synced, resuming plotting");
-                    continue 'outer;
-                }
-
                 match node_sync_status_change_notifications.next().await {
                     Some(new_node_sync_status) => {
                         node_sync_status = new_node_sync_status;
+
+                        if node_sync_status.is_synced() {
+                            info!("Node is synced, resuming plotting");
+                            continue 'outer;
+                        }
                     }
                     None => {
                         // Subscription ended, nothing left to do