From 4a087750826904358ee19682c44376b64f0b59c9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C3=A9bastien=20Blin?=
 <sebastien.blin@savoirfairelinux.com>
Date: Thu, 14 Jul 2022 13:45:23 -0400
Subject: [PATCH] videomanager: follow API changes

Use return value of registerSinkTarget

Change-Id: Ibc7e8076a9446175d173b10aff18255893b1a8f7
---
 src/libclient/directrenderer.cpp            | 3 ++-
 src/libclient/qtwrapper/videomanager_wrap.h | 5 +++--
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/libclient/directrenderer.cpp b/src/libclient/directrenderer.cpp
index c6078340e..51209d9b3 100644
--- a/src/libclient/directrenderer.cpp
+++ b/src/libclient/directrenderer.cpp
@@ -39,7 +39,8 @@ public:
         , parent_(parent)
     {
         configureTarget();
-        VideoManager::instance().registerSinkTarget(parent_->id(), target);
+        if (!VideoManager::instance().registerSinkTarget(parent_->id(), target))
+            qWarning() << "Cannot register " << parent_->id();
     };
     ~Impl()
     {
diff --git a/src/libclient/qtwrapper/videomanager_wrap.h b/src/libclient/qtwrapper/videomanager_wrap.h
index 64f1a24be..fc0ce1ecc 100644
--- a/src/libclient/qtwrapper/videomanager_wrap.h
+++ b/src/libclient/qtwrapper/videomanager_wrap.h
@@ -141,13 +141,14 @@ public Q_SLOTS: // METHODS
 
     void stopAudioDevice() { DRing::stopAudioDevice(); }
 
-    void registerSinkTarget(const QString& sinkID, const DRing::SinkTarget& target)
+    bool registerSinkTarget(const QString& sinkID, const DRing::SinkTarget& target)
     {
 #ifdef ENABLE_VIDEO
-        DRing::registerSinkTarget(sinkID.toStdString(), target);
+        return DRing::registerSinkTarget(sinkID.toStdString(), target);
 #else
         Q_UNUSED(sinkID)
         Q_UNUSED(target)
+        return false;
 #endif
     }
 
-- 
GitLab