[vlc-devel] [vlc-commits] pulse: restart when we need to move a passthrough input
thomas at gllm.fr
Mon Sep 26 14:38:56 CEST 2016
On Mon, Sep 26, 2016, at 14:16, Rémi Denis-Courmont wrote:
> Le lundi 26 septembre 2016, 11:34:40 Thomas Guillem a écrit :
> > On Sat, Sep 24, 2016, at 01:16, remi at remlab.net wrote:
> > > Le 22 sept. 2016 11:32, git at videolan.org a écrit :
> > > > vlc | branch: master | Thomas Guillem <thomas at gllm.fr> | Thu Sep 22
> > > > 16:40:43 2016 +0200| [a3925097aa23b6179ddf61bb04e43cd9a90e6f7f] |
> > > > committer: Thomas Guillem
> > > >
> > > > pulse: restart when we need to move a passthrough input
> > >
> > > Why? If the moved-to sink supports pass-through, there should be no
> > > problems. If it doesn't, then we should get a format-lost event so the
> > > change seems redundant. If our input is movef by another client or by
> > > the daemon, this patch has no effects. And if somehow policy rejects
> > > the change, the code just breakd audio needlessly.
> > Could I request a restart only if the sink move failed ?
> You don't know why it failed and that would be racy anyway.
> > I agree this is not the same behavior when you change a sink via
> > pavucontrol. But, for me, if the user explicitly selects a new device,
> > we shouldn't ignore it.
> Maybe we're missing a flag somewhere, or your PA configuration is not how
> actually want it, or your PA version is misdesigned or buggy.
I think pulse audio is well configured on debian testing. I understand
why it's impossible to change from SPDIF to PCM. I don't think there is
or there will be a callback that say: please re-configure your decoder,
the configured input format is not working anymore.
> Also I fail to see an explanation why we need this, please revert. This
> Rémi Denis-Courmont
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
More information about the vlc-devel