[Asterisk-code-review] bridge_channel: Fix for Doxygen. (asterisk[16])

George Joseph asteriskteam at digium.com
Thu Nov 18 09:22:36 CST 2021


George Joseph has submitted this change. ( https://gerrit.asterisk.org/c/asterisk/+/17357 )

Change subject: bridge_channel: Fix for Doxygen.
......................................................................

bridge_channel: Fix for Doxygen.

ASTERISK-29736

Change-Id: Ia5370289e6526001a6b52754b533bcea1a9d7e5c
---
M include/asterisk/bridge_channel.h
M include/asterisk/bridge_channel_internal.h
M main/bridge_channel.c
3 files changed, 14 insertions(+), 84 deletions(-)

Approvals:
  Kevin Harwell: Looks good to me, but someone else must approve
  George Joseph: Looks good to me, approved; Approved for Submit



diff --git a/include/asterisk/bridge_channel.h b/include/asterisk/bridge_channel.h
index fb2fd4f..21090ec 100644
--- a/include/asterisk/bridge_channel.h
+++ b/include/asterisk/bridge_channel.h
@@ -33,9 +33,9 @@
  * In general, anywhere else it is unsafe to use this API. Care should be
  * taken when using this API to ensure that the locking order remains
  * correct. The locking order must be:
- *  \arg The \ref \c ast_bridge
- *  \arg The \ref \c ast_bridge_channel
- *  \arg The \ref \c ast_channel
+ *  \arg The \ref ast_bridge "\c ast_bridge"
+ *  \arg The \ref ast_bridge_channel "\c ast_bridge_channel"
+ *  \arg The \ref ast_channel "\c ast_channel"
  *
  * \author Joshua Colp <jcolp at digium.com>
  * \author Richard Mudgett <rmudgett at digium.com>
@@ -225,8 +225,6 @@
  * \brief Lock the bridge_channel.
  *
  * \param bridge_channel What to lock
- *
- * \return Nothing
  */
 #define ast_bridge_channel_lock(bridge_channel)	_ast_bridge_channel_lock(bridge_channel, __FILE__, __PRETTY_FUNCTION__, __LINE__, #bridge_channel)
 static inline void _ast_bridge_channel_lock(struct ast_bridge_channel *bridge_channel, const char *file, const char *function, int line, const char *var)
@@ -238,8 +236,6 @@
  * \brief Unlock the bridge_channel.
  *
  * \param bridge_channel What to unlock
- *
- * \return Nothing
  */
 #define ast_bridge_channel_unlock(bridge_channel)	_ast_bridge_channel_unlock(bridge_channel, __FILE__, __PRETTY_FUNCTION__, __LINE__, #bridge_channel)
 static inline void _ast_bridge_channel_unlock(struct ast_bridge_channel *bridge_channel, const char *file, const char *function, int line, const char *var)
@@ -261,8 +257,6 @@
  *
  * \note The bridge_channel->bridge pointer changes because of a
  * bridge-merge/channel-move operation between bridges.
- *
- * \return Nothing
  */
 void ast_bridge_channel_lock_bridge(struct ast_bridge_channel *bridge_channel);
 
@@ -472,8 +466,6 @@
  *     Empty if default MOH class.
  *
  * \note This is intended to be called by bridge hooks.
- *
- * \return Nothing
  */
 void ast_bridge_channel_run_app(struct ast_bridge_channel *bridge_channel, const char *app_name, const char *app_args, const char *moh_class);
 
@@ -518,8 +510,6 @@
  *
  * \param bridge_channel Which channel to play the file on
  * \param playfile Sound filename to play.
- *
- * \return Nothing
  */
 typedef void (*ast_bridge_custom_play_fn)(struct ast_bridge_channel *bridge_channel, const char *playfile);
 
@@ -535,8 +525,6 @@
  *     Empty if default MOH class.
  *
  * \note This is intended to be called by bridge hooks.
- *
- * \return Nothing
  */
 void ast_bridge_channel_playfile(struct ast_bridge_channel *bridge_channel, ast_bridge_custom_play_fn custom_play, const char *playfile, const char *moh_class);
 
@@ -606,8 +594,6 @@
  * \param payload_size Size of the payload if payload is non-NULL.  A number otherwise.
  *
  * \note The payload MUST NOT have any resources that need to be freed.
- *
- * \return Nothing
  */
 typedef void (*ast_bridge_custom_callback_fn)(struct ast_bridge_channel *bridge_channel, const void *payload, size_t payload_size);
 
@@ -685,8 +671,6 @@
  *
  * \note This is intended to be called by bridge hooks and the
  * bridge channel thread.
- *
- * \return Nothing
  */
 void ast_bridge_channel_kick(struct ast_bridge_channel *bridge_channel, int cause);
 
@@ -710,7 +694,6 @@
  *
  * \param bridge_channel Channel that received a DTMF digit.
  * \param digit DTMF digit to add to collected digits or 0 for timeout event.
- * \param clear_digits clear the digits array prior to calling hooks
  *
  * \note Neither the bridge nor the bridge_channel locks should be held
  * when entering this function.
@@ -721,8 +704,6 @@
  * \note This is intended to be called by non-DTMF bridge hooks and the bridge
  * channel thread.  Calling from a DTMF bridge hook can potentially cause
  * unbounded recursion.
- *
- * \return Nothing
  */
 void ast_bridge_channel_feature_digit(struct ast_bridge_channel *bridge_channel, int digit);
 
@@ -747,8 +728,6 @@
  * \param bridge_channel Channel to map
  *
  * \note The bridge_channel's bridge must be locked prior to calling this function.
- *
- * \return Nothing
  */
 void ast_bridge_channel_stream_map(struct ast_bridge_channel *bridge_channel);
 
diff --git a/include/asterisk/bridge_channel_internal.h b/include/asterisk/bridge_channel_internal.h
index ba71e9f..082ed1e 100644
--- a/include/asterisk/bridge_channel_internal.h
+++ b/include/asterisk/bridge_channel_internal.h
@@ -30,8 +30,8 @@
  * by other members of the Bridging API.
  *
  * See Also:
+ * \arg \ref AstBridging
  * \arg \ref AstCREDITS
- * \arg \ref Ast
  */
 
 /*!
@@ -91,8 +91,6 @@
  * \param bridge_channel Channel that owes events to the original bridge.
  *
  * \note On entry, the orig_bridge is already locked.
- *
- * \return Nothing
  */
 void bridge_channel_settle_owed_events(struct ast_bridge *orig_bridge, struct ast_bridge_channel *bridge_channel);
 
@@ -102,8 +100,6 @@
  * \since 13.17.0
  *
  * \param bridge_channel Channel that the deferred frames should be pulled from and queued to.
- *
- * \return Nothing
  */
 void bridge_channel_queue_deferred_frames(struct ast_bridge_channel *bridge_channel);
 
@@ -156,18 +152,15 @@
  * \param bridge_channel Channel to pull.
  *
  * \note On entry, bridge_channel->bridge is already locked.
- *
- * \return Nothing
  */
 void bridge_channel_internal_pull(struct ast_bridge_channel *bridge_channel);
 
 /*!
+ * \internal
  * \brief Signal imparting threads to wake up.
  * \since 13.9.0
  *
  * \param chan Channel imparted that we need to signal.
- *
- * \return Nothing
  */
 void bridge_channel_impart_signal(struct ast_channel *chan);
 
diff --git a/main/bridge_channel.c b/main/bridge_channel.c
index 608db61..71739b7 100644
--- a/main/bridge_channel.c
+++ b/main/bridge_channel.c
@@ -113,7 +113,7 @@
 static AST_RWLIST_HEAD_STATIC(sync_structs, bridge_sync);
 
 /*!
- * \brief initialize a synchronous bridge object.
+ * \brief Initialize a synchronous bridge object.
  *
  * This both initializes the structure and adds it to the list of
  * synchronization structures.
@@ -414,8 +414,6 @@
  * \param src Channel to get accountcode from.
  *
  * \note Both channels are already locked.
- *
- * \return Nothing
  */
 static void channel_fill_empty_peeraccount(struct ast_channel *dest, struct ast_channel *src)
 {
@@ -437,8 +435,6 @@
  * \param src Channel to get peeraccount from.
  *
  * \note Both channels are already locked.
- *
- * \return Nothing
  */
 static void channel_fill_empty_accountcode(struct ast_channel *dest, struct ast_channel *src)
 {
@@ -460,8 +456,6 @@
  * \param c1 Second bridge channel to update.
  *
  * \note Both channels are already locked.
- *
- * \return Nothing
  */
 static void channel_set_empty_accountcodes(struct ast_channel *c0, struct ast_channel *c1)
 {
@@ -483,8 +477,6 @@
  * \param src Channel to get accountcode from.
  *
  * \note Both channels are already locked.
- *
- * \return Nothing
  */
 static void channel_update_peeraccount(struct ast_channel *dest, struct ast_channel *src)
 {
@@ -505,8 +497,6 @@
  * \param c1 Second channel to update.
  *
  * \note Both channels are already locked.
- *
- * \return Nothing
  */
 static void channel_update_peeraccounts(struct ast_channel *c0, struct ast_channel *c1)
 {
@@ -523,8 +513,6 @@
  * \param swap Channel being replaced by the joining channel.  May be NULL.
  *
  * \note The bridge must be locked prior to calling this function.
- *
- * \return Nothing
  */
 static void bridge_channel_update_accountcodes_joining(struct ast_bridge_channel *joining, struct ast_bridge_channel *swap)
 {
@@ -570,8 +558,6 @@
  * \param leaving Channel leaving the bridge. (Has already been removed actually)
  *
  * \note The bridge must be locked prior to calling this function.
- *
- * \return Nothing
  */
 static void bridge_channel_update_accountcodes_leaving(struct ast_bridge_channel *leaving)
 {
@@ -780,8 +766,6 @@
  * \param bridge_channel Channel that owes events to the bridge.
  *
  * \note On entry, the bridge_channel->bridge is already locked.
- *
- * \return Nothing
  */
 static void bridge_channel_cancel_owed_events(struct ast_bridge_channel *bridge_channel)
 {
@@ -848,8 +832,6 @@
  * \param bridge_channel Channel to suspend.
  *
  * \note This function assumes bridge_channel->bridge is locked.
- *
- * \return Nothing
  */
 void bridge_channel_internal_suspend_nolock(struct ast_bridge_channel *bridge_channel)
 {
@@ -869,8 +851,6 @@
  * \brief Suspend a channel from a bridge.
  *
  * \param bridge_channel Channel to suspend.
- *
- * \return Nothing
  */
 static void bridge_channel_suspend(struct ast_bridge_channel *bridge_channel)
 {
@@ -886,8 +866,6 @@
  * \param bridge_channel Channel to unsuspend.
  *
  * \note This function assumes bridge_channel->bridge is locked.
- *
- * \return Nothing
  */
 void bridge_channel_internal_unsuspend_nolock(struct ast_bridge_channel *bridge_channel)
 {
@@ -912,8 +890,6 @@
  * \brief Unsuspend a channel from a bridge.
  *
  * \param bridge_channel Channel to unsuspend.
- *
- * \return Nothing
  */
 static void bridge_channel_unsuspend(struct ast_bridge_channel *bridge_channel)
 {
@@ -1268,8 +1244,6 @@
  *
  * \param bridge_channel Which channel to run the application on.
  * \param data Action frame data to run the application.
- *
- * \return Nothing
  */
 static void bridge_channel_run_app(struct ast_bridge_channel *bridge_channel, struct bridge_run_app *data)
 {
@@ -1362,8 +1336,6 @@
  *
  * \param bridge_channel Which channel to play a file on.
  * \param payload Action frame payload to play a file.
- *
- * \return Nothing
  */
 static void bridge_channel_playfile(struct ast_bridge_channel *bridge_channel, struct bridge_playfile *payload)
 {
@@ -1434,8 +1406,6 @@
  *
  * \param bridge_channel Which channel to call the callback on.
  * \param data Action frame data to call the callback.
- *
- * \return Nothing
  */
 static void bridge_channel_do_callback(struct ast_bridge_channel *bridge_channel, struct bridge_custom_callback *data)
 {
@@ -1564,8 +1534,6 @@
  * \since 12.0.0
  *
  * \param bridge_channel Channel to run expired intervals on.
- *
- * \return Nothing
  */
 static void bridge_channel_handle_interval(struct ast_bridge_channel *bridge_channel)
 {
@@ -1877,8 +1845,6 @@
  * \since 12.8.0
  *
  * \param bridge_channel Channel to check expired interdigit timer on.
- *
- * \return Nothing
  */
 static void bridge_channel_handle_feature_timeout(struct ast_bridge_channel *bridge_channel)
 {
@@ -1922,7 +1888,10 @@
 	ao2_iterator_destroy(&iter);
 }
 
-/*! \brief Internal function that plays back DTMF on a bridge channel */
+/*!
+ * \internal
+ * \brief Play back DTMF on a bridge channel
+ */
 static void bridge_channel_dtmf_stream(struct ast_bridge_channel *bridge_channel, const char *dtmf)
 {
 	ast_debug(1, "Playing DTMF stream '%s' out to %p(%s)\n",
@@ -2061,8 +2030,6 @@
  * \param bridge_channel Channel to execute the action on.
  * \param action What to do.
  * \param data data from the action.
- *
- * \return Nothing
  */
 static void bridge_channel_handle_action(struct ast_bridge_channel *bridge_channel,
 	enum bridge_channel_action_type action, void *data)
@@ -2130,8 +2097,6 @@
  * \param bridge_channel Channel causing the check.
  *
  * \note On entry, bridge_channel->bridge is already locked.
- *
- * \return Nothing
  */
 static void bridge_channel_dissolve_check(struct ast_bridge_channel *bridge_channel)
 {
@@ -2324,8 +2289,6 @@
  *
  * \param bridge_channel Channel to execute the control frame action on.
  * \param fr Control frame to handle.
- *
- * \return Nothing
  */
 static void bridge_channel_handle_control(struct ast_bridge_channel *bridge_channel, struct ast_frame *fr)
 {
@@ -2403,8 +2366,6 @@
  *
  * \param chan Channel to send text to
  * \param f The frame containing the text data to send
- *
- * \return Nothing
  */
 static void sendtext_safe(struct ast_channel *chan, const struct ast_frame *f)
 {
@@ -2438,8 +2399,6 @@
  * \since 12.0.0
  *
  * \param bridge_channel Channel to write outgoing frame.
- *
- * \return Nothing
  */
 static void bridge_channel_handle_write(struct ast_bridge_channel *bridge_channel)
 {
@@ -2544,7 +2503,10 @@
 	bridge_frame_free(fr);
 }
 
-/*! \brief Internal function to handle DTMF from a channel */
+/*!
+ * \internal
+ * \brief Handle DTMF from a channel
+ */
 static struct ast_frame *bridge_handle_dtmf(struct ast_bridge_channel *bridge_channel, struct ast_frame *frame)
 {
 	struct ast_bridge_features *features = bridge_channel->features;
@@ -2802,8 +2764,6 @@
  * \param bridge_channel Channel to wait.
  *
  * \note Each channel does writing/reading in their own thread.
- *
- * \return Nothing
  */
 static void bridge_channel_wait(struct ast_bridge_channel *bridge_channel)
 {
@@ -2866,8 +2826,6 @@
  *
  * \param bridge_channel Which channel is involved.
  * \param type Specified join/leave event.
- *
- * \return Nothing
  */
 static void bridge_channel_event_join_leave(struct ast_bridge_channel *bridge_channel, enum ast_bridge_hook_type type)
 {

-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/17357
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: Ia5370289e6526001a6b52754b533bcea1a9d7e5c
Gerrit-Change-Number: 17357
Gerrit-PatchSet: 2
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20211118/d9fde5ac/attachment-0001.html>


More information about the asterisk-code-review mailing list