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 <[email protected]>
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
[email protected]
http://lists.freedesktop.org/mailman/listinfo/pulseaudio-commits
--
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic
_______________________________________________
pulseaudio-discuss mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
