[pulseaudio-discuss] [pulseaudio-commits] src/modules

Tanu Kaskinen tanu.kaskinen at linux.intel.com
Thu Jun 27 19:40:35 PDT 2013


On Thu, 2013-06-27 at 21:57 +0200, David Henningsson wrote:
> On 06/27/2013 06:19 PM, Tanu Kaskinen wrote:
> >   src/modules/alsa/alsa-mixer.c |    5 ++---
> >   1 file changed, 2 insertions(+), 3 deletions(-)
> >
> > New commits:
> > commit 2613e4c74733e67d56af165df4637bf902b08508
> > Author: Tanu Kaskinen <tanu.kaskinen at linux.intel.com>
> > Date:   Thu Jun 27 18:47:12 2013 +0300
> >
> >      alsa-mixer: Add a couple of assertions
> >
> >      I checked the code to ensure that the assertions hold currently.
> >
> > diff --git a/src/modules/alsa/alsa-mixer.c b/src/modules/alsa/alsa-mixer.c
> > index f4410d7..b2f6c2e 100644
> > --- a/src/modules/alsa/alsa-mixer.c
> > +++ b/src/modules/alsa/alsa-mixer.c
> > @@ -4530,10 +4530,9 @@ void pa_alsa_path_set_add_ports(
> >       pa_alsa_path *path;
> >       void *state;
> >
> > +    pa_assert(ps);
> >       pa_assert(ports);
> > -
> > -    if (!ps)
> > -        return;
> 
> Spontaneous NAK for the above change.
> 
> I like the code the way I wrote it. Please explain.

Why would you ever pass NULL path set? The whole point of the function
is to generate ports from a path set.

-- 
Tanu



More information about the pulseaudio-discuss mailing list