Summary: | When pausing music at the end of the fadeout increased volume for fraction of second appears | ||
---|---|---|---|
Product: | [Applications] amarok | Reporter: | Vit Pelcak <vit> |
Component: | Playback | Assignee: | Amarok Developers <amarok-bugs-dist> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | matej |
Priority: | NOR | ||
Version: | 2.8.0 | ||
Target Milestone: | 2.9 | ||
Platform: | Gentoo Packages | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/amarok/69f0e02aa3519d2b28cddbbb6d6535786823a4aa | Version Fixed In: | 2.8.1 |
Sentry Crash Report: | |||
Attachments: | Possible fix for fade-out glitch |
Description
Vit Pelcak
2013-08-19 14:59:10 UTC
I can't reproduce this here at all, what type of tracks are these? Is this only on one single track or on all tracks regardless of the type? Just to make sure: do you have ReplayGain active? Please disable it as it could falsify the results. I think I've also hit this in past and I may have a clue what we do wrong in the code. I'll look at this as time permits (not really soon unfortunately). @Vit: Does it issue happen always for you, or just sometimes? And did you change the default fade-out duration? It happens to me always and I use just default fade out. Created attachment 82033 [details]
Possible fix for fade-out glitch
Please try with the attached patch. Unfortunatelly, I didn't spot the difference. :-( Confirming because of multiple reports from users. Unfortunately I still can't reproduce the issue here, though. Git commit 69f0e02aa3519d2b28cddbbb6d6535786823a4aa by Mark Kretschmann. Committed on 13/11/2013 at 05:43. Pushed by markey into branch 'master'. Fix sound glitch with "fade-out on pause" with GStreamer. This is really just an ugly workaround for an issue with (Phonon-)GStreamer. Essentially there is no way to flush the audio buffers, so that implementing fade-out in combination with pausing becomes a bit of a gamble. Please test, as I cannot reproduce the sound glitch on my system! FIXED-IN: 2.8.1 BACKPORT M +1 -0 ChangeLog M +1 -1 src/EngineController.cpp http://commits.kde.org/amarok/69f0e02aa3519d2b28cddbbb6d6535786823a4aa |