[Asterisk-code-review] res/ari: Register Stasis application on WebSocket attempt (asterisk[13])

Joshua Colp asteriskteam at digium.com
Thu May 21 05:29:21 CDT 2015


Joshua Colp has posted comments on this change.

Change subject: res/ari: Register Stasis application on WebSocket attempt
......................................................................


Patch Set 3:

(1 comment)

Excessively minor comment, not even to make me even do a -1. :P

https://gerrit.asterisk.org/#/c/486/3/res/stasis/app.c
File res/stasis/app.c:

Line 874: 	if (data) {
        : 		app->data = ao2_bump(data);
        : 	}
app->data = ao2_bump(data);

It does the if internally.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia5dc60dc2b6bee76cd5aff0f69dd53b36e83f636
Gerrit-PatchSet: 3
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-HasComments: Yes



More information about the asterisk-code-review mailing list