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

David Henningsson david.henningsson at canonical.com
Mon Sep 23 00:55:00 PDT 2013


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);
         }
-- 
1.7.9.5



More information about the pulseaudio-discuss mailing list