[Asterisk-code-review] pbx: Add support for autohints. (asterisk[master])

Richard Mudgett asteriskteam at digium.com
Tue Apr 5 16:23:55 CDT 2016


Richard Mudgett has posted comments on this change.

Change subject: pbx: Add support for autohints.
......................................................................


Patch Set 4: Code-Review-1

(2 comments)

https://gerrit.asterisk.org/#/c/2499/4/main/pbx.c
File main/pbx.c:

Line 6192:         struct ast_hashtab_iter *iter;
use tabs to indent :)


PS4, Line 6483: 	/* Remove all autohints as the below iteration will recreate them */
              : 	ao2_callback(autohints, OBJ_UNLINK | OBJ_NODATA | OBJ_MULTIPLE, NULL, NULL);
Why not put this into context_table_create_autohints() as well?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I7e444c7da41b7b7d33374420fec658beeb18584e
Gerrit-PatchSet: 4
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Matthew Fredrickson <creslin at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-HasComments: Yes



More information about the asterisk-code-review mailing list