]> code.delx.au - pulseaudio/blobdiff - src/pulsecore/namereg.c
merge 'lennart' branch back into trunk.
[pulseaudio] / src / pulsecore / namereg.c
index bb5eb7bea22c64055335d24ea4bca5e4af3434b4..fe2be4671490778e6dd0f7a917b46bef1bd56764 100644 (file)
@@ -2,17 +2,19 @@
 
 /***
   This file is part of PulseAudio.
+
+  Copyright 2004-2006 Lennart Poettering
+
   PulseAudio is free software; you can redistribute it and/or modify
   it under the terms of the GNU Lesser General Public License as published
   by the Free Software Foundation; either version 2 of the License,
   or (at your option) any later version.
+
   PulseAudio is distributed in the hope that it will be useful, but
   WITHOUT ANY WARRANTY; without even the implied warranty of
   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
   General Public License for more details.
+
   You should have received a copy of the GNU Lesser General Public License
   along with PulseAudio; if not, write to the Free Software
   Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307
@@ -25,7 +27,6 @@
 
 #include <stdlib.h>
 #include <string.h>
-#include <assert.h>
 #include <string.h>
 #include <stdio.h>
 
@@ -36,6 +37,7 @@
 #include <pulsecore/sink.h>
 #include <pulsecore/core-subscribe.h>
 #include <pulsecore/core-util.h>
+#include <pulsecore/macro.h>
 
 #include "namereg.h"
 
@@ -54,23 +56,23 @@ static int is_valid_char(char c) {
         c == '_';
 }
 
-int pa_namereg_is_valid_name(const char *name) {
+static int is_valid_name(const char *name) {
     const char *c;
 
     if (*name == 0)
         return 0;
-    
+
     for (c = name; *c && (c-name < PA_NAME_MAX); c++)
         if (!is_valid_char(*c))
             return 0;
 
     if (*c)
         return 0;
-    
+
     return 1;
 }
 
-char* pa_namereg_cleanup_name(const char *name) {
+static char* cleanup_name(const char *name) {
     const char *a;
     char *b, *n;
 
@@ -78,7 +80,7 @@ char* pa_namereg_cleanup_name(const char *name) {
         return NULL;
 
     n = pa_xnew(char, strlen(name)+1);
-    
+
     for (a = name, b = n; *a && (a-name < PA_NAME_MAX); a++, b++)
         *b = is_valid_char(*a) ? *a : '_';
 
@@ -88,33 +90,33 @@ char* pa_namereg_cleanup_name(const char *name) {
 }
 
 void pa_namereg_free(pa_core *c) {
-    assert(c);
-    
+    pa_assert(c);
+
     if (!c->namereg)
         return;
-    
-    assert(pa_hashmap_size(c->namereg) == 0);
+
+    pa_assert(pa_hashmap_size(c->namereg) == 0);
     pa_hashmap_free(c->namereg, NULL, NULL);
 }
 
 const char *pa_namereg_register(pa_core *c, const char *name, pa_namereg_type_t type, void *data, int fail) {
     struct namereg_entry *e;
     char *n = NULL;
-    int r;
-    
-    assert(c);
-    assert(name);
-    assert(data);
+
+    pa_assert(c);
+    pa_assert(name);
+    pa_assert(data);
 
     if (!*name)
         return NULL;
-    
-    if (!pa_namereg_is_valid_name(name)) {
-        
+
+    if ((type == PA_NAMEREG_SINK || type == PA_NAMEREG_SOURCE) &&
+        !is_valid_name(name) ) {
+
         if (fail)
             return NULL;
 
-        if (!(name = n = pa_namereg_cleanup_name(name)))
+        if (!(name = n = cleanup_name(name)))
             return NULL;
     }
 
@@ -135,11 +137,11 @@ const char *pa_namereg_register(pa_core *c, const char *name, pa_namereg_type_t
             pa_xfree(n);
             return NULL;
         }
-        
+
         k = pa_xnew(char, l+4);
-        
+
         for (i = 2; i <= 99; i++) {
-            snprintf(k, l+4, "%s.%u", name, i);
+            pa_snprintf(k, l+4, "%s.%u", name, i);
 
             if (!(e = pa_hashmap_get(c->namereg, k)))
                 break;
@@ -150,30 +152,28 @@ const char *pa_namereg_register(pa_core *c, const char *name, pa_namereg_type_t
             pa_xfree(k);
             return NULL;
         }
-        
+
         pa_xfree(n);
         n = k;
     }
-    
+
     e = pa_xnew(struct namereg_entry, 1);
     e->type = type;
     e->name = n ? n : pa_xstrdup(name);
     e->data = data;
 
-    r = pa_hashmap_put(c->namereg, e->name, e);
-    assert (r >= 0);
+    pa_assert_se(pa_hashmap_put(c->namereg, e->name, e) >= 0);
 
     return e->name;
 }
 
 void pa_namereg_unregister(pa_core *c, const char *name) {
     struct namereg_entry *e;
-    
-    assert(c);
-    assert(name);
 
-    e = pa_hashmap_remove(c->namereg, name);
-    assert(e);
+    pa_assert(c);
+    pa_assert(name);
+
+    pa_assert_se(e = pa_hashmap_remove(c->namereg, name));
 
     pa_xfree(e->name);
     pa_xfree(e);
@@ -182,27 +182,27 @@ void pa_namereg_unregister(pa_core *c, const char *name) {
 void* pa_namereg_get(pa_core *c, const char *name, pa_namereg_type_t type, int autoload) {
     struct namereg_entry *e;
     uint32_t idx;
-    assert(c);
-    
+    pa_assert(c);
+
     if (!name) {
-        
+
         if (type == PA_NAMEREG_SOURCE)
             name = pa_namereg_get_default_source_name(c);
         else if (type == PA_NAMEREG_SINK)
             name = pa_namereg_get_default_sink_name(c);
-        
+
     } else if (strcmp(name, "@DEFAULT_SINK@") == 0) {
         if (type == PA_NAMEREG_SINK)
                name = pa_namereg_get_default_sink_name(c);
-        
+
     } else if (strcmp(name, "@DEFAULT_SOURCE@") == 0) {
         if (type == PA_NAMEREG_SOURCE)
             name = pa_namereg_get_default_source_name(c);
-        
+
     } else if (strcmp(name, "@DEFAULT_MONITOR@") == 0) {
         if (type == PA_NAMEREG_SOURCE) {
             pa_sink *k;
-            
+
             if ((k = pa_namereg_get(c, NULL, PA_NAMEREG_SINK, autoload)))
                 return k->monitor_source;
         }
@@ -211,7 +211,7 @@ void* pa_namereg_get(pa_core *c, const char *name, pa_namereg_type_t type, int a
 
     if (!name)
         return NULL;
-    
+
     if (c->namereg && (e = pa_hashmap_get(c->namereg, name)))
         if (e->type == type)
             return e->data;
@@ -220,12 +220,12 @@ void* pa_namereg_get(pa_core *c, const char *name, pa_namereg_type_t type, int a
 
         if (autoload) {
             pa_autoload_request(c, name, type);
-            
+
             if (c->namereg && (e = pa_hashmap_get(c->namereg, name)))
                 if (e->type == type)
                     return e->data;
         }
-        
+
         return NULL;
     }
 
@@ -241,9 +241,9 @@ void* pa_namereg_get(pa_core *c, const char *name, pa_namereg_type_t type, int a
 
 int pa_namereg_set_default(pa_core*c, const char *name, pa_namereg_type_t type) {
     char **s;
-    
-    assert(c);
-    assert(type == PA_NAMEREG_SINK || type == PA_NAMEREG_SOURCE);
+
+    pa_assert(c);
+    pa_assert(type == PA_NAMEREG_SINK || type == PA_NAMEREG_SOURCE);
 
     s = type == PA_NAMEREG_SINK ? &c->default_sink_name : &c->default_source_name;
 
@@ -253,9 +253,9 @@ int pa_namereg_set_default(pa_core*c, const char *name, pa_namereg_type_t type)
     if (name && *s && !strcmp(name, *s))
         return 0;
 
-    if (!pa_namereg_is_valid_name(name))
+    if (!is_valid_name(name))
         return -1;
-    
+
     pa_xfree(*s);
     *s = pa_xstrdup(name);
     pa_subscription_post(c, PA_SUBSCRIPTION_EVENT_SERVER|PA_SUBSCRIPTION_EVENT_CHANGE, PA_INVALID_INDEX);
@@ -265,12 +265,12 @@ int pa_namereg_set_default(pa_core*c, const char *name, pa_namereg_type_t type)
 
 const char *pa_namereg_get_default_sink_name(pa_core *c) {
     pa_sink *s;
-    
-    assert(c);
+
+    pa_assert(c);
 
     if (c->default_sink_name)
         return c->default_sink_name;
-    
+
     if ((s = pa_idxset_first(c->sinks, NULL)))
         pa_namereg_set_default(c, s->name, PA_NAMEREG_SINK);
 
@@ -280,8 +280,8 @@ const char *pa_namereg_get_default_sink_name(pa_core *c) {
 const char *pa_namereg_get_default_source_name(pa_core *c) {
     pa_source *s;
     uint32_t idx;
-    
-    assert(c);
+
+    pa_assert(c);
 
     if (c->default_source_name)
         return c->default_source_name;