Most Bus properties ported to C++

beast-gtk/bstbuseditor.cc

@@ -90,10 +90,7 @@ get_property_param (BstBusEditor *self,
   Bse::BusH bus = Bse::BusH::down_cast (bse_server.from_proxy (self->item));
   for (auto cxxpspec : Bse::introspection_fields_to_param_list (bus.__aida_aux_data__()))
     {
-      std::string pname = cxxpspec->name;
-      for (auto& c : pname)
-        if (c == '-')
-          c = '_';
+      std::string pname = Bst::name_to_identifier (cxxpspec->name);
 
       if (property == pname)
         return bst_param_new_property (cxxpspec, bus);

beast-gtk/bstparam.cc

@@ -426,20 +426,6 @@ bst_param_new_rec (GParamSpec *pspec,
 // == Aida::Parameter binding ==
 static Bse::ObjectS* aida_property_binding_object (GxkParam *param);
 
-static std::string
-name_to_identifier (const std::string &name)
-{
-  if (strchr (name.c_str(), '-'))
-    {
-      std::string identifier (name);
-      for (size_t i = 0; i < identifier.size(); i++)
-        if (identifier[i] == '-')
-          identifier[i] = '_';
-      return identifier;
-    }
-  return name;
-}
-
 static void
 aida_property_binding_set_value (GxkParam *param, const GValue *value)
 {
@@ -472,18 +458,18 @@ aida_property_binding_set_value (GxkParam *param, const GValue *value)
       Bse::warning ("%s: unsupported type: %s", __func__, g_type_name (G_PARAM_SPEC_VALUE_TYPE (param->pspec)));
       return;
     }
-  if (!objectp->__aida_set__ (name_to_identifier (pspec->name), any))
-    Bse::warning ("%s: __aida_set__: unknown value name: %s", __func__, name_to_identifier (pspec->name));
+  if (!objectp->__aida_set__ (Bst::name_to_identifier (pspec->name), any))
+    Bse::warning ("%s: __aida_set__: unknown value name: %s", __func__, Bst::name_to_identifier (pspec->name));
 }
 
 static void
 aida_property_binding_get_value (GxkParam *param, GValue *param_value)
 {
   Bse::ObjectS *objectp = aida_property_binding_object (param);
   GParamSpec *pspec = param->pspec;
-  Any any = objectp->__aida_get__ (name_to_identifier (pspec->name));
+  Any any = objectp->__aida_get__ (Bst::name_to_identifier (pspec->name));
   if (any.empty())
-    Bse::warning ("%s: __aida_get__: unknown value name: %s (empty return value)", __func__, name_to_identifier (pspec->name));
+    Bse::warning ("%s: __aida_get__: unknown value name: %s (empty return value)", __func__, Bst::name_to_identifier (pspec->name));
   GValue value = { 0, };
   switch (G_TYPE_FUNDAMENTAL (G_PARAM_SPEC_VALUE_TYPE (param->pspec)))
     {
@@ -568,7 +554,7 @@ bst_param_new_property (GParamSpec *pspec, const Bse::ObjectH handle)
   auto notify = [param] (const Aida::Event &event) {
     gxk_param_update (param);
   };
-  objectp->on (String ("notify:") + name_to_identifier (param->pspec->name), notify);
+  objectp->on (String ("notify:") + Bst::name_to_identifier (param->pspec->name), notify);
   gxk_param_set_size_group (param, param_size_group);
   return param;
 }

beast-gtk/bstutils.cc

@@ -159,6 +159,21 @@ monitor_fields_from_shm (int64 shm_id, uint32 shm_offset)
   return MonitorFieldU { (char*) (shm_start + shm_offset) };
 }
 
+// param spec helper: name -> identifier
+std::string
+name_to_identifier (const std::string &name)
+{
+  if (strchr (name.c_str(), '-'))
+    {
+      std::string identifier (name);
+      for (size_t i = 0; i < identifier.size(); i++)
+        if (identifier[i] == '-')
+          identifier[i] = '_';
+      return identifier;
+    }
+  return name;
+}
+
 } // Bst
 
 /* --- variables --- */
@@ -425,6 +440,7 @@ bst_background_handler2_add (gboolean       (*handler) (gpointer data),
   bst_background_handler_add (handler, data, free_func, 2);
 }
 
+
 /* --- packing utilities --- */
 #define SPACING 3
 static void

beast-gtk/bstutils.hh

@@ -39,6 +39,8 @@ public:
 
 MonitorFieldU monitor_fields_from_shm (int64 shm_id, uint32 shm_offset);
 
+std::string name_to_identifier (const std::string &name);
+
 } // Bst
 
 // == Bse Server (BSE remote origin) ==