[Asterisk-code-review] apps: Fix for Doxygen. (asterisk[master])

Alexander Traud asteriskteam at digium.com
Mon Nov 15 07:24:24 CST 2021


Alexander Traud has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/17363 )


Change subject: apps: Fix for Doxygen.
......................................................................

apps: Fix for Doxygen.

ASTERISK-29740

Change-Id: Icb6fbcfea0a5f1c82caa5001902b6a786adbf307
---
M apps/app_agent_pool.c
M apps/app_alarmreceiver.c
M apps/app_bridgewait.c
M apps/app_chanspy.c
M apps/app_confbridge.c
M apps/app_dial.c
M apps/app_jack.c
M apps/app_meetme.c
M apps/app_minivm.c
M apps/app_mixmonitor.c
M apps/app_page.c
M apps/app_queue.c
M apps/app_voicemail.c
M apps/confbridge/conf_state.c
M apps/confbridge/include/confbridge.h
15 files changed, 67 insertions(+), 117 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/63/17363/1

diff --git a/apps/app_agent_pool.c b/apps/app_agent_pool.c
index e0df4a4..91dfe9f 100644
--- a/apps/app_agent_pool.c
+++ b/apps/app_agent_pool.c
@@ -24,8 +24,14 @@
  *
  * See Also:
  * \arg \ref AstCREDITS
- * \arg \ref Config_agent
+ * \arg \ref agents.conf "Config_agent"
  */
+
+/*!
+ * \page agents.conf agents.conf
+ * \verbinclude agents.conf.sample
+ */
+
 /*** MODULEINFO
 	<support_level>core</support_level>
  ***/
@@ -646,8 +652,6 @@
  * \brief Lock the agent.
  *
  * \param agent Agent to lock
- *
- * \return Nothing
  */
 #define agent_lock(agent)	_agent_lock(agent, __FILE__, __PRETTY_FUNCTION__, __LINE__, #agent)
 static inline void _agent_lock(struct agent_pvt *agent, const char *file, const char *function, int line, const char *var)
@@ -659,8 +663,6 @@
  * \brief Unlock the agent.
  *
  * \param agent Agent to unlock
- *
- * \return Nothing
  */
 #define agent_unlock(agent)	_agent_unlock(agent, __FILE__, __PRETTY_FUNCTION__, __LINE__, #agent)
 static inline void _agent_unlock(struct agent_pvt *agent, const char *file, const char *function, int line, const char *var)
@@ -678,8 +680,6 @@
  * \note Assumes the agent lock is already obtained.
  *
  * \note Defined locking order is channel lock then agent lock.
- *
- * \return Nothing
  */
 static struct ast_channel *agent_lock_logged(struct agent_pvt *agent)
 {
@@ -744,8 +744,6 @@
  * \since 12.0.0
  *
  * \param agent_id Which agent needs the device state updated.
- *
- * \return Nothing
  */
 static void agent_devstate_changed(const char *agent_id)
 {
@@ -1005,8 +1003,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
  */
 static void clear_agent_status(struct ast_bridge_channel *bridge_channel, const void *payload, size_t payload_size)
 {
@@ -1022,8 +1018,6 @@
  * \param agent Which agent is connecting to the caller.
  *
  * \note The agent is locked on entry and not locked on exit.
- *
- * \return Nothing
  */
 static void agent_connect_caller(struct ast_bridge_channel *bridge_channel, struct agent_pvt *agent)
 {
@@ -1357,8 +1351,6 @@
  * bridge_channel->bridge_pvt.
  *
  * \note On entry, self is already locked.
- *
- * \return Nothing
  */
 static void bridge_agent_hold_pull(struct ast_bridge *self, struct ast_bridge_channel *bridge_channel)
 {
@@ -1376,8 +1368,6 @@
  * references to the bridge so it can be destroyed.
  *
  * \note On entry, self must NOT be locked.
- *
- * \return Nothing
  */
 static void bridge_agent_hold_dissolving(struct ast_bridge *self)
 {
@@ -1469,8 +1459,6 @@
  * \param agent Which agent logging out.
  *
  * \note On entry agent is already locked.  On exit it is no longer locked.
- *
- * \return Nothing
  */
 static void agent_logout(struct agent_pvt *agent)
 {
@@ -1508,8 +1496,6 @@
  *
  * \param agent Which agent.
  * \param logged The logged in channel.
- *
- * \return Nothing
  */
 static void agent_run(struct agent_pvt *agent, struct ast_channel *logged)
 {
@@ -2028,8 +2014,6 @@
  *
  * \param agent What to setup channel config values on.
  * \param chan Channel logging in as an agent.
- *
- * \return Nothing
  */
 static void agent_login_channel_config(struct agent_pvt *agent, struct ast_channel *chan)
 {
diff --git a/apps/app_alarmreceiver.c b/apps/app_alarmreceiver.c
index 07885d2..cfc76e6 100644
--- a/apps/app_alarmreceiver.c
+++ b/apps/app_alarmreceiver.c
@@ -203,7 +203,6 @@
  * family, then create it and set its value to 1.
  *
  * \param key A database key to increment
- * \return Nothing
  */
 static void database_increment(char *key)
 {
diff --git a/apps/app_bridgewait.c b/apps/app_bridgewait.c
index b17cddf..9326ed1 100644
--- a/apps/app_bridgewait.c
+++ b/apps/app_bridgewait.c
@@ -391,17 +391,6 @@
 	ao2_cleanup(wrapper);
 }
 
-/*!
- * \internal
- * \since 12.0.0
- * \brief Application callback for the bridgewait application
- *
- * \param chan channel running the application
- * \param data Arguments to the application
- *
- * \retval 0 Ran successfully and the call didn't hang up
- * \retval -1 Failed or the call was hung up by the time the channel exited the holding bridge
- */
 static enum wait_bridge_roles validate_role(const char *role)
 {
 	if (!strcmp(role, "participant")) {
@@ -413,6 +402,17 @@
 	}
 }
 
+/*!
+ * \internal
+ * \since 12.0.0
+ * \brief Application callback for the bridgewait application
+ *
+ * \param chan channel running the application
+ * \param data Arguments to the application
+ *
+ * \retval 0 Ran successfully and the call didn't hang up
+ * \retval -1 Failed or the call was hung up by the time the channel exited the holding bridge
+ */
 static int bridgewait_exec(struct ast_channel *chan, const char *data)
 {
 	char *bridge_name = DEFAULT_BRIDGE_NAME;
diff --git a/apps/app_chanspy.c b/apps/app_chanspy.c
index e02f25d..0bfceb9 100644
--- a/apps/app_chanspy.c
+++ b/apps/app_chanspy.c
@@ -559,7 +559,7 @@
  * \brief Publish the chanspy message over Stasis-Core
  * \param spyer The channel doing the spying
  * \param spyee Who is being spied upon
- * \start start If non-zero, the spying is starting. Otherwise, the spyer is
+ * \param start If non-zero, the spying is starting. Otherwise, the spyer is
  * finishing
  */
 static void publish_chanspy_message(struct ast_channel *spyer,
diff --git a/apps/app_confbridge.c b/apps/app_confbridge.c
index 4290521..9de1c3f 100644
--- a/apps/app_confbridge.c
+++ b/apps/app_confbridge.c
@@ -797,8 +797,8 @@
  *
  * \note Must be called with the conference locked
  *
- * \retval 1, conference is recording.
- * \retval 0, conference is NOT recording.
+ * \retval 1 conference is recording.
+ * \retval 0 conference is NOT recording.
  */
 static int conf_is_recording(struct confbridge_conference *conference)
 {
@@ -980,7 +980,8 @@
  * \param bridge_channel The bridged channel involved
  *
  * \note if caller is NULL, the announcment will be sent to all participants in the conference.
- * \return Returns 0 on success, -1 if the user hung up
+ * \retval 0 on success.
+ * \retval -1 if the user hung up.
  */
 static int announce_user_count(struct confbridge_conference *conference, struct confbridge_user *user,
 			       struct ast_bridge_channel *bridge_channel)
@@ -1030,7 +1031,9 @@
  * \param user User to play audio prompt to
  * \param filename Prompt to play
  *
- * \return Returns 0 on success, -1 if the user hung up
+ * \retval 0 on success.
+ * \retval -1 if the user hung up.
+ *
  * \note Generally this should be called when the conference is unlocked to avoid blocking
  * the entire conference while the sound is played. But don't unlock the conference bridge
  * in the middle of a state transition.
@@ -1167,8 +1170,6 @@
  * \brief Destroy a conference bridge
  *
  * \param obj The conference bridge object
- *
- * \return Returns nothing
  */
 static void destroy_conference_bridge(void *obj)
 {
@@ -1389,8 +1390,6 @@
  * \brief Unsuspend MOH for the conference user.
  *
  * \param user Conference user to unsuspend MOH on.
- *
- * \return Nothing
  */
 static void conf_moh_unsuspend(struct confbridge_user *user)
 {
@@ -1406,8 +1405,6 @@
  * \brief Suspend MOH for the conference user.
  *
  * \param user Conference user to suspend MOH on.
- *
- * \return Nothing
  */
 static void conf_moh_suspend(struct confbridge_user *user)
 {
@@ -4355,8 +4352,6 @@
  * \since 12.0.0
  *
  * \param tech What to unregister.
- *
- * \return Nothing
  */
 static void unregister_channel_tech(struct ast_channel_tech *tech)
 {
diff --git a/apps/app_dial.c b/apps/app_dial.c
index 51f5f45..82f6d22 100644
--- a/apps/app_dial.c
+++ b/apps/app_dial.c
@@ -1167,8 +1167,6 @@
  * \param chan Channel to get connected line updated.
  * \param peer Channel providing connected line information.
  * \param is_caller Non-zero if chan is the calling channel.
- *
- * \return Nothing
  */
 static void update_connected_line_from_peer(struct ast_channel *chan, struct ast_channel *peer, int is_caller)
 {
@@ -2210,8 +2208,6 @@
  * \param peer Peer channel for bridge.
  * \param opts Dialing option flags.
  * \param opt_args Dialing option argument strings.
- *
- * \return Nothing
  */
 static void setup_peer_after_bridge_goto(struct ast_channel *chan, struct ast_channel *peer, struct ast_flags64 *opts, char *opt_args[])
 {
diff --git a/apps/app_jack.c b/apps/app_jack.c
index 1609972..1690521 100644
--- a/apps/app_jack.c
+++ b/apps/app_jack.c
@@ -615,8 +615,6 @@
  * Read data from the input ringbuffer, which is the properly resampled audio
  * that was read from the jack input port.  Write it to the channel in 20 ms frames,
  * or fill up an output frame instead if one is provided.
- *
- * \return Nothing.
  */
 static void handle_jack_audio(struct ast_channel *chan, struct jack_data *jack_data,
 	struct ast_frame *out_frame)
diff --git a/apps/app_meetme.c b/apps/app_meetme.c
index 27d985b..17d42a3 100644
--- a/apps/app_meetme.c
+++ b/apps/app_meetme.c
@@ -1336,7 +1336,7 @@
  * \brief Generate a stasis message associated with a meetme event
  * \since 12.0.0
  *
- * \param meetme_confere The conference responsible for generating this message
+ * \param meetme_conference The conference responsible for generating this message
  * \param chan The channel involved in the message (NULL allowed)
  * \param user The conference user involved in the message (NULL allowed)
  * \param message_type the type the stasis message being generated
@@ -2963,7 +2963,8 @@
  * \param confflags flags used by conf for various options
  * \param chan ast_channel belonging to the user who called the menu
  * \param user which meetme conference user invoked the menu
- * \param recordingtmp character buffer which may hold the name of the conference recording file
+ * \param recordingtmp, recordingtmp_size character buffer which may hold the name of the conference recording file
+ * \param cap_slin
  */
 static void meetme_menu_admin_extended(enum menu_modes *menu_mode, int *dtmf,
 	struct ast_conference *conf, struct ast_flags64 *confflags, struct ast_channel *chan,
@@ -3150,7 +3151,8 @@
  * \param confflags flags used by conf for various options
  * \param chan ast_channel belonging to the user who called the menu
  * \param user which meetme conference user invoked the menu
- * \param recordingtmp character buffer which may hold the name of the conference recording file
+ * \param recordingtmp,recordingtmp_size character buffer which may hold the name of the conference recording file
+ * \param cap_slin
  */
 static void meetme_menu(enum menu_modes *menu_mode, int *dtmf,
 	struct ast_conference *conf, struct ast_flags64 *confflags, struct ast_channel *chan,
diff --git a/apps/app_minivm.c b/apps/app_minivm.c
index 2d26e78..29a872a 100644
--- a/apps/app_minivm.c
+++ b/apps/app_minivm.c
@@ -26,7 +26,7 @@
  * based on the Comedian Mail voicemail system (app_voicemail.c).
  *
  * \par See also
- * \arg \ref Config_minivm_examples
+ * \arg \ref minivm.conf "Config_minivm"
  * \arg \ref App_minivm
  *
  * \ingroup applications
@@ -78,16 +78,16 @@
  *		- English, GB		en_gb
  *
  * \par See also
- * \arg \ref Config_minivm
+ * \arg \ref minivm.conf "Config_minivm"
  * \arg \ref Config_minivm_examples
  * \arg \ref Minivm_directories
  * \arg \ref app_minivm.c
  * \arg Comedian mail: app_voicemail.c
- * \arg \ref descrip_minivm_accmess
- * \arg \ref descrip_minivm_greet
- * \arg \ref descrip_minivm_record
- * \arg \ref descrip_minivm_delete
- * \arg \ref descrip_minivm_notify
+ * \arg \ref minivm_accmess_exec
+ * \arg \ref minivm_greet_exec
+ * \arg \ref minivm_record_exec
+ * \arg \ref minivm_delete_exec
+ * \arg \ref minivm_notify_exec
  *
  * \arg \ref App_minivm_todo
  */
@@ -113,6 +113,13 @@
  * Back: \ref App_minivm
  */
 
+/*!
+ * \page minivm.conf minivm.conf
+ * \verbinclude minivm.conf.sample
+ *
+ * Back: \ref App_minivm
+ */
+
 /*! \page Config_minivm_examples Example dialplan for Mini-Voicemail
  * \section Example dialplan scripts for Mini-Voicemail
  *  \verbinclude extensions_minivm.conf.sample
diff --git a/apps/app_mixmonitor.c b/apps/app_mixmonitor.c
index 374c567..a0eb1db 100644
--- a/apps/app_mixmonitor.c
+++ b/apps/app_mixmonitor.c
@@ -631,6 +631,7 @@
  * \brief Copies the mixmonitor to all voicemail recipients
  * \param mixmonitor The mixmonitor that needs to forward its file to recipients
  * \param ext Format of the file that was saved
+ * \param filename
  */
 static void copy_to_voicemail(struct mixmonitor *mixmonitor, const char *ext, const char *filename)
 {
diff --git a/apps/app_page.c b/apps/app_page.c
index 7f498ee..c4a64a2 100644
--- a/apps/app_page.c
+++ b/apps/app_page.c
@@ -174,8 +174,6 @@
  *
  * \param chan Setup bridge profile on this channel.
  * \param options Options to setup bridge profile.
- *
- * \return Nothing
  */
 static void setup_profile_bridge(struct ast_channel *chan, struct page_options *options)
 {
@@ -192,8 +190,6 @@
  *
  * \param chan Setup user profile on this channel.
  * \param options Options to setup paged user profile.
- *
- * \return Nothing
  */
 static void setup_profile_paged(struct ast_channel *chan, struct page_options *options)
 {
@@ -216,8 +212,6 @@
  *
  * \param chan Setup user profile on this channel.
  * \param options Options to setup caller user profile.
- *
- * \return Nothing
  */
 static void setup_profile_caller(struct ast_channel *chan, struct page_options *options)
 {
diff --git a/apps/app_queue.c b/apps/app_queue.c
index cfe49c3..2821946 100644
--- a/apps/app_queue.c
+++ b/apps/app_queue.c
@@ -1661,7 +1661,7 @@
 #define ANNOUNCEPOSITION_YES 1 /*!< We announce position */
 #define ANNOUNCEPOSITION_NO 2 /*!< We don't announce position */
 #define ANNOUNCEPOSITION_MORE_THAN 3 /*!< We say "Currently there are more than <limit>" */
-#define ANNOUNCEPOSITION_LIMIT 4 /*!< We not announce position more than <limit> */
+#define ANNOUNCEPOSITION_LIMIT 4 /*!< We not announce position more than \<limit\> */
 
 struct call_queue {
 	AST_DECLARE_STRING_FIELDS(
@@ -1875,6 +1875,7 @@
  * \brief ao2_callback, Decreases queuepos of all followers with a queuepos greater than arg.
  * \param obj the member being acted on
  * \param arg pointer to an integer containing the position value that was removed and requires reduction for anything above
+ * \param flag unused
  */
 static int queue_member_decrement_followers(void *obj, void *arg, int flag)
 {
@@ -1893,6 +1894,7 @@
  *        on them. This callback should always be ran before performing mass unlinking of delmarked members from queues.
  * \param obj member being acted on
  * \param arg pointer to the queue members are being removed from
+ * \param flag unused
  */
 static int queue_delme_members_decrement_followers(void *obj, void *arg, int flag)
 {
@@ -2238,8 +2240,6 @@
  * \param blob The information being published.
  *
  * \note The json blob reference is passed to this function.
- *
- * \return Nothing
  */
 static void queue_publish_member_blob(struct stasis_message_type *type, struct ast_json *blob)
 {
@@ -2643,7 +2643,8 @@
  *
  * This function recrusively checks if the context child is included in the context parent.
  *
- * \return 1 if child is included in parent, 0 if not
+ * \retval 1 if child is included in parent
+ * \retval 0 if not
  */
 static int context_included(const char *parent, const char *child);
 static int context_included(const char *parent, const char *child)
@@ -3428,7 +3429,7 @@
  * \brief If removing a single member from a queue, use this function instead of ao2_unlinking.
  *        This will perform round robin queue position reordering for the remaining members.
  * \param queue Which queue the member is being removed from
- * \param member Which member is being removed from the queue
+ * \param mem Which member is being removed from the queue
  */
 static void member_remove_from_queue(struct call_queue *queue, struct member *mem)
 {
@@ -4279,8 +4280,6 @@
  * \since 1.8
  *
  * \param doomed callattempt structure to destroy.
- *
- * \return Nothing
  */
 static void callattempt_free(struct callattempt *doomed)
 {
@@ -4957,8 +4956,6 @@
  * \param chan Channel to get connected line updated.
  * \param peer Channel providing connected line information.
  * \param is_caller Non-zero if chan is the calling channel.
- *
- * \return Nothing
  */
 static void update_connected_line_from_peer(struct ast_channel *chan, struct ast_channel *peer, int is_caller)
 {
@@ -5759,7 +5756,7 @@
 
 /*!
  * \brief update the queue status
- * \retval Always 0
+ * \retval 0 always
 */
 static int update_queue(struct call_queue *q, struct member *member, int callcompletedinsl, time_t starttime)
 {
@@ -6163,7 +6160,6 @@
  *
  * \param userdata Data pertaining to the particular call in the queue.
  * \param sub The stasis subscription on which the message occurred.
- * \param topic The topic for this event.
  * \param msg The stasis message for the bridge enter event
  */
 static void handle_bridge_enter(void *userdata, struct stasis_subscription *sub,
@@ -6197,7 +6193,6 @@
  *
  * \param userdata Data pertaining to the particular call in the queue.
  * \param sub The stasis subscription on which the message occurred.
- * \param topic The topic for this event.
  * \param msg The stasis message for the blind transfer event
  */
 static void handle_blind_transfer(void *userdata, struct stasis_subscription *sub,
@@ -6257,7 +6252,6 @@
  *
  * \param userdata Data pertaining to the particular call in the queue.
  * \param sub The stasis subscription on which the message occurred.
- * \param topic The topic for this event.
  * \param msg The stasis message for the attended transfer event.
  */
 static void handle_attended_transfer(void *userdata, struct stasis_subscription *sub,
@@ -6332,7 +6326,6 @@
  *
  * \param userdata Data pertaining to the particular call in the queue.
  * \param sub The stasis subscription on which the message occurred.
- * \param topic The topic for this event.
  * \param msg The stasis message for the local optimization begin event
  */
 static void handle_local_optimization_begin(void *userdata, struct stasis_subscription *sub,
@@ -6384,7 +6377,6 @@
  *
  * \param userdata Data pertaining to the particular call in the queue.
  * \param sub The stasis subscription on which the message occurred.
- * \param topic The topic for this event.
  * \param msg The stasis message for the local optimization end event
  */
 static void handle_local_optimization_end(void *userdata, struct stasis_subscription *sub,
@@ -6449,7 +6441,6 @@
  *
  * \param userdata Data pertaining to the particular call in the queue.
  * \param sub The stasis subscription on which the message occurred.
- * \param topic The topic for this event.
  * \param msg The stasis message for the hangup event.
  */
 static void handle_hangup(void *userdata, struct stasis_subscription *sub,
@@ -6651,8 +6642,6 @@
  * \param peer Peer channel for bridge.
  * \param opts Dialing option flags.
  * \param opt_args Dialing option argument strings.
- *
- * \return Nothing
  */
 static void setup_peer_after_bridge_goto(struct ast_channel *chan, struct ast_channel *peer, struct ast_flags *opts, char *opt_args[])
 {
@@ -7538,8 +7527,6 @@
  * \param paused Set to 1 if the member is being paused or 0 to unpause.
  *
  * \pre The q is locked on entry.
- *
- * \return Nothing
  */
 static void set_queue_member_pause(struct call_queue *q, struct member *mem, const char *reason, int paused)
 {
@@ -7677,8 +7664,6 @@
  * \param ringinuse Set to 1 if the member is called when inuse.
  *
  * \pre The q is locked on entry.
- *
- * \return Nothing
  */
 static void set_queue_member_ringinuse(struct call_queue *q, struct member *mem, int ringinuse)
 {
@@ -7730,7 +7715,7 @@
  * \param[in] queuename If specified, only act on a member if it belongs to this queue
  * \param[in] interface Interface of queue member(s) having priority set.
  * \param[in] property Which queue property is being set
- * \param[in] penalty Value penalty is being changed to for each member
+ * \param[in] value Value penalty is being changed to for each member
  */
 static int set_member_value(const char *queuename, const char *interface, int property, int value)
 {
@@ -9443,7 +9428,6 @@
  * \param cfg The configuration which we are reading
  * \param mask Tells us what information we need to reload
  * \param queuename The name of the queue we are reloading information from
- * \retval void
  */
 static void reload_single_queue(struct ast_config *cfg, struct ast_flags *mask, const char *queuename)
 {
@@ -9657,7 +9641,7 @@
  * \param queuename The name of the queue to reset the statistics
  * for. If this is NULL or zero-length, then this means to reset
  * the statistics for all queues
- * \retval void
+ * \retval 0 always
  */
 static int clear_stats(const char *queuename)
 {
@@ -9933,8 +9917,8 @@
  * list (followed immediately by a \0, not a space) since it is used for
  * checking tab-completion and a word at the end is still being tab-completed.
  *
- * \return Returns 1 if the word is found
- * \return Returns 0 if the word is not found
+ * \retval 1 if the word is found
+ * \retval 0 if the word is not found
 */
 static int word_in_list(const char *list, const char *word) {
 	int list_len, word_len = strlen(word);
@@ -11549,7 +11533,7 @@
 }
 
 /* \brief Find a member by looking up queuename and interface.
- * \return Returns a member or NULL if member not found.
+ * \return member or NULL if member not found.
 */
 static struct member *find_member_by_queuename_and_interface(const char *queuename, const char *interface)
 {
diff --git a/apps/app_voicemail.c b/apps/app_voicemail.c
index 39bfd93..eb3ad94 100644
--- a/apps/app_voicemail.c
+++ b/apps/app_voicemail.c
@@ -26,7 +26,7 @@
  *         (http://www.washington.edu/imap/)
  *
  * \par See also
- * \arg \ref Config_vm
+ * \arg \ref voicemail.conf "Config_voicemail"
  * \note For information about voicemail IMAP storage, https://wiki.asterisk.org/wiki/display/AST/IMAP+Voicemail+Storage
  * \ingroup applications
  * \todo This module requires res_adsi to load. This needs to be optional
@@ -1091,7 +1091,7 @@
  * \brief Parse the given mailbox_id into mailbox and context.
  * \since 12.0.0
  *
- * \param mailbox_id The mailbox at context string to separate.
+ * \param mailbox_id The mailbox\@context string to separate.
  * \param mailbox Where the mailbox part will start.
  * \param context Where the context part will start.  ("default" if not present)
  *
@@ -2850,9 +2850,9 @@
 	return 0;
 }
 
-/**
+/*!
  * \brief Determines if the given folder has messages.
- * \param mailbox The @ delimited string for user at context. If no context is found, uses 'default' for the context.
+ * \param mailbox The \@ delimited string for user\@context. If no context is found, uses 'default' for the context.
  * \param folder the folder to look in
  *
  * This function is used when the mailbox is stored in an IMAP back end.
@@ -5947,9 +5947,9 @@
 	return nummsgs;
 }
 
-/**
+/*!
  * \brief Determines if the given folder has messages.
- * \param mailbox The @ delimited string for user at context. If no context is found, uses 'default' for the context.
+ * \param mailbox The \@ delimited string for user\@context. If no context is found, uses 'default' for the context.
  *
  * This function is used when the mailbox is stored in an ODBC back end.
  * This invokes the messagecount(). Here we are interested in the presence of messages (> 0) only, not the actual count.
@@ -6119,7 +6119,7 @@
 	return ret;
 }
 
-/**
+/*!
  * \brief Determines if the given folder has messages.
  * \param mailbox The \@ delimited string for user\@context. If no context is found, uses 'default' for the context.
  * \param folder the folder to look in
@@ -6152,7 +6152,7 @@
 
 /*!
  * \brief Check the given mailbox's message count.
- * \param mailbox The @ delimited string for user at context. If no context is found, uses 'default' for the context.
+ * \param mailbox The \@ delimited string for user\@context. If no context is found, uses 'default' for the context.
  * \param urgentmsgs  urgent message count.
  * \param newmsgs new message count.
  * \param oldmsgs old message count pointer
diff --git a/apps/confbridge/conf_state.c b/apps/confbridge/conf_state.c
index 305732f..0c05a4c 100644
--- a/apps/confbridge/conf_state.c
+++ b/apps/confbridge/conf_state.c
@@ -52,8 +52,6 @@
  * \brief Mute the user and play MOH if the user requires it.
  *
  * \param user Conference user to mute and optionally start MOH on.
- *
- * \return Nothing
  */
 static void conf_mute_moh_inactive_waitmarked(struct confbridge_user *user)
 {
diff --git a/apps/confbridge/include/confbridge.h b/apps/confbridge/include/confbridge.h
index 5ae49b7..95a0732 100644
--- a/apps/confbridge/include/confbridge.h
+++ b/apps/confbridge/include/confbridge.h
@@ -457,8 +457,6 @@
  * \brief Update the actual mute status of the user and set it on the bridge.
  *
  * \param user User to update the mute status.
- *
- * \return Nothing
  */
 void conf_update_user_mute(struct confbridge_user *user);
 
@@ -466,8 +464,6 @@
  * \brief Stop MOH for the conference user.
  *
  * \param user Conference user to stop MOH on.
- *
- * \return Nothing
  */
 void conf_moh_stop(struct confbridge_user *user);
 
@@ -475,8 +471,6 @@
  * \brief Start MOH for the conference user.
  *
  * \param user Conference user to start MOH on.
- *
- * \return Nothing
  */
 void conf_moh_start(struct confbridge_user *user);
 
@@ -487,8 +481,6 @@
 
 /*! \brief Callback to execute any time we transition from zero to one active users
  * \param conference The conference bridge with a single active user joined
- * \retval 0 success
- * \retval -1 failure
  */
 void conf_handle_first_join(struct confbridge_conference *conference);
 
@@ -702,7 +694,7 @@
  * \since 13.22.0
  * \since 15.5.0
  *
- * \param confbridge_name The name to search for
+ * \param conference_name The name to search for
  *
  * \return ConfBridge (which must be unreffed) or NULL.
  */

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

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: Icb6fbcfea0a5f1c82caa5001902b6a786adbf307
Gerrit-Change-Number: 17363
Gerrit-PatchSet: 1
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20211115/690ceda2/attachment-0001.html>


More information about the asterisk-code-review mailing list