[pulseaudio-discuss] [Patch] RAOP: fix audio synchronisation, take two
Tanu Kaskinen
tanuk at iki.fi
Thu Sep 21 13:54:14 UTC 2017
On Tue, 2017-09-19 at 19:56 +0200, Colin Leroy wrote:
> On 18 September 2017 at 22h47, Tanu Kaskinen wrote:
>
> Hi,
>
> > No, the function doesn't set the parameter when the argument is not
> > present.
>
> Oh, right, I misread it. I could have initialized the argument to an
> arbitrary value and compare that after parsing, but that would be
> uglier.
>
> > Since the function returns zero both when a valid value is provided
> > and when the argument is not set, you need to call
> > pa_modargs_get_value() in order to know whether to set the
> > latency_set flag or not, so it's not as useless as I first thought.
>
> Indeed. Reworked that part a bit as I didn't need a temporary variable
> in fact.
>
> > If the argument is set, but parsing it fails, you should log an error
> > message and fail. (Somehow it didn't occur to me to complain about
> > this earlier.)
>
> Thanks, done.
Thanks! Applied. I took the liberty to remove the
pa_modargs_get_value() from raop-sink.c, because it wasn't necessary
there.
--
Tanu
https://www.patreon.com/tanuk
More information about the pulseaudio-discuss
mailing list