[pulseaudio-discuss] [Patch] RAOP: fix audio synchronisation, take two

Colin Leroy colin at colino.net
Sun Sep 17 18:54:19 UTC 2017


On 17 September 2017 at 19h51, Tanu Kaskinen wrote:

Hi, 

> "git am" fails to apply the patch on top of the master branch (as it
> failed to apply the previous versions too, but I forgot to mention
> that earlier).

Sorry, I forgot to pull :) 

> The commit message could explain also why pa_smoother_reset() is used
> instead of pause()/resume().

Done.

> Why do you check the return value of pa_modargs_get_value() before
> calling pa_modargs_get_value_u32()? As far as I can tell, that makes
> no difference.

pa_modargs_get_value_u32() (and its friends) set the parameter to 0 if
it's not present, returning -1 only in case of integer conversion error.

> Nitpicking: the convention is to use the PA_USEC_PER_MSEC constant
> instead of bare "1000".

Fixed too :)

I attached an updated patch.

-- 
Colin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-RAOP-Announce-real-latency.patch
Type: text/x-patch
Size: 7713 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/pulseaudio-discuss/attachments/20170917/7da357b4/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 801 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/pulseaudio-discuss/attachments/20170917/7da357b4/attachment.sig>


More information about the pulseaudio-discuss mailing list