[Asterisk-code-review] followme: initialize all config items on reload (asterisk[11])

Richard Mudgett asteriskteam at digium.com
Thu Aug 18 11:38:44 CDT 2016


Richard Mudgett has posted comments on this change.

Change subject: followme: initialize all config items on reload
......................................................................


Patch Set 1: Code-Review-1

(1 comment)

https://gerrit.asterisk.org/#/c/3581/1/apps/app_followme.c
File apps/app_followme.c:

PS1, Line 306: 	f->moh[0] = '\0';
             : 	f->context[0] = '\0';
These are not handled in the same manner as the master version.  Context in particular does not get cleared as the default value in init_profile().


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ief829e16374ad1e0ecfd63e6ee4923b5a1d1c150
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 11
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Tzafrir Cohen <tzafrir.cohen at xorcom.com>
Gerrit-HasComments: Yes



More information about the asterisk-code-review mailing list