[Differential] [Commented On] D1366: Add Event Sounds stream to Applications list

[broulik (Kai Uwe Broulik)]

broulik added inline comments.

INLINE COMMENTS
  src/context.cpp:150 Braces pls
  src/context.cpp:424 Avoid creating a temporary list keys() just to find something in it, use constFind and std::distance 
  src/eventstream.cpp:29-32 Initializer list:
  : SinkInput(parent)
  , m_clientIndex(PA_INVALID_INDEX)
  , ...
  src/eventstream.cpp:41 const
  src/kcm/package/contents/ui/StreamListItem.qml:74 I can't move notification sounds to another device? :(

REPOSITORY
  rPLASMAPA Plasma Audio Volume Applet

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

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

To: drosca, Plasma, Plasma: Design
Cc: broulik, colomar, plasma-devel, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160510/55d09bae/attachment.html>