Automatic handle latency changes in AudioGraph

Author: kunitoki

modules/yup_audio_graph/graph/yup_AudioGraphProcessor.cpp

@@ -410,6 +410,11 @@ class AudioGraphProcessor::Pimpl final : private AudioProcessor::Listener
         if (details.latencyChanged)
         {
             latencyChangeCounter.fetch_add (1);
+
+            if (! commitInProgress.load())
+            {
+                ignoreUnused (commitChanges());
+            }
         }
     }
 

modules/yup_audio_graph/graph/yup_AudioGraphProcessor.h

@@ -29,11 +29,11 @@ namespace yup
     Topology edits are made to a control-thread graph model. commitChanges()
     validates the model, prepares newly compiled nodes for the current playback
     configuration, and publishes an immutable processing plan. Child processor
-    latency notifications mark the graph dirty; call commitChanges() from the
-    control thread to rebuild delay compensation. Metadata edits such as node
-    positions and properties are saved by the model without invalidating the
-    compiled plan. processBlock() only swaps pending plans at block boundaries
-    and keeps retired plans alive until a later control-thread commit or destruction.
+    latency notifications are handled by the graph as host notifications and
+    rebuild delay compensation. Metadata edits such as node positions and
+    properties are saved by the model without invalidating the compiled plan.
+    processBlock() only swaps pending plans at block boundaries and keeps retired
+    plans alive until a later control-thread commit or destruction.
 */
 class YUP_API AudioGraphProcessor final : public AudioProcessor
 {

tests/yup_audio_graph/yup_AudioGraphProcessor.cpp

@@ -1220,11 +1220,6 @@ TEST (AudioGraphProcessorTests, ExternalTopologyEditsDriveDirtyRevision)
     ASSERT_NE (nullptr, processor);
 
     processor->setLatencySamplesForTest (16);
-    EXPECT_TRUE (graph.hasUncommittedChanges());
-    EXPECT_EQ (0, graph.getLatencySamples());
-    EXPECT_EQ (latencyQueriesAfterCommit, latencyQueryCount.load());
-
-    EXPECT_TRUE (graph.commitChanges().wasOk());
     EXPECT_FALSE (graph.hasUncommittedChanges());
     EXPECT_EQ (16, graph.getLatencySamples());
     EXPECT_GT (latencyQueryCount.load(), latencyQueriesAfterCommit);