[PATCH] Added set-(sink|source)-latency-offset commands to pactl and pacmd.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Tanu,

Thanks for your comments! I'll have a go at implementing your suggestions
maybe next week when I've got some time.

Cheers,

Chris

On Tue, Jan 12, 2016 at 6:56 PM, Tanu Kaskinen <tanuk at iki.fi> wrote:

> On Tue, 2016-01-12 at 09:45 +0200, Tanu Kaskinen wrote:
> > On Tue, 2016-01-12 at 17:05 +1100, Chris Billington wrote:
> > > Hi all,
> > >
> > > Just to bump this thread, what are the odds of getting this patch
> included
> > > in pulseaudio? It would be useful to many of the network audio
> streaming
> > > services that use pulseaudio (specifically my one :p).
> >
> > The odds are good, since this is a welcome addition. I don't know when
> > the patch will be merged, however (in any case it's too late for 8.0).
> > I don't promise a quick review myself due to lack of time. Peter
> > already commented, though, and he is able to approve patches. Peter, do
> > you plan to review this newest version of the patch?
> >
> > I do have some comments from just glancing at the patch, though:
> >
> > It would have been nice to separate the patch to the part that modifies
> > the protocol and the part(s) that add the new feature to pactl and
> > pacmd.
> >
> > The commit message should contain the justification for the change
> > (which you provided in a separate mail).
> >
> > The references to 8.0 need to be changed to 9.0.
> >
> > It would be good to have better separation between the sink/source
> > latency offset and the port latency offset. The patch doesn't seem to
> > handle it particularly well if a sink has a latency offset set and the
> > same sink also has a port that has a latency offset set. It would make
> > sense to add those two together when querying the sink latency.
> >
> > It would be good to save the offset on disk. The place to do that would
> > be module-device-restore. If you try to implement this, however, it
> > might turn out to be a bit tricky, because module-device-restore deals
> > with both per-sink and per-port entries depending on whether the sink
> > has ports. The sink latency offset should not be saved separately for
> > each port, but it's not obvious how to achieve that when module-device-
> > restore is in the "per-port mode".
>
> I forgot one more thing that would be good to have: updates to the bash
> and zsh completion files.
>
> --
> Tanu
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/pulseaudio-discuss/attachments/20160113/8052d69e/attachment.html>


[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux