res_pjsip_mwi.c: Fix mid-line log message line breaks.
authorRichard Mudgett <rmudgett@digium.com>
Tue, 30 Jun 2015 16:11:19 +0000 (11:11 -0500)
committerRichard Mudgett <rmudgett@digium.com>
Mon, 6 Jul 2015 21:10:57 +0000 (16:10 -0500)
* Add create_mwi_subscriptions_for_endpoint() doxygen comment.

Change-Id: I3c3f921f4ec749fb65b62d2f6fa0d4d1888b94e2

res/res_pjsip_mwi.c

index 36490d9..701f6fb 100644 (file)
@@ -642,8 +642,8 @@ static struct mwi_subscription *mwi_subscribe_single(
                 * between accepting the subscription and sending the first
                 * NOTIFY...
                 */
-               ast_log(LOG_WARNING, "Unable to locate aor %s. MWI "
-                       "subscription failed.\n", name);
+               ast_log(LOG_WARNING, "Unable to locate aor %s. MWI subscription failed.\n",
+                       name);
                return NULL;
        }
 
@@ -661,7 +661,6 @@ static struct mwi_subscription *mwi_subscribe_all(
        struct mwi_subscription *sub;
 
        sub = mwi_create_subscription(endpoint, sip_sub);
-
        if (!sub) {
                return NULL;
        }
@@ -683,16 +682,15 @@ static int mwi_new_subscribe(struct ast_sip_endpoint *endpoint,
        }
 
        aor = ast_sip_location_retrieve_aor(resource);
-
        if (!aor) {
-               ast_log(LOG_WARNING, "Unable to locate aor %s. MWI "
-                       "subscription failed.\n", resource);
+               ast_log(LOG_WARNING, "Unable to locate aor %s. MWI subscription failed.\n",
+                       resource);
                return 404;
        }
 
        if (ast_strlen_zero(aor->mailboxes)) {
-               ast_log(LOG_NOTICE, "AOR %s has no configured mailboxes. "
-                       "MWI subscription failed\n", resource);
+               ast_log(LOG_NOTICE, "AOR %s has no configured mailboxes. MWI subscription failed.\n",
+                       resource);
                return 404;
        }
 
@@ -842,6 +840,7 @@ static void mwi_stasis_cb(void *userdata, struct stasis_subscription *sub,
        }
 }
 
+/*! \note Called with the unsolicited_mwi conainer lock held. */
 static int create_mwi_subscriptions_for_endpoint(void *obj, void *arg, int flags)
 {
        RAII_VAR(struct mwi_subscription *, aggregate_sub, NULL, ao2_cleanup);
@@ -982,7 +981,6 @@ static void mwi_contact_added(const void *object)
        ao2_lock(unsolicited_mwi);
 
        mwi_subs = ao2_find(unsolicited_mwi, endpoint_id, OBJ_SEARCH_KEY | OBJ_MULTIPLE | OBJ_NOLOCK | OBJ_UNLINK);
-
        if (mwi_subs) {
                for (; (mwi_sub = ao2_iterator_next(mwi_subs)); ao2_cleanup(mwi_sub)) {
                        unsubscribe(mwi_sub, NULL, 0);