]> code.delx.au - pulseaudio/commitdiff
daemon: use pa_streq instead of plain strcmp
authorDeng Zhengrong <dzrongg@gmail.com>
Fri, 25 May 2012 02:17:51 +0000 (10:17 +0800)
committerArun Raghavan <arun.raghavan@collabora.co.uk>
Tue, 29 May 2012 03:17:40 +0000 (08:47 +0530)
Just noticed that in daemon-conf.c, it uses plain strcmp(),
while in PulseAudio, it should be better to use pa_streq().

src/daemon/daemon-conf.c

index 3ef7f812ada7645f5db06aa25c8794b778818cc4..dd2e7b6357ef3729bffb2d16fc5a5672a3aea6fe 100644 (file)
@@ -180,12 +180,12 @@ int pa_daemon_conf_set_log_target(pa_daemon_conf *c, const char *string) {
     pa_assert(c);
     pa_assert(string);
 
-    if (!strcmp(string, "auto"))
+    if (pa_streq(string, "auto"))
         c->auto_log_target = 1;
-    else if (!strcmp(string, "syslog")) {
+    else if (pa_streq(string, "syslog")) {
         c->auto_log_target = 0;
         c->log_target = PA_LOG_SYSLOG;
-    } else if (!strcmp(string, "stderr")) {
+    } else if (pa_streq(string, "stderr")) {
         c->auto_log_target = 0;
         c->log_target = PA_LOG_STDERR;
     } else if (pa_startswith(string, "file:")) {
@@ -251,11 +251,11 @@ int pa_daemon_conf_set_local_server_type(pa_daemon_conf *c, const char *string)
     pa_assert(c);
     pa_assert(string);
 
-    if (!strcmp(string, "user"))
+    if (pa_streq(string, "user"))
         c->local_server_type = PA_SERVER_TYPE_USER;
-    else if (!strcmp(string, "system")) {
+    else if (pa_streq(string, "system")) {
         c->local_server_type = PA_SERVER_TYPE_SYSTEM;
-    } else if (!strcmp(string, "none")) {
+    } else if (pa_streq(string, "none")) {
         c->local_server_type = PA_SERVER_TYPE_NONE;
     } else
         return -1;