From 666cd10b6d11b5ba314f48a5453fa52e24803420 Mon Sep 17 00:00:00 2001
From: George Kiagiadakis <george.kiagiadakis@collabora.com>
Date: Mon, 25 May 2020 19:06:52 +0300
Subject: [PATCH] algorithms: cleanup unused code

---
 modules/module-si-adapter/algorithms.c | 114 -------------------------
 modules/module-si-adapter/algorithms.h |   4 -
 2 files changed, 118 deletions(-)

diff --git a/modules/module-si-adapter/algorithms.c b/modules/module-si-adapter/algorithms.c
index c0058075..052f3c9c 100644
--- a/modules/module-si-adapter/algorithms.c
+++ b/modules/module-si-adapter/algorithms.c
@@ -8,126 +8,12 @@
 
 #include <wp/wp.h>
 #include <pipewire/pipewire.h>
-#include <spa/debug/types.h>
-#include <spa/param/audio/type-info.h>
 
-// for choose_sensible_raw_audio_format
 #include <spa/param/format-utils.h>
 #include <spa/param/audio/format.h>
 
 #include "algorithms.h"
 
-gboolean
-multiport_link_create (
-    GVariant * src_data,
-    GVariant * sink_data,
-    CreateLinkCb create_link_cb,
-    gpointer user_data,
-    GError ** error)
-{
-  g_autoptr (GPtrArray) in_ports = NULL;
-  GVariantIter *iter;
-  GVariant *child;
-  guint32 out_node_id, in_node_id;
-  guint32 out_port_id, in_port_id;
-  guint32 out_channel, in_channel;
-  guint8 direction;
-  guint i;
-  gboolean link_all = FALSE;
-
-  /* tuple format:
-      uint32 node_id;
-      uint32 port_id;
-      uint32 channel;  // enum spa_audio_channel
-      uint8 direction; // enum spa_direction
-   */
-  if (!g_variant_is_of_type (src_data, G_VARIANT_TYPE("a(uuuy)")))
-    goto invalid_argument;
-  if (!g_variant_is_of_type (sink_data, G_VARIANT_TYPE("a(uuuy)")))
-    goto invalid_argument;
-
-  /* transfer the in ports to an array so that we can
-     delete them when they are linked */
-  in_ports = g_ptr_array_new_full (g_variant_n_children (sink_data),
-      (GDestroyNotify) g_variant_unref);
-
-  g_variant_get (sink_data, "a(uuuy)", &iter);
-  while ((child = g_variant_iter_next_value (iter))) {
-    g_variant_get (child, "(uuuy)", NULL, NULL, NULL, &direction);
-    /* remove non-input direction ports right away */
-    if (direction == PW_DIRECTION_INPUT)
-      g_ptr_array_add (in_ports, child);
-    else
-      g_variant_unref (child);
-  }
-  g_variant_iter_free (iter);
-
-  /* now loop over the out ports and figure out where they should be linked */
-  g_variant_get (src_data, "a(uuuy)", &iter);
-
-  /* special case for mono inputs: link to all outputs,
-     since we don't support proper channel mapping yet */
-  if (g_variant_iter_n_children (iter) == 1)
-    link_all = TRUE;
-
-  while (g_variant_iter_loop (iter, "(uuuy)", &out_node_id, &out_port_id,
-              &out_channel, &direction))
-  {
-    /* skip non-output ports right away */
-    if (direction != PW_DIRECTION_OUTPUT)
-      continue;
-
-    for (i = 0; i < in_ports->len; i++) {
-      child = g_ptr_array_index (in_ports, i);
-      g_variant_get (child, "(uuuy)", &in_node_id, &in_port_id, &in_channel, NULL);
-
-      /* the channel has to match, unless we don't have any information
-         on channel ordering on either side */
-      if (link_all ||
-          out_channel == in_channel ||
-          out_channel == SPA_AUDIO_CHANNEL_UNKNOWN ||
-          in_channel == SPA_AUDIO_CHANNEL_UNKNOWN)
-      {
-        g_autoptr (WpProperties) props = NULL;
-
-        /* Create the properties */
-        props = wp_properties_new_empty ();
-        wp_properties_setf(props, PW_KEY_LINK_OUTPUT_NODE, "%u", out_node_id);
-        wp_properties_setf(props, PW_KEY_LINK_OUTPUT_PORT, "%u", out_port_id);
-        wp_properties_setf(props, PW_KEY_LINK_INPUT_NODE, "%u", in_node_id);
-        wp_properties_setf(props, PW_KEY_LINK_INPUT_PORT, "%u", in_port_id);
-
-        g_debug ("Create pw link: %u:%u (%s) -> %u:%u (%s)",
-            out_node_id, out_port_id,
-            spa_debug_type_find_name (spa_type_audio_channel, out_channel),
-            in_node_id, in_port_id,
-            spa_debug_type_find_name (spa_type_audio_channel, in_channel));
-
-        /* and the link */
-        create_link_cb (props, user_data);
-
-        /* continue to link all input ports, if requested */
-        if (link_all)
-          continue;
-
-        /* and remove the linked input port from the array */
-        g_ptr_array_remove_index (in_ports, i);
-
-        /* break out of the for loop; go for the next out port */
-        break;
-      }
-    }
-  }
-
-  return TRUE;
-
-invalid_argument:
-  g_set_error (error, WP_DOMAIN_LIBRARY, WP_LIBRARY_ERROR_INVALID_ARGUMENT,
-      "Endpoint node/port descriptions don't have the required fields");
-  return FALSE;
-}
-
-
 static enum spa_audio_format
 select_format (WpSpaPod *value)
 {
diff --git a/modules/module-si-adapter/algorithms.h b/modules/module-si-adapter/algorithms.h
index 300a1f18..90a4cf2b 100644
--- a/modules/module-si-adapter/algorithms.h
+++ b/modules/module-si-adapter/algorithms.h
@@ -6,10 +6,6 @@
  * SPDX-License-Identifier: MIT
  */
 
-typedef void (*CreateLinkCb) (WpProperties *, gpointer);
-gboolean multiport_link_create (GVariant * src_data, GVariant * sink_data,
-    CreateLinkCb create_link_cb, gpointer user_data, GError ** error);
-
 struct spa_audio_info_raw;
 gboolean choose_sensible_raw_audio_format (WpIterator *formats,
     gint channels_preference, struct spa_audio_info_raw *result);