[pulseaudio-discuss] [PATCH] alsa: Turn one assertion into two

[Tanu Kaskinen]

On Mon, 2013-09-23 at 09:55 +0200, David Henningsson wrote:
> According to coding style, one should have one assertion per line
> and not combine assertions.
> 
> Signed-off-by: David Henningsson <david.henningsson at canonical.com>
> ---
>  src/modules/alsa/module-alsa-card.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/modules/alsa/module-alsa-card.c b/src/modules/alsa/module-alsa-card.c
> index 96e88e5..be982ed 100644
> --- a/src/modules/alsa/module-alsa-card.c
> +++ b/src/modules/alsa/module-alsa-card.c
> @@ -378,8 +378,8 @@ static int report_jack_state(snd_hctl_elem_t *elem, unsigned int mask) {
>                  pa_assert(port);
>              }
>              else {
> -                pa_assert(jack->path && jack->path->port);
> -                port = jack->path->port;
> +                pa_assert(jack->path);
> +                pa_assert_se(port = jack->path->port);
>              }
>              report_port_state(port, u);
>          }

Ack.

-- 
Tanu