[Asterisk-code-review] Change in asterisk[master]: cel_pgsql: Fix name string for log on unable allocate memory...

Matt Jordan (Code Review) asteriskteam at digium.com
Tue Apr 14 14:05:36 CDT 2015


Matt Jordan has posted comments on this change.

Change subject: cel_pgsql: Fix name string for log on unable allocate memory for LENGTHEN_BUF1 and LENGTHEN_BUF2.
......................................................................


Patch Set 1:

(1 comment)

Since this is a bug fix, please cherry-pick the change to 11 and 13 as well.

https://gerrit.asterisk.org/#/c/107/1//COMMIT_MSG
Commit Message:

Line 7: cel_pgsql: Fix name string for log on unable allocate memory for LENGTHEN_BUF1
      : and LENGTHEN_BUF2.
Summary lines should only be 80 characters in length.

Generally, you should also be opening an ASTERISK issue for bugs that you find. That issue should be referenced in the commit message.

Having issues in the issue tracker helps to prevent things from falling through the cracks.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Icc818697d7d66d34bfe3048cdd15ca2b06c89744
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Rodrigo Ramirez Norambuena <decipher.hk at gmail.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-HasComments: Yes



More information about the asterisk-code-review mailing list