From fcbf198fee5425340e9416e26d3faeb8a0aa9958 Mon Sep 17 00:00:00 2001
From: Robbert van der Helm <mail@robbertvanderhelm.nl>
Date: Fri, 18 Dec 2020 21:35:07 +0100
Subject: [PATCH] Implement IEditController::plainParamToNormalized

---
 src/common/logging/vst3.cpp                   | 11 ++++++++++
 src/common/logging/vst3.h                     |  2 ++
 src/common/serialization/vst3.h               |  1 +
 .../vst3/plugin/edit-controller.h             | 22 +++++++++++++++++++
 .../bridges/vst3-impls/plugin-proxy.cpp       |  5 ++---
 src/wine-host/bridges/vst3.cpp                |  5 +++++
 6 files changed, 43 insertions(+), 3 deletions(-)

diff --git a/src/common/logging/vst3.cpp b/src/common/logging/vst3.cpp
index c8d57369..f5bdc1b8 100644
--- a/src/common/logging/vst3.cpp
+++ b/src/common/logging/vst3.cpp
@@ -344,6 +344,17 @@ void Vst3Logger::log_request(
     });
 }
 
+void Vst3Logger::log_request(
+    bool is_host_vst,
+    const YaEditController2::PlainParamToNormalized& request) {
+    log_request_base(is_host_vst, [&](auto& message) {
+        message << request.instance_id
+                << ": IEditController::plainParamToNormalized(id = "
+                << request.id << ", plainValue = " << request.plain_value
+                << ")";
+    });
+}
+
 void Vst3Logger::log_request(bool is_host_vst,
                              const YaPluginBase::Initialize& request) {
     log_request_base(is_host_vst, [&](auto& message) {
diff --git a/src/common/logging/vst3.h b/src/common/logging/vst3.h
index c5a84b53..f098ad92 100644
--- a/src/common/logging/vst3.h
+++ b/src/common/logging/vst3.h
@@ -92,6 +92,8 @@ class Vst3Logger {
                      const YaEditController2::GetParamValueByString&);
     void log_request(bool is_host_vst,
                      const YaEditController2::NormalizedParamToPlain&);
+    void log_request(bool is_host_vst,
+                     const YaEditController2::PlainParamToNormalized&);
     void log_request(bool is_host_vst, const YaPluginBase::Initialize&);
     void log_request(bool is_host_vst, const YaPluginBase::Terminate&);
     void log_request(bool is_host_vst, const YaPluginFactory::Construct&);
diff --git a/src/common/serialization/vst3.h b/src/common/serialization/vst3.h
index 97bf7619..f0e3b2c1 100644
--- a/src/common/serialization/vst3.h
+++ b/src/common/serialization/vst3.h
@@ -82,6 +82,7 @@ using ControlRequest = std::variant<Vst3PluginProxy::Construct,
                                     YaEditController2::GetParamStringByValue,
                                     YaEditController2::GetParamValueByString,
                                     YaEditController2::NormalizedParamToPlain,
+                                    YaEditController2::PlainParamToNormalized,
                                     YaPluginBase::Initialize,
                                     YaPluginBase::Terminate,
                                     YaPluginFactory::Construct,
diff --git a/src/common/serialization/vst3/plugin/edit-controller.h b/src/common/serialization/vst3/plugin/edit-controller.h
index fd970b82..adea2cca 100644
--- a/src/common/serialization/vst3/plugin/edit-controller.h
+++ b/src/common/serialization/vst3/plugin/edit-controller.h
@@ -275,6 +275,28 @@ class YaEditController2 : public Steinberg::Vst::IEditController,
     virtual Steinberg::Vst::ParamValue PLUGIN_API normalizedParamToPlain(
         Steinberg::Vst::ParamID id,
         Steinberg::Vst::ParamValue valueNormalized) override = 0;
+
+    /**
+     * Message to pass through a call to
+     * `IEditController::plainParamToNormalized(id, plain_value)` to the Wine
+     * plugin host.
+     */
+    struct PlainParamToNormalized {
+        using Response = PrimitiveWrapper<Steinberg::Vst::ParamValue>;
+
+        native_size_t instance_id;
+
+        Steinberg::Vst::ParamID id;
+        Steinberg::Vst::ParamValue plain_value;
+
+        template <typename S>
+        void serialize(S& s) {
+            s.value8b(instance_id);
+            s.value4b(id);
+            s.value8b(plain_value);
+        }
+    };
+
     virtual Steinberg::Vst::ParamValue PLUGIN_API
     plainParamToNormalized(Steinberg::Vst::ParamID id,
                            Steinberg::Vst::ParamValue plainValue) override = 0;
diff --git a/src/plugin/bridges/vst3-impls/plugin-proxy.cpp b/src/plugin/bridges/vst3-impls/plugin-proxy.cpp
index fe1ed8d1..c04f4e9f 100644
--- a/src/plugin/bridges/vst3-impls/plugin-proxy.cpp
+++ b/src/plugin/bridges/vst3-impls/plugin-proxy.cpp
@@ -286,9 +286,8 @@ Steinberg::Vst::ParamValue PLUGIN_API
 Vst3PluginProxyImpl::plainParamToNormalized(
     Steinberg::Vst::ParamID id,
     Steinberg::Vst::ParamValue plainValue) {
-    // TODO: Implement
-    bridge.logger.log("TODO IEditController::plainParamToNormalized()");
-    return Steinberg::kNotImplemented;
+    return bridge.send_message(YaEditController2::PlainParamToNormalized{
+        .instance_id = instance_id(), .id = id, .plain_value = plainValue});
 }
 
 Steinberg::Vst::ParamValue PLUGIN_API
diff --git a/src/wine-host/bridges/vst3.cpp b/src/wine-host/bridges/vst3.cpp
index 2c27817d..5c1e73c2 100644
--- a/src/wine-host/bridges/vst3.cpp
+++ b/src/wine-host/bridges/vst3.cpp
@@ -313,6 +313,11 @@ void Vst3Bridge::run() {
                     .edit_controller->normalizedParamToPlain(
                         request.id, request.value_normalized);
             },
+            [&](const YaEditController2::PlainParamToNormalized& request) {
+                return object_instances[request.instance_id]
+                    .edit_controller->plainParamToNormalized(
+                        request.id, request.plain_value);
+            },
             [&](YaPluginBase::Initialize& request)
                 -> YaPluginBase::Initialize::Response {
                 // If we got passed a host context, we'll create a proxy object