[Asterisk-code-review] cel/cel pgsql.c: Use the 'SEP' macro when appending a column... (asterisk[master])

Ashley Sanders asteriskteam at digium.com
Wed May 13 15:49:37 CDT 2015


Hello Kevin Harwell, Jonathan Rose,

I'd like you to reexamine a change.  Please visit

    https://gerrit.asterisk.org/460

to look at the new patch set (#3).

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

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
---
M cel/cel_pgsql.c
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/60/460/3
-- 
To view, visit https://gerrit.asterisk.org/460
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib8b6138b06a48381723108a05ab8752cb8700509
Gerrit-PatchSet: 3
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Rodrigo Ramirez Norambuena <decipher.hk at gmail.com>
Gerrit-Reviewer: Ashley Sanders <asanders at digium.com>
Gerrit-Reviewer: Jonathan Rose <jrose at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>



More information about the asterisk-code-review mailing list