<div dir="ltr">Thanks Tanu,<div><br></div><div>I'm on something of a deadline writing a thesis before moving country, so I won't be able to get back to this for probably about two months. But I don't plan on abandoning it since I use this functionality daily to stream audio in my house.</div><div><br></div><div>I'll implement <span style="font-size:12.8px">querying of latency offsets and bump the version number to whatever it is at the time I get back to it.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Thanks for looking at it,</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Chris</span></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jul 15, 2016 at 4:29 AM, Tanu Kaskinen <span dir="ltr"><<a href="mailto:tanuk@iki.fi" target="_blank">tanuk@iki.fi</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Sat, 2016-01-23 at 12:31 +1100, Chris Billington wrote:<br>
> Bumped protocol version number to 31.<br>
<br>
</span>As time has passed by, the version should now be bumped to 32.<br>
<span class=""><br>
> These commands allow applications to set the latency offset of individual<br>
> sources and sinks, using functionality introduced in the last commit.<br>
><br>
> This is useful for setting the latency offset of a source or sink that is not<br>
> tied to a particular port, such as a null sink being used to capture output<br>
> and stream it over a network. If the application doing this knows the network<br>
> latency, then it can use these new protocol commands to inform PulseAudio of<br>
> it, so that video and audio may remain in sync.<br>
<br>
</span>I noticed that neither this patch nor any other patch implements<br>
querying the sink/source latency offset. I think it's pretty important<br>
to be able to also query the current offsets, if there's a mechanism<br>
for setting the offsets.<br>
<br>
Adding the querying support should be pretty simple. Basically you just<br>
need to add a new field to the pa_sink/source_info struct and send the<br>
value in the GET_SINK/SOURCE_INFO command. pactl should be updated too<br>
to print the value in the "list sinks/sources" commands.<br>
<br>
This patch looks good otherwise.<br>
<span class="HOEnZb"><font color="#888888"><br>
-- <br>
Tanu<br>
</font></span></blockquote></div><br></div>