From 90bc678377412c3b3d39e44693a4cb206a6ac4cd Mon Sep 17 00:00:00 2001
From: itsmattkc <itsmattkc@gmail.com>
Date: Fri, 10 Jul 2020 18:26:30 +1000
Subject: [PATCH] nodeinputarray: delete sub-params immediately instead of
 queuing

Queuing the destructor caused a potential desync in the node graph that
disconnected blocks after they were reconnected elsewhere. Destroying
immediately keeps this logic synchronized.
---
 app/node/inputarray.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/node/inputarray.cpp b/app/node/inputarray.cpp
index 8ca54f435..b1ec0caa8 100644
--- a/app/node/inputarray.cpp
+++ b/app/node/inputarray.cpp
@@ -59,7 +59,7 @@ void NodeInputArray::SetSize(int size)
   if (size < old_size) {
     // If the new size is less, delete all extraneous parameters
     for (int i=size;i<old_size;i++) {
-      sub_params_.at(i)->deleteLater();
+      delete sub_params_.at(i);
     }
   }