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

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Thu Sep 10 14:54:38 CDT 2015


Mark Michelson has submitted this change and it was merged.

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(-)

Approvals:
  Mark Michelson: Looks good to me, approved
  Anonymous Coward #1000019: Verified
  Ashley Sanders: Looks good to me, but someone else must approve
  Jonathan Rose: Looks good to me, but someone else must approve



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: merged
Gerrit-Change-Id: Ie62ff1f4b7adc1a12fa0303f53926af249b25e20
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Ashley Sanders <asanders at digium.com>
Gerrit-Reviewer: Jonathan Rose <jrose at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>



More information about the asterisk-commits mailing list