[Asterisk-code-review] res pjsip pubsub.c: Add some notification comments. (asterisk[13])

Richard Mudgett asteriskteam at digium.com
Thu Sep 10 13:18:43 CDT 2015


Richard Mudgett has uploaded a new change for review.

  https://gerrit.asterisk.org/1244

Change subject: res_pjsip_pubsub.c: Add some notification comments.
......................................................................

res_pjsip_pubsub.c: Add some notification comments.

Change-Id: Ie62ff1f4b7adc1a12fa0303f53926af249b25e20
---
M res/res_pjsip_pubsub.c
1 file changed, 2 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/44/1244/1

diff --git a/res/res_pjsip_pubsub.c b/res/res_pjsip_pubsub.c
index 194107f..c28446e 100644
--- a/res/res_pjsip_pubsub.c
+++ b/res/res_pjsip_pubsub.c
@@ -1171,6 +1171,7 @@
 		return;
 	}
 
+	/* We notify subscription shutdown only on the tree leaves. */
 	if (sub->handler->subscription_shutdown) {
 		sub->handler->subscription_shutdown(sub);
 	}
@@ -2523,6 +2524,7 @@
 		return 0;
 	}
 
+	/* We notify subscription establishment only on the tree leaves. */
 	if (sub->handler->notifier->subscription_established(sub)) {
 		return -1;
 	}

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie62ff1f4b7adc1a12fa0303f53926af249b25e20
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>



More information about the asterisk-code-review mailing list