From f98ba70764fa5cd5234be13e833f59ccb5c20ea6 Mon Sep 17 00:00:00 2001
From: Robbert van der Helm <mail@robbertvanderhelm.nl>
Date: Fri, 11 Mar 2022 13:27:23 +0100
Subject: [PATCH] Comment on the CLAP gesture handling

---
 src/wrapper/clap/wrapper.rs | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/wrapper/clap/wrapper.rs b/src/wrapper/clap/wrapper.rs
index 4a1e7c7b..c6c81368 100644
--- a/src/wrapper/clap/wrapper.rs
+++ b/src/wrapper/clap/wrapper.rs
@@ -663,6 +663,9 @@ impl<P: ClapPlugin> Wrapper<P> {
                         OutputParamChangeType::Normal => {
                             CLAP_EVENT_IS_LIVE | CLAP_EVENT_SHOULD_RECORD
                         }
+                        // XXX: Apparently you should have `CLAP_EVENT_SHOULD_RECORD` here, even if
+                        //      we're repeating old values. This parameter gesture handling in CLAP
+                        //      is currently a bit weird and error prone.
                         OutputParamChangeType::BeginGesture => {
                             CLAP_EVENT_IS_LIVE | CLAP_EVENT_SHOULD_RECORD | CLAP_EVENT_BEGIN_ADJUST
                         }