Bug#734100: Fixed in VLC 3.0

[Modestas Vainius]

Hello,

Sekmadienis 01 Lapkritis 2015 22:43:17 rašė:
> reopen 734100
> thanks
> 
> On Sunday 01 November 2015 22:03:41 Modestas Vainius wrote:
> > Hello,
> > 
> > Sekmadienis 01 Lapkritis 2015 18:10:54 rašė:
> > > tags 734100 + fixed-upstream
> > > thanks
> > > 
> > > This should be fixed in upstream VLC 3.0.
> > 
> > Thanks a lot. Could this package be backported to current packages?
> 
> VLC 3.0 is nowhere near release at this point in time. This is up to the
> Debian Multimedia team, not me, but I would not advise uploading it to
> unstable. And I do not know if there are enough human resources to maintain
> an extra VLC 3.0 in experimental.

What I really meant was backporting of those patches to 2.2.1 packages. 
Patches seemed pretty "short" at the glance hence I tried to do that myself.

47f74a83c161173b0d15e95dab8ceb7c97de51b4.patch was strightforward, applies 
cleanly.

Unfortunately, 6ae2905ef7fbc7de3a3a4a1bdf8ad6df46ce570a wasn't that 
strightforward. Had to do some modifications: 1st hunk already there in 2.2.1, 
adapt DecoderPlayAudio hunks and remove the last 2 hunks (from DecoderThread 
routine) since those locks are not present there in 2.2.1 codebase. I attach 
both patches.

The end result builds fine and seems to fix the problem on my machine. Not 
sure if patches break anything on 2.2.1 (i.e. if they need anything else from 
3.0.0 to work properly). Rémi, maybe you could comment on that?

> 
> > This
> > problem has been annoying me for a long time and lately it got only worse.
> > Unfortunately, increasing dmix buffer has some unwanted side effects in
> > other apps (audio-video desync).
> 
> Either those other apps are buggy, or dmix is buggy. The audio delay value
> should account for the dmix buffer.
> 
> Also, PulseAudio is better at mixing multiple apps.

Well, yeah, but I kind of dislike PulseAudio since its inception. I'm slowly 
warming up to the idea that I may have to use it one day but that time has not 
come yet...

-- 
Modestas Vainius <modestas at vainius.eu>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 47f74a83c161173b0d15e95dab8ceb7c97de51b4.patch
Type: text/x-patch
Size: 1530 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-multimedia-maintainers/attachments/20151102/7c89c9f7/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 6ae2905ef7fbc7de3a3a4a1bdf8ad6df46ce570a.patch
Type: text/x-patch
Size: 952 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-multimedia-maintainers/attachments/20151102/7c89c9f7/attachment-0003.bin>