From: Rodrigo Ramírez Norambuena <decipher.hk@gmail.com>
Date: Wed, 6 May 2015 10:28:28 +0000 (-0400)
Subject: cel/cel_pgsql.c: Use the 'SEP' macro when appending a column name
X-Git-Tag: 14.0.0-beta1~952^2
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=refs%2Fchanges%2F60%2F460%2F3;p=thirdparty%2Fasterisk.git

cel/cel_pgsql.c: Use the 'SEP' macro when appending a column name

When appending a column name to the sql buffer, the predicate, "if first is
non-null, use empty string; else, use comma", is identical to the 'SEP' macro
definition. Since they are the same, this patch replaces the redundant
predicate statement with the 'SEP' macro.

Change-Id: Ib8b6138b06a48381723108a05ab8752cb8700509
---

diff --git a/cel/cel_pgsql.c b/cel/cel_pgsql.c
index 2d7f0dfb04..bc76b2740d 100644
--- a/cel/cel_pgsql.c
+++ b/cel/cel_pgsql.c
@@ -201,7 +201,7 @@ static void pgsql_log(struct ast_event *event)
 		AST_RWLIST_RDLOCK(&psql_columns);
 		AST_RWLIST_TRAVERSE(&psql_columns, cur, list) {
 			LENGTHEN_BUF1(strlen(cur->name) + 2);
-			ast_str_append(&sql, 0, "%s\"%s\"", first ? "" : ",", cur->name);
+			ast_str_append(&sql, 0, "%s\"%s\"", SEP, cur->name);
 
 			if (strcmp(cur->name, "eventtime") == 0) {
 				if (strncmp(cur->type, "int", 3) == 0) {