work

.gitmodules

@@ -1,3 +1,6 @@
 [submodule "submodules/rohrkabel"]
 	path = submodules/rohrkabel
 	url = https://github.com/Soundux/rohrkabel
+[submodule "submodules/channel"]
+	path = submodules/channel
+	url = https://github.com/Soundux/channel.git

CMakeLists.txt

@@ -63,15 +63,20 @@ if(GIT_FOUND AND EXISTS "${PROJECT_SOURCE_DIR}/.git")
   endif()
 endif()
 
-if(NOT EXISTS "${PROJECT_SOURCE_DIR}/submodules/rohrkabel/CMakeLists.txt")
-  message(FATAL_ERROR "Rohrkabel was not found since you are not in a Git checkout or have GIT_SUBMODULE disabled. Please provide rohrkabel manually to `./submodules/rohrkabel`.")
+function(add_git_subdirectory SUBMODULE)
+  if(NOT EXISTS "${PROJECT_SOURCE_DIR}/submodules/${SUBMODULE}/CMakeLists.txt")
+    message(FATAL_ERROR "Submodule ${SUBMODULE} was not found since you are not in a Git checkout or have GIT_SUBMODULE disabled. Please provide ${SUBMODULE} manually to `./submodules/${SUBMODULE}`.")
   endif()
 
-add_subdirectory(submodules/rohrkabel)
+  add_subdirectory(submodules/${SUBMODULE})
+endfunction()
+
+add_git_subdirectory(rohrkabel)
+add_git_subdirectory(channel)
 
 add_executable(discord-screenaudio ${discord-screenaudio_SRC})
 
-target_link_libraries(discord-screenaudio Qt::Widgets Qt::WebEngineWidgets rohrkabel)
+target_link_libraries(discord-screenaudio Qt::Widgets Qt::WebEngineWidgets rohrkabel channel)
 
 if(KF5Notifications_FOUND)
   target_link_libraries(discord-screenaudio KF5::Notifications)

src/main.cpp

@@ -24,9 +24,6 @@ int main(int argc, char *argv[]) {
       "Custom Discord client with the ability to stream audio on Linux");
   parser.addHelpOption();
   parser.addVersionOption();
-  QCommandLineOption virtmicOption("virtmic", "Start the Virtual Microphone",
-                                   "target");
-  parser.addOption(virtmicOption);
   QCommandLineOption degubOption("remote-debugging",
                                  "Open Chromium Remote Debugging on port 9222");
   parser.addOption(degubOption);
@@ -36,10 +33,6 @@ int main(int argc, char *argv[]) {
 
   parser.process(app);
 
-  if (parser.isSet(virtmicOption)) {
-    Virtmic::start(parser.value(virtmicOption));
-  }
-
   qputenv("QTWEBENGINE_CHROMIUM_FLAGS",
           "--enable-features=WebRTCPipeWireCapturer " +
               qgetenv("QTWEBENGINE_CHROMIUM_FLAGS"));

src/mainwindow.h

@@ -1,6 +1,7 @@
 #pragma once
 
 #include "centralwidget.h"
+#include "virtmic.h"
 
 #include <QMainWindow>
 #include <QMenu>

src/streamdialog.cpp

@@ -1,4 +1,5 @@
 #include "streamdialog.h"
+#include "mainwindow.h"
 #include "virtmic.h"
 
 #include <QComboBox>

src/userscript.cpp

@@ -42,8 +42,12 @@ void UserScript::setupHelpMenu() {
   aboutData.addCredit(
       "Curve", "For creating the Rohrkabel library used in this project.",
       QString(), "https://github.com/Curve");
-  aboutData.addComponent("Rohrkabel", "A C++ RAII Pipewire-API Wrapper", "1.3",
+  aboutData.addComponent("Rohrkabel", "A C++ RAII Pipewire-API Wrapper", "1.5",
                          "https://github.com/Soundux/rohrkabel");
+  aboutData.addComponent(
+      "Soundux/channel ",
+      "A C++ implementation of Rust's std::sync::mpsc::channel", nullptr,
+      "https://github.com/Soundux/channel");
   m_helpMenu = new KHelpMenu(MainWindow::instance(), aboutData);
 #endif
 }

src/virtmic.cpp

@@ -1,95 +1,99 @@
 #include "virtmic.h"
 #include "log.h"
 
-#include <rohrkabel/loop/main.hpp>
-#include <rohrkabel/registry/registry.hpp>
+QThread virtmicThread;
+std::unique_ptr<pipewire::sender<Virtmic::set_target, Virtmic::terminate>>
+    senderr;
+std::unique_ptr<cr::receiver<Virtmic::new_targets>> receiverr;
 
-namespace Virtmic {
-
-const QStringList EXCLUDE_TARGETS{"Chromium input", "discord-screenaudio"};
-
-QVector<QString> getTargets() {
-  auto main_loop = pipewire::main_loop();
-  auto context = pipewire::context(main_loop);
-  auto core = pipewire::core(context);
-  auto reg = pipewire::registry(core);
-
-  QVector<QString> targets;
-
-  auto reg_listener = reg.listen<pipewire::registry_listener>();
-  reg_listener.on<pipewire::registry_event::global>(
-      [&](const pipewire::global &global) {
-        if (global.type == pipewire::node::type) {
-          auto node = reg.bind<pipewire::node>(global.id);
-          auto info = node.info();
-          QString name;
-          if (info.props.count("application.name") &&
-              info.props["application.name"] != "")
-            name = QString::fromStdString(info.props["application.name"]);
-          else
-            name = QString::fromStdString(
-                info.props["application.process.binary"]);
-
-          if (name != "" && !EXCLUDE_TARGETS.contains(name) &&
-              !targets.contains(name)) {
-            targets.append(name);
+void Virtmic::instance() {
+  if (!virtmicThread.isRunning()) {
+    auto [main_sender, main_receiver] = cr::channel<new_targets>();
+    auto [pw_sender, pw_receiver] = pipewire::channel<set_target, terminate>();
+    Virtmic *virtmic =
+        new Virtmic(std::move(pw_receiver), std::move(main_sender));
+    virtmic->moveToThread(&virtmicThread);
+    virtmicThread.start();
+    QMetaObject::invokeMethod(virtmic, "run");
+    receiverr = std::make_unique<cr::receiver<Virtmic::new_targets>>(
+        std::move(main_receiver));
+    senderr = std::make_unique<
+        pipewire::sender<Virtmic::set_target, Virtmic::terminate>>(
+        std::move(pw_sender));
   }
 }
-      });
-  core.update();
 
-  return targets;
+void Virtmic::setTarget(QString target) {
+  senderr.get()->send<Virtmic::set_target>({target});
 }
 
-void start(QString _target) {
-  std::map<std::uint32_t, pipewire::port> ports;
-  std::unique_ptr<pipewire::port> virt_fl, virt_fr;
+void Virtmic::getTargets() { senderr.get()->send<Virtmic::get_targets>(); }
 
-  std::map<std::uint32_t, pipewire::node_info> nodes;
-  std::map<std::uint32_t, pipewire::link_factory> links;
+Virtmic::Virtmic(pipewire::receiver<set_target, terminate> receiver,
+                 cr::sender<new_targets> sender) {
+  m_receiver = std::make_unique<pipewire::receiver<set_target, terminate>>(
+      std::move(receiver));
+  m_sender = std::make_unique<cr::sender<new_targets>>(std::move(sender));
+  virtual_mic = std::make_unique<pipewire::proxy>(
+      std::move(*core.create("adapter",
+                             {{"node.name", "discord-screenaudio-virtmic"},
+                              {"media.class", "Audio/Source/Virtual"},
+                              {"factory.name", "support.null-audio-sink"},
+                              {"audio.channels", "2"},
+                              {"audio.position", "FL,FR"}},
+                             pipewire::node::type, pipewire::node::version,
+                             pipewire::update_strategy::none)
+                     .get()));
+  metadata_listener.on<pipewire::registry_event::global>(
+      [&](const auto &global) { globalEvent(global); });
+  metadata_listener.on<pipewire::registry_event::global_removed>(
+      [&](const std::uint32_t id) { globalRemovedEvent(id); });
+}
 
-  auto main_loop = pipewire::main_loop();
-  auto context = pipewire::context(main_loop);
-  auto core = pipewire::core(context);
-  auto reg = pipewire::registry(core);
+void Virtmic::run() {
+  while (true) {
+    main_loop.run();
+  }
+}
 
-  auto link = [&](const std::string &target, pipewire::core &core) {
+void Virtmic::link() {
   for (const auto &[port_id, port] : ports) {
+    auto port_info = port.info();
     if (!virt_fl || !virt_fr)
       continue;
 
     if (links.count(port_id))
       continue;
 
-      if (port.info().direction == pipewire::port_direction::input)
+    if (port_info.direction == pipewire::port_direction::input)
       continue;
 
-      if (!port.info().props.count("node.id"))
+    if (!port_info.props.count("node.id"))
       continue;
 
-      auto parent_id = std::stoul(port.info().props["node.id"]);
+    auto parent_id = std::stoul(port_info.props["node.id"]);
 
     if (!nodes.count(parent_id))
       continue;
 
-      auto &parent = nodes.at(parent_id);
-      std::string name;
+    auto &parent = nodes[parent_id];
+    QString name;
     if (parent.props.count("application.name") &&
         parent.props["application.name"] != "")
-        name = parent.props["application.name"];
+      name = QString::fromStdString(parent.props["application.name"]);
     else
-        name = parent.props["application.process.binary"];
+      name = QString::fromStdString(parent.props["application.process.binary"]);
 
     if (name == target ||
-          (target == "[All Desktop Audio]" &&
-           !EXCLUDE_TARGETS.contains(QString::fromStdString(name)))) {
-        auto fl = port.info().props["audio.channel"] == "FL";
-        links.emplace(
-            port_id,
-            core.create<pipewire::link_factory>(
-                {fl ? virt_fl->info().id : virt_fr->info().id, port_id}));
+        (target == "[All Desktop Audio]" && !EXCLUDE_TARGETS.contains(name))) {
+      auto fl = port_info.props["audio.channel"] == "FL";
+      links.emplace(port_id,
+                    *core.create_simple<pipewire::link>(fl ? virt_fl->info().id
+                                                           : virt_fr->info().id,
+                                                        port_id)
+                         .get());
       qDebug(virtmicLog) << QString("Link: %1:%2 -> %3")
-                                  .arg(QString::fromStdString(name))
+                                .arg(name)
                                 .arg(port_id)
                                 .arg(fl ? virt_fl->info().id
                                         : virt_fr->info().id)
@@ -97,31 +101,13 @@ void start(QString _target) {
                                 .data();
     }
   }
-  };
-
-  std::string target = _target.toLatin1().toStdString();
-
-  auto virtual_mic = core.create("adapter",
-                                 {{"node.name", "discord-screenaudio-virtmic"},
-                                  {"media.class", "Audio/Source/Virtual"},
-                                  {"factory.name", "support.null-audio-sink"},
-                                  {"audio.channels", "2"},
-                                  {"audio.position", "FL,FR"}},
-                                 pipewire::node::type, pipewire::node::version,
-                                 pipewire::update_strategy::none);
-
-  if (target == "[None]") {
-    while (true) {
-      main_loop.run();
-    }
-    return;
 }
 
-  auto reg_events = reg.listen<pipewire::registry_listener>();
-  reg_events.on<pipewire::registry_event::global>(
-      [&](const pipewire::global &global) {
+void Virtmic::unlink() { links.clear(); }
+
+void Virtmic::globalEvent(const pipewire::global &global) {
   if (global.type == pipewire::node::type) {
-          auto node = reg.bind<pipewire::node>(global.id);
+    auto node = *reg.bind<pipewire::node>(global.id).get();
     auto info = node.info();
     std::string name;
     if (info.props.count("application.name") &&
@@ -138,17 +124,17 @@ void start(QString _target) {
 
     if (!nodes.count(global.id)) {
       nodes.emplace(global.id, node.info());
-            link(target, core);
+      link();
     }
   }
   if (global.type == pipewire::port::type) {
-          auto port = reg.bind<pipewire::port>(global.id);
+    auto port = *reg.bind<pipewire::port>(global.id).get();
     auto info = port.info();
 
     if (info.props.count("node.id")) {
       auto node_id = std::stoul(info.props["node.id"]);
 
-            if (node_id == virtual_mic.id() &&
+      if (node_id == virtual_mic.get()->id() &&
           info.direction == pipewire::port_direction::input) {
         if (info.props["audio.channel"] == "FL") {
           virt_fl = std::make_unique<pipewire::port>(std::move(port));
@@ -159,13 +145,12 @@ void start(QString _target) {
         ports.emplace(global.id, std::move(port));
       }
 
-            link(target, core);
+      link();
+    }
   }
 }
-      });
 
-  reg_events.on<pipewire::registry_event::global_removed>(
-      [&](const std::uint32_t id) {
+void Virtmic::globalRemovedEvent(const std::uint32_t id) {
   if (nodes.count(id)) {
     auto info = nodes.at(id);
     std::string name;
@@ -186,11 +171,4 @@ void start(QString _target) {
   if (links.count(id)) {
     links.erase(id);
   }
-      });
-
-  while (true) {
-    main_loop.run();
 }
-}
-
-} // namespace Virtmic

src/virtmic.h

@@ -1,12 +1,63 @@
 #pragma once
 
-#include <QString>
-#include <QVector>
+#include <cr/channel.hpp>
 #include <iostream>
+#include <rohrkabel/channel/channel.hpp>
+#include <rohrkabel/main_loop.hpp>
+#include <rohrkabel/registry/registry.hpp>
 
-namespace Virtmic {
+#include <QMap>
+#include <QScopedPointer>
+#include <QStringList>
+#include <QThread>
 
-QVector<QString> getTargets();
-void start(QString _target);
+class Virtmic : public QObject {
+  Q_OBJECT
+public:
+  static void setTarget(QString target);
+  static void getTargets();
 
-} // namespace Virtmic
+public:
+  struct set_target {
+    QString name;
+  };
+  struct get_targets {};
+  struct terminate {};
+  struct new_targets {
+    QStringList targets;
+  };
+
+protected:
+  static void instance();
+
+protected:
+  Virtmic(pipewire::receiver<set_target, terminate> receiver,
+          cr::sender<new_targets> sender);
+  void run();
+
+private:
+  std::unique_ptr<pipewire::receiver<set_target, terminate>> m_receiver;
+  std::unique_ptr<cr::sender<new_targets>> m_sender;
+
+  const QStringList EXCLUDE_TARGETS{"Chromium input", "discord-screenaudio"};
+  QString target;
+
+  pipewire::main_loop main_loop = pipewire::main_loop();
+  pipewire::context context = pipewire::context(main_loop);
+  pipewire::core core = pipewire::core(context);
+  pipewire::registry reg = pipewire::registry(core);
+
+  pipewire::registry_listener metadata_listener =
+      reg.listen<pipewire::registry_listener>();
+  std::unique_ptr<pipewire::proxy> virtual_mic;
+
+  std::map<uint32_t, pipewire::port> ports;
+  std::unique_ptr<pipewire::port> virt_fl, virt_fr;
+  std::map<uint32_t, pipewire::node_info> nodes;
+  std::map<uint32_t, pipewire::link> links;
+
+  void link();
+  void unlink();
+  void globalEvent(const pipewire::global &global);
+  void globalRemovedEvent(const std::uint32_t id);
+};

submodules/channel

@@ -0,0 +1 @@
+Subproject commit 6977815409b4c3c02d74a7aee3fc29f01d632feb

submodules/rohrkabel

@@ -1 +1 @@
-Subproject commit 04bfb921c44fb0d2337df70f5660899bc8d2844f
+Subproject commit 8a7705be070190a88b9a9d3619fa2fb7eabc951e