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

David Henningsson david.henningsson at canonical.com
Thu Jun 27 12:57:41 PDT 2013


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.

> +    pa_assert(core);

This one I don't care about.

>
>       PA_HASHMAP_FOREACH(path, ps->paths, state) {
>           if (!path->settings || !path->settings->next) {
>
> _______________________________________________
> pulseaudio-commits mailing list
> pulseaudio-commits at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/pulseaudio-commits
>



-- 
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic


More information about the pulseaudio-discuss mailing list