[Asterisk-code-review] bridges: Fix for Doxygen. (asterisk[18])

Friendly Automation asteriskteam at digium.com
Thu Nov 18 11:18:59 CST 2021


Friendly Automation has submitted this change. ( https://gerrit.asterisk.org/c/asterisk/+/17392 )

Change subject: bridges: Fix for Doxygen.
......................................................................

bridges: Fix for Doxygen.

ASTERISK-29743

Change-Id: I6e1bbbaa5875e19994a328ab40a5d429c6010e8b
---
M bridges/bridge_holding.c
M bridges/bridge_softmix.c
2 files changed, 3 insertions(+), 21 deletions(-)

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



diff --git a/bridges/bridge_holding.c b/bridges/bridge_holding.c
index f667f76..fa8939e 100644
--- a/bridges/bridge_holding.c
+++ b/bridges/bridge_holding.c
@@ -110,8 +110,6 @@
  * \since 12.0.0
  *
  * \param bridge_channel Channel to setup idle mode.
- *
- * \return Nothing
  */
 static void participant_idle_mode_setup(struct ast_bridge_channel *bridge_channel)
 {
@@ -410,8 +408,6 @@
  * \param bridge_channel Which channel to operate on.
  * \param payload Data to pass to the callback. (NULL if none).
  * \param payload_size Size of the payload if payload is non-NULL.  A number otherwise.
- *
- * \return Nothing
  */
 static void deferred_action(struct ast_bridge_channel *bridge_channel, const void *payload, size_t payload_size)
 {
diff --git a/bridges/bridge_softmix.c b/bridges/bridge_softmix.c
index f61732b..0165bb3 100644
--- a/bridges/bridge_softmix.c
+++ b/bridges/bridge_softmix.c
@@ -403,8 +403,6 @@
  * \since 12.0.0
  *
  * \param softmix_data Bridge mixing data.
- *
- * \return Nothing
  */
 static void softmix_poke_thread(struct softmix_bridge_data *softmix_data)
 {
@@ -615,8 +613,8 @@
  * all current participants, and the current participants need
  * to have destinations streams added for the joiner's sources.
  *
+ * \param bridge
  * \param joiner The channel that is joining the softmix bridge
- * \param participants The current participants in the softmix bridge
  */
 static void sfu_topologies_on_join(struct ast_bridge *bridge,
 	struct ast_bridge_channel *joiner)
@@ -884,8 +882,6 @@
  * \param bridge Which bridge is getting the frame
  * \param bridge_channel Which channel is writing the frame.
  * \param frame What is being written.
- *
- * \return Nothing
  */
 static void softmix_bridge_write_video(struct ast_bridge *bridge, struct ast_bridge_channel *bridge_channel, struct ast_frame *frame)
 {
@@ -937,8 +933,6 @@
  * \param bridge Which bridge is getting the frame
  * \param bridge_channel Which channel is writing the frame.
  * \param frame What is being written.
- *
- * \return Nothing
  */
 static void softmix_bridge_write_voice(struct ast_bridge *bridge, struct ast_bridge_channel *bridge_channel, struct ast_frame *frame)
 {
@@ -1037,8 +1031,6 @@
  * \since 13.21.0, 15.4.0
  *
  * \param bridge_channel Which channel's talking to clear
- *
- * \return Nothing
  */
 static void clear_talking(struct ast_bridge_channel *bridge_channel)
 {
@@ -1062,8 +1054,6 @@
  *
  * \param bridge Which bridge we are in
  * \param bridge_channel Which channel we are checking
- *
- * \return Nothing
  */
 static void softmix_bridge_check_voice(struct ast_bridge *bridge, struct ast_bridge_channel *bridge_channel)
 {
@@ -1192,8 +1182,6 @@
  * \param bridge Which bridge is getting the frame
  * \param bridge_channel Which channel is writing the frame.
  * \param frame What is being written.
- *
- * \return Nothing
  */
 static void softmix_bridge_write_text(struct ast_bridge *bridge,
 	struct ast_bridge_channel *bridge_channel, struct ast_frame *frame)
@@ -1610,8 +1598,8 @@
  * \brief Analyse mixing statistics and change bridges internal rate
  * if necessary.
  *
- * \retval 0, no changes to internal rate
- * \retval 1, internal rate was changed, update all the channels on the next mixing iteration.
+ * \retval 0 no changes to internal rate
+ * \retval 1 internal rate was changed, update all the channels on the next mixing iteration.
  */
 static unsigned int analyse_softmix_stats(struct softmix_stats *stats,
 		struct softmix_bridge_data *softmix_data, int binaural_active)
@@ -2125,8 +2113,6 @@
  * \since 12.0.0
  *
  * \param bridge Which bridge is being stopped.
- *
- * \return Nothing
  */
 static void softmix_bridge_stop(struct ast_bridge *bridge)
 {

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

Gerrit-Project: asterisk
Gerrit-Branch: 18
Gerrit-Change-Id: I6e1bbbaa5875e19994a328ab40a5d429c6010e8b
Gerrit-Change-Number: 17392
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/e1624771/attachment-0001.html>


More information about the asterisk-code-review mailing list