D28822: Create StreamRestore channel if none exists

Nicolas Fella noreply at phabricator.kde.org
Tue Apr 14 14:46:27 BST 2020


nicolasfella created this revision.
nicolasfella added reviewers: Plasma, drosca, broulik.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
nicolasfella requested review of this revision.

REVISION SUMMARY
  We need one channel to be able to control the volume. This restores D7485 <https://phabricator.kde.org/D7485> which was broken by D20450 <https://phabricator.kde.org/D20450> while fixing a different issue.
  This patch tries to combine the two by only creating a new channel if none exists and not messing with existing ones.

TEST PLAN
  Can control the notification volume in the KCM now. The slider position matches the one in pavucontrol

REPOSITORY
  R115 Plasma Audio Volume Applet

BRANCH
  volcontrol

REVISION DETAIL
  https://phabricator.kde.org/D28822

AFFECTED FILES
  src/streamrestore.cpp

To: nicolasfella, #plasma, drosca, broulik
Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, ragreen, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200414/9b8f3a87/attachment.html>


More information about the Plasma-devel mailing list