D5657: StreamRestore: Don't write unnecessary changes

[David Rosca]

drosca created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  Applet updates mute state with every volume change - it does
  writeChanges(volume) immediately following with writeChanges(mute)
  and it seems to make PulseAudio to ignore the volume change.
  This makes sure that writeChanges() is only called when something really
  changed.
  
  BUG: 367688
  FIXED-IN: 5.10.0

TEST PLAN
  I can't reproduce the issue myself, but it was confirmed in bugreport
  that this patch fixes the issue.

REPOSITORY
  R115 Plasma Audio Volume Applet

BRANCH
  master

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

AFFECTED FILES
  src/streamrestore.cpp

To: drosca, #plasma
Cc: plasma-devel, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170429/61a564fb/attachment.html>