[Differential] [Accepted] D2270: Try to be more clever about what sink to use for volume control

davidedmundson (David Edmundson) noreply at phabricator.kde.org
Sun Jul 24 19:19:28 UTC 2016


davidedmundson accepted this revision.
davidedmundson added a reviewer: davidedmundson.
davidedmundson added inline comments.
This revision is now accepted and ready to land.

INLINE COMMENTS

> device.cpp:68
> +        return IdleState;
> +    case 2:  // PA_X_SUSPENDED
> +        return SuspendedState;

Use the values from pa_sink_state ?

> device.h:128
>      quint32 m_activePortIndex = -1;
> +    State m_state;
>  };

it's worth initialising this to Unknown.

REPOSITORY
  rPLASMAPA Plasma Audio Volume Applet

BRANCH
  preferred-devices (branched from master)

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: drosca, #plasma, davidedmundson
Cc: davidedmundson, ochurlaud, plasma-devel, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160724/0c9f110c/attachment-0001.html>


More information about the Plasma-devel mailing list