Package: clementine
Version: 1.3.1+git276-g3485bbe43+dfsg-1.2
Severity: wishlist
Tags: patch

In its current form, clementine is power hungry. Just for basic song
playback, it consume on an average 25% of CPU time, on my quad-core i7.

Attached patch, picked from a PR on the upstream repo, substantially
cuts down the power consumptionto half.

Please have a look at the patch. It looks simple enough that it should
be okay to include into Debian.

Thanks

-- System Information:
Debian Release: buster/sid
  APT prefers unstable
  APT policy: (990, 'unstable'), (900, 'testing'), (500, 'unstable-debug'), 
(500, 'testing-debug'), (100, 'experimental'), (1, 'experimental-debug')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.14.0-2-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_IN, LC_CTYPE=en_IN (charmap=UTF-8), LANGUAGE=en_IN:en 
(charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages clementine depends on:
ii  gstreamer1.0-plugins-base       1.12.4-1
ii  gstreamer1.0-plugins-good       1.12.4-1
ii  gstreamer1.0-plugins-ugly       1.12.4-1
ii  libc6                           2.25-5
ii  libcdio17                       1.0.0-2
ii  libchromaprint1                 1.4.2-1
ii  libcrypto++6                    5.6.4-8
ii  libfftw3-double3                3.3.7-1
ii  libgcc1                         1:7.2.0-18
ii  libgdk-pixbuf2.0-0              2.36.11-1
ii  libglib2.0-0                    2.54.2-4
ii  libgpod4                        0.8.3-11
ii  libgstreamer-plugins-base1.0-0  1.12.4-1
ii  libgstreamer1.0-0               1.12.4-1
ii  libimobiledevice6               1.2.1~git20171128.5a854327+dfsg-0.1
ii  liblastfm1                      1.0.9-1
ii  libmtp9                         1.1.13-1
ii  libmygpo-qt1                    1.0.9-2
ii  libplist3                       2.0.0-2
ii  libprojectm2v5                  2.1.0+dfsg-4+b3
ii  libprotobuf10                   3.0.0-9.1
ii  libpulse0                       11.1-4
ii  libqjson0                       0.8.1-3+b1
ii  libqt4-dbus                     4:4.8.7+dfsg-11
ii  libqt4-network                  4:4.8.7+dfsg-11
ii  libqt4-opengl                   4:4.8.7+dfsg-11
ii  libqt4-sql                      4:4.8.7+dfsg-11
ii  libqt4-sql-sqlite               4:4.8.7+dfsg-11
ii  libqt4-xml                      4:4.8.7+dfsg-11
ii  libqtcore4                      4:4.8.7+dfsg-11
ii  libqtgui4                       4:4.8.7+dfsg-11
ii  libqxt-gui0                     0.6.2-3
ii  libsqlite3-0                    3.21.0-1
ii  libstdc++6                      7.2.0-18
ii  libtag1v5                       1.11.1+dfsg.1-0.2
ii  libx11-6                        2:1.6.4-3
ii  projectm-data                   2.1.0+dfsg-4
ii  zlib1g                          1:1.2.8.dfsg-5

Versions of packages clementine recommends:
ii  gstreamer1.0-pulseaudio  1.12.4-1

Versions of packages clementine suggests:
ii  gstreamer1.0-plugins-bad  1.12.4-2

-- no debconf information
Pull power saving changes.
Available as PR at:
https://github.com/clementine-player/Clementine/pull/5511
--- a/src/playlist/playlistview.cpp
+++ b/src/playlist/playlistview.cpp
@@ -120,7 +120,7 @@
       last_height_(-1),
       last_width_(-1),
       force_background_redraw_(false),
-      glow_enabled_(true),
+      glow_enabled_(false),
       currently_glowing_(false),
       glow_intensity_step_(0),
       rating_delegate_(nullptr),
@@ -1068,7 +1068,7 @@
 void PlaylistView::ReloadSettings() {
   QSettings s;
   s.beginGroup(Playlist::kSettingsGroup);
-  glow_enabled_ = s.value("glow_effect", true).toBool();
+  glow_enabled_ = s.value("glow_effect", false).toBool();
 
   if (setting_initial_header_layout_ || upgrading_from_qheaderview_) {
     header_->SetStretchEnabled(s.value("stretch", true).toBool());
--- a/src/ui/mainwindow.cpp
+++ b/src/ui/mainwindow.cpp
@@ -158,7 +158,7 @@
 const char* MainWindow::kAllFilesFilterSpec = QT_TR_NOOP("All Files (*)");
 
 namespace {
-const int kTrackSliderUpdateTimeMs = 40;
+const int kTrackSliderUpdateTimeMs = 500;
 const int kTrackPositionUpdateTimeMs = 1000;
 }
 

Reply via email to