[Asterisk-code-review] cel/cel pgsql.c: Use 'SEP' defined on append of column name ... (asterisk[master])

Rodrigo Ramirez Norambuena asteriskteam at digium.com
Wed May 13 12:00:48 CDT 2015


Rodrigo Ramirez Norambuena has uploaded a new change for review.

  https://gerrit.asterisk.org/460

Change subject: cel/cel_pgsql.c: Use 'SEP' defined on append of column name to sql setence.
......................................................................

cel/cel_pgsql.c: Use 'SEP' defined on append of column name to sql setence.

To separate of the column and values in SQL sentence is defined a 'SEP'
var, it was not used when append the column name to sql string buffer.

Change-Id: Ib8b6138b06a48381723108a05ab8752cb8700509
---
M cel/cel_pgsql.c
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/60/460/1

diff --git a/cel/cel_pgsql.c b/cel/cel_pgsql.c
index 2d7f0df..bc76b27 100644
--- a/cel/cel_pgsql.c
+++ b/cel/cel_pgsql.c
@@ -201,7 +201,7 @@
 		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) {

-- 
To view, visit https://gerrit.asterisk.org/460
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib8b6138b06a48381723108a05ab8752cb8700509
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Rodrigo Ramirez Norambuena <decipher.hk at gmail.com>



More information about the asterisk-code-review mailing list