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

Friendly Automation asteriskteam at digium.com
Thu Nov 18 14:12:27 CST 2021


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

Change subject: channel: Fix for Doxygen.
......................................................................

channel: Fix for Doxygen.

ASTERISK-29751

Change-Id: Ie04da5029c57ebee44733bdf05013156abe80176
---
M include/asterisk/channel.h
M main/channel.c
2 files changed, 23 insertions(+), 187 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/include/asterisk/channel.h b/include/asterisk/channel.h
index b59e4f1..4c4974a 100644
--- a/include/asterisk/channel.h
+++ b/include/asterisk/channel.h
@@ -110,7 +110,6 @@
 	\li \see ast_channel_early_bridge()
 	\li \see ast_channel_bridge()
 	\li \see app_meetme.c
-	\li \ref AstRTPbridge
 	\li \see ast_rtp_bridge()
 	\li \ref Def_Channel
 */
@@ -1324,7 +1323,8 @@
  * \note The channel does not need to be locked before calling this function.
  * \param[in] chan channel to queue frame onto
  * \param[in] cause the hangup cause
- * \return 0 on success, -1 on error
+ * \retval 0 on success
+ * \retval -1 on error
  * \since 1.6.1
  */
 int ast_queue_hangup_with_cause(struct ast_channel *chan, int cause);
@@ -1414,8 +1414,6 @@
  * \param chan the channel to change the name of
  * \param newname the name to change to
  *
- * \return nothing
- *
  * \note this function must _NEVER_ be used when any channels are locked
  * regardless if it is the channel who's name is being changed or not because
  * it invalidates our channel container locking order... lock container first,
@@ -1429,7 +1427,7 @@
  * This function will unlink the channel from the global channels container
  * if it is still there and also release the current reference to the channel.
  *
- * \return NULL, convenient for clearing invalid pointers
+ * \retval NULL, convenient for clearing invalid pointers
  * \note Absolutely _NO_ channel locks should be held before calling this function.
  *
  * \since 1.8
@@ -1492,8 +1490,6 @@
  * \pre The chan and requestor channels are already locked.
  *
  * \note Pre-existing accountcodes on chan will be overwritten.
- *
- * \return Nothing
  */
 void ast_channel_req_accountcodes(struct ast_channel *chan, const struct ast_channel *requestor, enum ast_channel_requestor_relationship relationship);
 
@@ -1508,8 +1504,6 @@
  * \pre The chan and requestor channels are already locked.
  *
  * \note Pre-existing accountcodes on chan will not be overwritten.
- *
- * \return Nothing
  */
 void ast_channel_req_accountcodes_precious(struct ast_channel *chan, const struct ast_channel *requestor, enum ast_channel_requestor_relationship relationship);
 
@@ -1575,7 +1569,6 @@
 /*!
  * \brief Unregister a channel technology
  * \param tech Structure defining channel technology or "type" that was previously registered
- * \return No return value.
  */
 void ast_channel_unregister(const struct ast_channel_tech *tech);
 
@@ -1593,15 +1586,12 @@
  * performs all stream stopping, etc, on the channel that needs to end.
  * chan is no longer valid after this call.
  * \param chan channel to hang up (NULL tolerant)
- * \return Nothing
  */
 void ast_hangup(struct ast_channel *chan);
 
 /*!
  * \brief Soft hangup all active channels.
  * \since 13.3.0
- *
- * \return Nothing
  */
 void ast_softhangup_all(void);
 
@@ -1609,7 +1599,7 @@
  * \brief Softly hangup up a channel
  *
  * \param chan channel to be soft-hung-up
- * \param reason an AST_SOFTHANGUP_* reason code
+ * \param cause an AST_SOFTHANGUP_* reason code
  *
  * \details
  * Call the protocol layer, but don't destroy the channel structure
@@ -1620,14 +1610,14 @@
  *
  * \return Returns 0 regardless
  */
-int ast_softhangup(struct ast_channel *chan, int reason);
+int ast_softhangup(struct ast_channel *chan, int cause);
 
 /*!
  * \brief Softly hangup up a channel (no channel lock)
  * \param chan channel to be soft-hung-up
- * \param reason an AST_SOFTHANGUP_* reason code
+ * \param cause an AST_SOFTHANGUP_* reason code
  */
-int ast_softhangup_nolock(struct ast_channel *chan, int reason);
+int ast_softhangup_nolock(struct ast_channel *chan, int cause);
 
 /*!
  * \brief Clear a set of softhangup flags from a channel
@@ -1638,8 +1628,6 @@
  *
  * \param chan the channel to clear the flag on
  * \param flag the flag or flags to clear
- *
- * \return Nothing.
  */
 void ast_channel_clear_softhangup(struct ast_channel *chan, int flag);
 
@@ -1743,7 +1731,6 @@
  * \brief Lock the given channel, then request softhangup on the channel with the given causecode
  * \param chan channel on which to hang up
  * \param causecode cause code to use (Zero if don't use cause code)
- * \return Nothing
  */
 void ast_channel_softhangup_withcause_locked(struct ast_channel *chan, int causecode);
 
@@ -1770,7 +1757,6 @@
  *
  * \pre chan is locked
  *
- * \return Nothing
  * \since 1.6.1
  */
 void ast_channel_setwhentohangup_tv(struct ast_channel *chan, struct timeval offset);
@@ -1995,7 +1981,7 @@
 
 /*!
  * \brief Waits for activity on a group of channels
- * \param chan an array of pointers to channels
+ * \param c an array of pointers to channels
  * \param n number of channels that are to be waited upon
  * \param fds an array of fds to wait upon
  * \param nfds the number of fds to wait upon
@@ -2009,7 +1995,7 @@
  * came first.  If the FD came first, it will be returned in outfd, otherwise, outfd
  * will be -1
  */
-struct ast_channel *ast_waitfor_nandfds(struct ast_channel **chan, int n,
+struct ast_channel *ast_waitfor_nandfds(struct ast_channel **c, int n,
 	int *fds, int nfds, int *exception, int *outfd, int *ms);
 
 /*!
@@ -2384,10 +2370,10 @@
 /*!
  * \brief Gives the string form of a given cause code.
  *
- * \param state cause to get the description of
+ * \param cause cause to get the description of
  * \return the text form of the binary cause code given
  */
-const char *ast_cause2str(int state) attribute_pure;
+const char *ast_cause2str(int cause) attribute_pure;
 
 /*!
  * \brief Convert the string form of a cause code to a number
@@ -2400,10 +2386,12 @@
 /*!
  * \brief Gives the string form of a given channel state
  *
- * \param ast_channel_state state to get the name of
+ * \param state state to get the name of
  * \return the text form of the binary state given
+ *
+ * \note This function is not reentrant.
  */
-const char *ast_state2str(enum ast_channel_state);
+const char *ast_state2str(enum ast_channel_state state);
 
 /*!
  * \brief Gives the string form of a given transfer capability
@@ -2546,8 +2534,6 @@
  * \param caller Caller id information
  * \param update What caller information to update.  NULL if all.
  *
- * \return Nothing
- *
  * \note The channel does not need to be locked before calling this function.
  */
 void ast_channel_set_caller(struct ast_channel *chan, const struct ast_party_caller *caller, const struct ast_set_party_caller *update);
@@ -2561,8 +2547,6 @@
  * \param caller Caller id information
  * \param update What caller information to update.  NULL if all.
  *
- * \return Nothing
- *
  * \note The channel does not need to be locked before calling this function.
  */
 void ast_channel_set_caller_event(struct ast_channel *chan, const struct ast_party_caller *caller, const struct ast_set_party_caller *update);
@@ -2605,8 +2589,6 @@
  *
  * \param chan Chan to put into autoservice.
  * \param peer Chan to run hangup handlers and hangup.
- *
- * \return Nothing
  */
 void ast_autoservice_chan_hangup_peer(struct ast_channel *chan, struct ast_channel *peer);
 
@@ -2656,10 +2638,10 @@
  * \retval -1 on error
  * \retval 0 if not supported
  * \retval 1 if supported and requested
- * \param chan current channel
- * \param dest destination extension for transfer
- * \param protocol specific error code in case of failure
- * Example, sip 0 success, else sip error code
+ * \param chan channel to transfer
+ * \param dest destination extension to transfer to
+ * \param protocol protocol is the protocol result
+ * SIP example, 0=success, 3xx-6xx is SIP error code
  */
 int ast_transfer_protocol(struct ast_channel *chan, char *dest, int *protocol);
 
@@ -2718,7 +2700,6 @@
  * \param chan The channel to operate on
  * \param state The ast_silence_generator pointer return by a previous call to
  * ast_channel_start_silence_generator.
- * \return nothing
  *
  * \details
  * This function will stop the operating silence generator and return the channel
@@ -2738,7 +2719,6 @@
  * \brief Copy the full linkedid channel id structure from one channel to another
  * \param dest Destination to copy linkedid to
  * \param source Source channel to copy linkedid from
- * \return void
  */
 void ast_channel_internal_copy_linkedid(struct ast_channel *dest, struct ast_channel *source);
 
@@ -2746,7 +2726,6 @@
  * \brief Swap uniqueid and linkedid beteween two channels
  * \param a First channel
  * \param b Second channel
- * \return void
  *
  * \note
  * This is used in masquerade to exchange identities
@@ -2757,7 +2736,6 @@
  * \brief Swap topics beteween two channels
  * \param a First channel
  * \param b Second channel
- * \return void
  *
  * \note
  * This is used in masquerade to exchange topics for message routing
@@ -2768,7 +2746,6 @@
  * \brief Swap endpoint_forward between two channels
  * \param a First channel
  * \param b Second channel
- * \return void
  *
  * \note
  * This is used in masquerade to exchange endpoint details if one of the two or both
@@ -2780,7 +2757,6 @@
  * \brief Swap snapshots beteween two channels
  * \param a First channel
  * \param b Second channel
- * \return void
  *
  * \note
  * This is used in masquerade to exchange snapshots
@@ -2792,7 +2768,6 @@
  * \param chan The channel to set the uniqueid to
  * \param uniqueid The uniqueid to set
  * \param linkedid The linkedid to set
- * \return void
  *
  * \note
  * This is used only by ast_cel_fabricate_channel_from_event()
@@ -3131,8 +3106,6 @@
  * \since 1.8
  *
  * \param init Name structure to initialize.
- *
- * \return Nothing
  */
 void ast_party_name_init(struct ast_party_name *init);
 
@@ -3142,8 +3115,6 @@
  *
  * \param dest Destination party name
  * \param src Source party name
- *
- * \return Nothing
  */
 void ast_party_name_copy(struct ast_party_name *dest, const struct ast_party_name *src);
 
@@ -3161,8 +3132,6 @@
  *
  * \param init Party name structure to initialize.
  * \param guide Source party name to use as a guide in initializing.
- *
- * \return Nothing
  */
 void ast_party_name_set_init(struct ast_party_name *init, const struct ast_party_name *guide);
 
@@ -3172,8 +3141,6 @@
  *
  * \param dest The name one wishes to update
  * \param src The new name values to update the dest
- *
- * \return Nothing
  */
 void ast_party_name_set(struct ast_party_name *dest, const struct ast_party_name *src);
 
@@ -3182,8 +3149,6 @@
  * \since 1.8
  *
  * \param doomed The party name to destroy.
- *
- * \return Nothing
  */
 void ast_party_name_free(struct ast_party_name *doomed);
 
@@ -3192,8 +3157,6 @@
  * \since 1.8
  *
  * \param init Number structure to initialize.
- *
- * \return Nothing
  */
 void ast_party_number_init(struct ast_party_number *init);
 
@@ -3203,8 +3166,6 @@
  *
  * \param dest Destination party number
  * \param src Source party number
- *
- * \return Nothing
  */
 void ast_party_number_copy(struct ast_party_number *dest, const struct ast_party_number *src);
 
@@ -3222,8 +3183,6 @@
  *
  * \param init Party number structure to initialize.
  * \param guide Source party number to use as a guide in initializing.
- *
- * \return Nothing
  */
 void ast_party_number_set_init(struct ast_party_number *init, const struct ast_party_number *guide);
 
@@ -3233,8 +3192,6 @@
  *
  * \param dest The number one wishes to update
  * \param src The new number values to update the dest
- *
- * \return Nothing
  */
 void ast_party_number_set(struct ast_party_number *dest, const struct ast_party_number *src);
 
@@ -3243,8 +3200,6 @@
  * \since 1.8
  *
  * \param doomed The party number to destroy.
- *
- * \return Nothing
  */
 void ast_party_number_free(struct ast_party_number *doomed);
 
@@ -3253,8 +3208,6 @@
  * \brief Initialize the given subaddress structure.
  *
  * \param init Subaddress structure to initialize.
- *
- * \return Nothing
  */
 void ast_party_subaddress_init(struct ast_party_subaddress *init);
 
@@ -3264,8 +3217,6 @@
  *
  * \param dest Destination party subaddress
  * \param src Source party subaddress
- *
- * \return Nothing
  */
 void ast_party_subaddress_copy(struct ast_party_subaddress *dest, const struct ast_party_subaddress *src);
 
@@ -3283,8 +3234,6 @@
  *
  * \param init Party subaddress structure to initialize.
  * \param guide Source party subaddress to use as a guide in initializing.
- *
- * \return Nothing
  */
 void ast_party_subaddress_set_init(struct ast_party_subaddress *init, const struct ast_party_subaddress *guide);
 
@@ -3294,8 +3243,6 @@
  *
  * \param dest The subaddress one wishes to update
  * \param src The new subaddress values to update the dest
- *
- * \return Nothing
  */
 void ast_party_subaddress_set(struct ast_party_subaddress *dest, const struct ast_party_subaddress *src);
 
@@ -3304,8 +3251,6 @@
  * \brief Destroy the party subaddress contents
  *
  * \param doomed The party subaddress to destroy.
- *
- * \return Nothing
  */
 void ast_party_subaddress_free(struct ast_party_subaddress *doomed);
 
@@ -3314,8 +3259,6 @@
  * \since 11.0
  *
  * \param update_id The update marker for a corresponding party id.
- *
- * \return Nothing
  */
 void ast_set_party_id_all(struct ast_set_party_id *update_id);
 
@@ -3324,8 +3267,6 @@
  * \since 1.8
  *
  * \param init Party id structure to initialize.
- *
- * \return Nothing
  */
 void ast_party_id_init(struct ast_party_id *init);
 
@@ -3335,8 +3276,6 @@
  *
  * \param dest Destination party id
  * \param src Source party id
- *
- * \return Nothing
  */
 void ast_party_id_copy(struct ast_party_id *dest, const struct ast_party_id *src);
 
@@ -3354,8 +3293,6 @@
  *
  * \param init Party id structure to initialize.
  * \param guide Source party id to use as a guide in initializing.
- *
- * \return Nothing
  */
 void ast_party_id_set_init(struct ast_party_id *init, const struct ast_party_id *guide);
 
@@ -3366,8 +3303,6 @@
  * \param dest The id one wishes to update
  * \param src The new id values to update the dest
  * \param update What id information to update.  NULL if all.
- *
- * \return Nothing
  */
 void ast_party_id_set(struct ast_party_id *dest, const struct ast_party_id *src, const struct ast_set_party_id *update);
 
@@ -3376,8 +3311,6 @@
  * \since 1.8
  *
  * \param doomed The party id to destroy.
- *
- * \return Nothing
  */
 void ast_party_id_free(struct ast_party_id *doomed);
 
@@ -3396,8 +3329,6 @@
  * \since 11.0
  *
  * \param id The party id to invalidate.
- *
- * \return Nothing
  */
 void ast_party_id_invalidate(struct ast_party_id *id);
 
@@ -3406,8 +3337,6 @@
  * \since 11.0
  *
  * \param id The party id to reset.
- *
- * \return Nothing
  */
 void ast_party_id_reset(struct ast_party_id *id);
 
@@ -3446,8 +3375,6 @@
  * \param dest The resulting merged party id.
  * \param base The party id which is merged.
  * \param overlay The party id which is used to merge into.
- *
- * \return Nothing
  */
 void ast_party_id_merge_copy(struct ast_party_id *dest, struct ast_party_id *base, struct ast_party_id *overlay);
 
@@ -3456,8 +3383,6 @@
  * \since 1.8
  *
  * \param init Dialed structure to initialize.
- *
- * \return Nothing
  */
 void ast_party_dialed_init(struct ast_party_dialed *init);
 
@@ -3467,8 +3392,6 @@
  *
  * \param dest Destination dialed party
  * \param src Source dialed party
- *
- * \return Nothing
  */
 void ast_party_dialed_copy(struct ast_party_dialed *dest, const struct ast_party_dialed *src);
 
@@ -3486,8 +3409,6 @@
  *
  * \param init Caller structure to initialize.
  * \param guide Source dialed to use as a guide in initializing.
- *
- * \return Nothing
  */
 void ast_party_dialed_set_init(struct ast_party_dialed *init, const struct ast_party_dialed *guide);
 
@@ -3500,8 +3421,6 @@
  *
  * \param dest The dialed one wishes to update
  * \param src The new dialed values to update the dest
- *
- * \return Nada
  */
 void ast_party_dialed_set(struct ast_party_dialed *dest, const struct ast_party_dialed *src);
 
@@ -3510,8 +3429,6 @@
  * \since 1.8
  *
  * \param doomed The dialed party to destroy.
- *
- * \return Nothing
  */
 void ast_party_dialed_free(struct ast_party_dialed *doomed);
 
@@ -3520,8 +3437,6 @@
  * \brief Initialize the given caller structure.
  *
  * \param init Caller structure to initialize.
- *
- * \return Nothing
  */
 void ast_party_caller_init(struct ast_party_caller *init);
 
@@ -3531,8 +3446,6 @@
  *
  * \param dest Destination caller
  * \param src Source caller
- *
- * \return Nothing
  */
 void ast_party_caller_copy(struct ast_party_caller *dest, const struct ast_party_caller *src);
 
@@ -3550,8 +3463,6 @@
  *
  * \param init Caller structure to initialize.
  * \param guide Source caller to use as a guide in initializing.
- *
- * \return Nothing
  */
 void ast_party_caller_set_init(struct ast_party_caller *init, const struct ast_party_caller *guide);
 
@@ -3565,8 +3476,6 @@
  * \param dest The caller one wishes to update
  * \param src The new caller values to update the dest
  * \param update What caller information to update.  NULL if all.
- *
- * \return Nada
  */
 void ast_party_caller_set(struct ast_party_caller *dest, const struct ast_party_caller *src, const struct ast_set_party_caller *update);
 
@@ -3575,8 +3484,6 @@
  * \since 1.8
  *
  * \param doomed The caller party to destroy.
- *
- * \return Nothing
  */
 void ast_party_caller_free(struct ast_party_caller *doomed);
 
@@ -3585,8 +3492,6 @@
  * \brief Initialize the given connected line structure.
  *
  * \param init Connected line structure to initialize.
- *
- * \return Nothing
  */
 void ast_party_connected_line_init(struct ast_party_connected_line *init);
 
@@ -3596,8 +3501,6 @@
  *
  * \param dest Destination connected line
  * \param src Source connected line
- *
- * \return Nothing
  */
 void ast_party_connected_line_copy(struct ast_party_connected_line *dest, const struct ast_party_connected_line *src);
 
@@ -3615,8 +3518,6 @@
  *
  * \param init Connected line structure to initialize.
  * \param guide Source connected line to use as a guide in initializing.
- *
- * \return Nothing
  */
 void ast_party_connected_line_set_init(struct ast_party_connected_line *init, const struct ast_party_connected_line *guide);
 
@@ -3630,8 +3531,6 @@
  * \param dest The connected line one wishes to update
  * \param src The new connected line values to update the dest
  * \param update What connected line information to update.  NULL if all.
- *
- * \return Nothing
  */
 void ast_party_connected_line_set(struct ast_party_connected_line *dest, const struct ast_party_connected_line *src, const struct ast_set_party_connected_line *update);
 
@@ -3642,8 +3541,6 @@
  * \param connected Collected caller information for the connected line
  * \param caller Caller information.
  *
- * \return Nothing
- *
  * \warning This is a shallow copy.
  * \warning DO NOT call ast_party_connected_line_free() on the filled in
  * connected line structure!
@@ -3655,8 +3552,6 @@
  * \brief Destroy the connected line information contents
  *
  * \param doomed The connected line information to destroy.
- *
- * \return Nothing
  */
 void ast_party_connected_line_free(struct ast_party_connected_line *doomed);
 
@@ -3664,8 +3559,6 @@
  * \brief Initialize the given redirecting reason structure
  *
  * \param init Redirecting reason structure to initialize
- *
- * \return Nothing
  */
 void ast_party_redirecting_reason_init(struct ast_party_redirecting_reason *init);
 
@@ -3674,8 +3567,6 @@
  *
  * \param dest Destination redirecting reason
  * \param src Source redirecting reason
- *
- * \return Nothing
  */
 void ast_party_redirecting_reason_copy(struct ast_party_redirecting_reason *dest,
 		const struct ast_party_redirecting_reason *src);
@@ -3693,8 +3584,6 @@
  *
  * \param init Redirecting reason structure to initialize.
  * \param guide Source redirecting reason to use as a guide in initializing.
- *
- * \return Nothing
  */
 void ast_party_redirecting_reason_set_init(struct ast_party_redirecting_reason *init,
 		const struct ast_party_redirecting_reason *guide);
@@ -3707,8 +3596,6 @@
  *
  * \param dest The redirecting reason one wishes to update
  * \param src The new redirecting reason values to update the dest
- *
- * \return Nothing
  */
 void ast_party_redirecting_reason_set(struct ast_party_redirecting_reason *dest,
 		const struct ast_party_redirecting_reason *src);
@@ -3717,8 +3604,6 @@
  * \brief Destroy the redirecting reason contents
  *
  * \param doomed The redirecting reason to destroy.
- *
- * \return Nothing
  */
 void ast_party_redirecting_reason_free(struct ast_party_redirecting_reason *doomed);
 
@@ -3727,8 +3612,6 @@
  * \since 1.8
  *
  * \param init Redirecting structure to initialize.
- *
- * \return Nothing
  */
 void ast_party_redirecting_init(struct ast_party_redirecting *init);
 
@@ -3738,8 +3621,6 @@
  *
  * \param dest Destination redirecting
  * \param src Source redirecting
- *
- * \return Nothing
  */
 void ast_party_redirecting_copy(struct ast_party_redirecting *dest, const struct ast_party_redirecting *src);
 
@@ -3757,8 +3638,6 @@
  *
  * \param init Redirecting id structure to initialize.
  * \param guide Source redirecting id to use as a guide in initializing.
- *
- * \return Nothing
  */
 void ast_party_redirecting_set_init(struct ast_party_redirecting *init, const struct ast_party_redirecting *guide);
 
@@ -3772,8 +3651,6 @@
  * \param dest The redirecting one wishes to update
  * \param src The new redirecting values to update the dest
  * \param update What redirecting information to update.  NULL if all.
- *
- * \return Nothing
  */
 void ast_party_redirecting_set(struct ast_party_redirecting *dest, const struct ast_party_redirecting *src, const struct ast_set_party_redirecting *update);
 
@@ -3782,8 +3659,6 @@
  * \brief Destroy the redirecting information contents
  *
  * \param doomed The redirecting information to destroy.
- *
- * \return Nothing
  */
 void ast_party_redirecting_free(struct ast_party_redirecting *doomed);
 
@@ -3794,8 +3669,6 @@
  * \param dest Destination connected line information
  * \param src Source caller information
  *
- * \return Nothing
- *
  * \note Assumes locks are already acquired
  */
 void ast_connected_line_copy_from_caller(struct ast_party_connected_line *dest, const struct ast_party_caller *src);
@@ -3807,8 +3680,6 @@
  * \param dest Destination caller information
  * \param src Source connected line information
  *
- * \return Nothing
- *
  * \note Assumes locks are already acquired
  */
 void ast_connected_line_copy_to_caller(struct ast_party_caller *dest, const struct ast_party_connected_line *src);
@@ -3821,8 +3692,6 @@
  * \param connected Connected line information
  * \param update What connected line information to update.  NULL if all.
  *
- * \return Nothing
- *
  * \note The channel does not need to be locked before calling this function.
  */
 void ast_channel_set_connected_line(struct ast_channel *chan, const struct ast_party_connected_line *connected, const struct ast_set_party_connected_line *update);
@@ -3867,8 +3736,6 @@
  * \param chan Asterisk channel to indicate connected line information
  * \param connected Connected line information
  * \param update What connected line information to update.  NULL if all.
- *
- * \return Nothing
  */
 void ast_channel_update_connected_line(struct ast_channel *chan, const struct ast_party_connected_line *connected, const struct ast_set_party_connected_line *update);
 
@@ -3879,8 +3746,6 @@
  * \param chan Asterisk channel to indicate connected line information
  * \param connected Connected line information
  * \param update What connected line information to update.  NULL if all.
- *
- * \return Nothing
  */
 void ast_channel_queue_connected_line_update(struct ast_channel *chan, const struct ast_party_connected_line *connected, const struct ast_set_party_connected_line *update);
 
@@ -3892,8 +3757,6 @@
  * \param redirecting Redirecting id information
  * \param update What redirecting information to update.  NULL if all.
  *
- * \return Nothing
- *
  * \note The channel does not need to be locked before calling this function.
  */
 void ast_channel_set_redirecting(struct ast_channel *chan, const struct ast_party_redirecting *redirecting, const struct ast_set_party_redirecting *update);
@@ -3937,8 +3800,6 @@
  * \param chan Asterisk channel to indicate redirecting id information
  * \param redirecting Redirecting id information
  * \param update What redirecting information to update.  NULL if all.
- *
- * \return Nothing
  */
 void ast_channel_update_redirecting(struct ast_channel *chan, const struct ast_party_redirecting *redirecting, const struct ast_set_party_redirecting *update);
 
@@ -3949,8 +3810,6 @@
  * \param chan Asterisk channel to indicate redirecting id information
  * \param redirecting Redirecting id information
  * \param update What redirecting information to update.  NULL if all.
- *
- * \return Nothing
  */
 void ast_channel_queue_redirecting_update(struct ast_channel *chan, const struct ast_party_redirecting *redirecting, const struct ast_set_party_redirecting *update);
 
@@ -4607,7 +4466,7 @@
 
 /*!
  * \since 13.17.0
- * \param Clear a flag on a channel
+ * \brief Clear a flag on a channel
  *
  * \param chan The channel to clear the flag from
  * \param flag The flag to clear
@@ -4940,8 +4799,6 @@
  * \param digit DTMF digit to stop.
  * \param start DTMF digit start time.
  * \param why Reason bridge broken.
- *
- * \return Nothing
  */
 void ast_channel_end_dtmf(struct ast_channel *chan, char digit, struct timeval start, const char *why);
 
@@ -5047,7 +4904,7 @@
 /*!
  * \brief Retrieve the default stream of a specific media type on a channel
  *
- * \param channel The channel to get the stream from
+ * \param chan The channel to get the stream from
  * \param type The media type of the default stream
  *
  * \pre chan is locked
@@ -5060,11 +4917,11 @@
 /*!
  * \brief Determine if a channel is multi-stream capable
  *
- * \param channel The channel to test
+ * \param chan The channel to test
  *
  * \pre chan is locked
  *
- * \return Returns true if the channel is multi-stream capable.
+ * \retval true if the channel is multi-stream capable.
  */
 int ast_channel_is_multistream(struct ast_channel *chan);
 
diff --git a/main/channel.c b/main/channel.c
index 30f66d4..546f4f8 100644
--- a/main/channel.c
+++ b/main/channel.c
@@ -629,12 +629,6 @@
 	return -1;
 }
 
-/*! \brief Gives the string form of a given channel state.
- *
- * \note This function is not reentrant.
- *
- * \param state
- */
 const char *ast_state2str(enum ast_channel_state state)
 {
 	char *buf;
@@ -2627,8 +2621,6 @@
  * \since 13.11.0
  *
  * \param chan Channel to set answered time.
- *
- * \return Nothing
  */
 static void set_channel_answer_time(struct ast_channel *chan)
 {
@@ -5870,8 +5862,6 @@
  * \param new_chan Channel inheriting information.
  * \param parent Channel new_chan inherits information.
  * \param orig Channel being replaced by the call forward channel.
- *
- * \return Nothing
  */
 static void call_forward_inherit(struct ast_channel *new_chan, struct ast_channel *parent, struct ast_channel *orig)
 {
@@ -6379,8 +6369,6 @@
  * \param precious TRUE if pre-existing accountcodes on chan will not be overwritten.
  *
  * \pre The chan and requestor channels are already locked.
- *
- * \return Nothing
  */
 static void channel_req_accountcodes(struct ast_channel *chan, const struct ast_channel *requestor, enum ast_channel_requestor_relationship relationship, int precious)
 {
@@ -6500,11 +6488,6 @@
 /*!
   \brief Transfer a call to dest, if the channel supports transfer
 
-  \param chan channel to transfer
-  \param dest destination to transfer to
-  \param protocol is the protocol result
-  SIP example, 0=success, 3xx-6xx is SIP error code
-
   Called by:
 	\arg app_transfer
 	\arg the manager interface
@@ -7823,8 +7806,6 @@
  * \param v_obj A pointer to the object we want the key printed.
  * \param where User data needed by prnt to determine where to put output.
  * \param prnt Print output callback function to use.
- *
- * \return Nothing
  */
 static void prnt_channel_key(void *v_obj, void *where, ao2_prnt_fn *prnt)
 {
@@ -10336,8 +10317,6 @@
  * \since 13.14.0
  *
  * \param in_intercept_mode New intercept mode.  (Non-zero if in intercept mode)
- *
- * \return Nothing
  */
 static void channel_set_intercept_mode(int in_intercept_mode)
 {

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

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: Ie04da5029c57ebee44733bdf05013156abe80176
Gerrit-Change-Number: 17479
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/9b2e0ef0/attachment-0001.html>


More information about the asterisk-code-review mailing list