Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 588584 - <media-video/smplayer-16.8.0: incorrect volume levels in GUI/OSD with >=media-video/mpv-0.18.1
Summary: <media-video/smplayer-16.8.0: incorrect volume levels in GUI/OSD with >=media...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Qt Bug Alias
URL:
Whiteboard:
Keywords:
Depends on: 591444
Blocks:
  Show dependency tree
 
Reported: 2016-07-11 09:32 UTC by Coacher
Modified: 2016-08-18 14:27 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Coacher 2016-07-11 09:32:05 UTC
Hello.

mpv-0.18.1 has changed how softvol works [1,2] and smplayer needs to adapt.

The above changes result in the following behaviour:

- if softvol control is OFF in smplayer and maximum amplification is X (X>100),
  then OSD volume bar has maximum volume level X, i.e. there's a non-filled bar
  area even when volume is set to 100.
  IIRC the default maximum amplification value used in smplayer is 110.

- if softvol control is ON in smplayer and maximum amplification is X (X>100),
  then changing volume to 100 via GUI volume slider actually changes it to X.
  OSD volume bar displays volume level correctly though.

This is with smplayer-16.6.0 and mpv-0.18.1.
mpv-0.18.1 will appear in tree in about a week, see bug 588492.


[1]: https://github.com/mpv-player/mpv/issues/3322
[2]: https://github.com/mpv-player/mpv/commit/995c47da9afacfcc30d8a17d2f2a2268fbf4fdb9
Comment 1 Coacher 2016-07-11 09:42:38 UTC
Please note that software volume and regular volume work properly.
Only the way volume level is presented to the user is broken/inconsistent.
Comment 2 Coacher 2016-07-12 06:26:07 UTC
Same with smplayer-16.7.0.
Comment 3 Coacher 2016-07-12 08:28:24 UTC
Fixed in https://app.assembla.com/spaces/smplayer/subversion/commits/8016
Should be available in the next smplayer release after 16.7.0.
Comment 4 Coacher 2016-08-15 15:00:10 UTC
Fixed in smplayer-16.8.0, which was released about 2 weeks ago.
Comment 5 Coacher 2016-08-18 14:27:17 UTC
I can confirm this is fixed in smplayer-16.8.0.