[Asterisk-code-review] progtests: Fix Doxygen left-overs. (asterisk[19])

Alexander Traud asteriskteam at digium.com
Mon Dec 6 10:13:12 CST 2021


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


Change subject: progtests: Fix Doxygen left-overs.
......................................................................

progtests: Fix Doxygen left-overs.

Change-Id: I5b5cf9c9cbbe00ba8b379a8d162ac67445d39016
---
M apps/app_voicemail.c
M include/asterisk/cdr.h
M include/asterisk/channel.h
M include/asterisk/conversions.h
M include/asterisk/core_local.h
M include/asterisk/endpoints.h
M include/asterisk/http.h
M include/asterisk/image.h
M include/asterisk/stasis_app_device_state.h
M include/asterisk/stasis_cache_pattern.h
M include/asterisk/stasis_endpoints.h
M include/asterisk/stasis_internal.h
M include/asterisk/threadpool.h
M main/stasis_endpoints.c
M res/ari/internal.h
M res/parking/parking_ui.c
M res/stasis_recording/stored.c
M tests/test_bridging.c
18 files changed, 43 insertions(+), 45 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/28/17628/1

diff --git a/apps/app_voicemail.c b/apps/app_voicemail.c
index a668a5a..ebfa196 100644
--- a/apps/app_voicemail.c
+++ b/apps/app_voicemail.c
@@ -15934,7 +15934,7 @@
  * \param vms The voicemail state corresponding to an open mailbox
  * \param msg_ids An array of message identifiers
  * \param num_msgs The number of identifiers in msg_ids
- * \param msg_nums [out] The message indexes corresponding to the given
+ * \param[out] msg_nums The message indexes corresponding to the given
  * \param vmu
  * message IDs
  * \pre vms must have open_mailbox() called on it prior to this function.
diff --git a/include/asterisk/cdr.h b/include/asterisk/cdr.h
index cf3e2f2..e2a5a90 100644
--- a/include/asterisk/cdr.h
+++ b/include/asterisk/cdr.h
@@ -494,7 +494,6 @@
 /*!
  * \brief Free a CDR record
  * \param cdr ast_cdr structure to free
- * Returns nothing
  */
 void ast_cdr_free(struct ast_cdr *cdr);
 
diff --git a/include/asterisk/channel.h b/include/asterisk/channel.h
index bc12de8..e9dcf6a 100644
--- a/include/asterisk/channel.h
+++ b/include/asterisk/channel.h
@@ -4504,7 +4504,7 @@
  *
  * \param chan Channel to get variables for.
  * \return List of channel variables.
- * \return \c NULL on error
+ * \retval NULL on error
  */
 struct varshead *ast_channel_get_manager_vars(struct ast_channel *chan);
 
@@ -4534,7 +4534,7 @@
  *
  * \param chan Channel to get variables for.
  * \return List of channel variables.
- * \return \c NULL on error
+ * \retval NULL on error
  */
 struct varshead *ast_channel_get_ari_vars(struct ast_channel *chan);
 
@@ -4546,7 +4546,7 @@
  *
  * \param chan Channel to get variables for
  * \return List of channel variables.
- * \return \c NULL on error
+ * \retval NULL on error
  */
 struct varshead *ast_channel_get_vars(struct ast_channel *chan);
 
diff --git a/include/asterisk/conversions.h b/include/asterisk/conversions.h
index 852d80f..fb7309a 100644
--- a/include/asterisk/conversions.h
+++ b/include/asterisk/conversions.h
@@ -37,7 +37,7 @@
  *       or greater than INT_MAX)
  *
  * \param str The string to convert
- * \param res [out] The converted value
+ * \param[out] res The converted value
  *
  * \returns -1 if it fails to convert, 0 on success
  */
@@ -55,7 +55,7 @@
  *   Once converted the number is out of range (greater than UINT_MAX)
  *
  * \param str The string to convert
- * \param res [out] The converted value
+ * \param[out] res The converted value
  *
  * \returns -1 if it fails to convert, 0 on success
  */
@@ -73,7 +73,7 @@
  *       or greater than LONG_MAX)
  *
  * \param str The string to convert
- * \param res [out] The converted value
+ * \param[out] res The converted value
  *
  * \returns -1 if it fails to convert, 0 on success
  */
@@ -91,7 +91,7 @@
  *   Once converted the number is out of range (greater than ULONG_MAX)
  *
  * \param str The string to convert
- * \param res [out] The converted value
+ * \param[out] res The converted value
  *
  * \returns -1 if it fails to convert, 0 on success
  */
@@ -109,7 +109,7 @@
  *       or greater than INTMAX_MAX)
  *
  * \param str The string to convert
- * \param res [out] The converted value
+ * \param[out] res The converted value
  *
  * \returns -1 if it fails to convert, 0 on success
  */
@@ -127,7 +127,7 @@
  *   Once converted the number is out of range (greater than UINTMAX_MAX)
  *
  * \param str The string to convert
- * \param res [out] The converted value
+ * \param[out] res The converted value
  *
  * \returns -1 if it fails to convert, 0 on success
  */
diff --git a/include/asterisk/core_local.h b/include/asterisk/core_local.h
index 27e9244..fbe8528 100644
--- a/include/asterisk/core_local.h
+++ b/include/asterisk/core_local.h
@@ -51,9 +51,9 @@
  * \since 13.17.0, 14.6.0
  *
  * \param chan Must be a local channel
- * \param tech_pvt [out] channel's private tech (ref and lock added)
- * \param base_chan [out] One side of the local channel (ref and lock added)
- * \param base_owner [out] Other side of the local channel (ref and lock added)
+ * \param[out] tech_pvt channel's private tech (ref and lock added)
+ * \param[out] base_chan One side of the local channel (ref and lock added)
+ * \param[out] base_owner Other side of the local channel (ref and lock added)
  */
 void ast_local_lock_all(struct ast_channel *chan, void **tech_pvt,
 	struct ast_channel **base_chan, struct ast_channel **base_owner);
diff --git a/include/asterisk/endpoints.h b/include/asterisk/endpoints.h
index ee73dc6..0c5edce 100644
--- a/include/asterisk/endpoints.h
+++ b/include/asterisk/endpoints.h
@@ -62,7 +62,7 @@
  *
  * \param state Endpoint state.
  * \return String representation of \a state.
- * \return \c "?" if \a state isn't in \ref ast_endpoint_state.
+ * \retval "?" if \a state isn't in \ref ast_endpoint_state.
  */
 const char *ast_endpoint_state_to_string(enum ast_endpoint_state state);
 
@@ -89,7 +89,7 @@
  *
  * \param id Tech[/resource] id to look for.
  * \return Associated endpoint.
- * \return \c NULL if not found.
+ * \retval NULL if not found.
  *
  * \since 12
  */
@@ -106,7 +106,7 @@
  * \param tech Technology for this endpoint.
  * \param resource Name of this endpoint.
  * \return Newly created endpoint.
- * \return \c NULL on error.
+ * \retval NULL on error.
  * \since 12
  */
 struct ast_endpoint *ast_endpoint_create(const char *tech, const char *resource);
@@ -127,7 +127,7 @@
  *
  * \param endpoint The endpoint.
  * \return Tec of the endpoint.
- * \return \c NULL if endpoint is \c NULL.
+ * \retval NULL if endpoint is \c NULL.
  * \since 12
  */
 const char *ast_endpoint_get_tech(const struct ast_endpoint *endpoint);
@@ -142,7 +142,7 @@
  *
  * \param endpoint The endpoint.
  * \return Resource name of the endpoint.
- * \return \c NULL if endpoint is \c NULL.
+ * \retval NULL if endpoint is \c NULL.
  * \since 12
  */
 const char *ast_endpoint_get_resource(const struct ast_endpoint *endpoint);
@@ -154,7 +154,7 @@
  *
  * \param endpoint The endpoint.
  * \return Tech/resource id of the endpoint.
- * \return \c NULL if endpoint is \c NULL.
+ * \retval NULL if endpoint is \c NULL.
  * \since 12
  */
 const char *ast_endpoint_get_id(const struct ast_endpoint *endpoint);
@@ -164,7 +164,7 @@
  *
  * \param endpoint The endpoint.
  * \return state.
- * \return \c AST_ENDPOINT_UNKNOWN if endpoint is \c NULL.
+ * \retval AST_ENDPOINT_UNKNOWN if endpoint is \c NULL.
  * \since 13.4
  */
 enum ast_endpoint_state ast_endpoint_get_state(const struct ast_endpoint *endpoint);
diff --git a/include/asterisk/http.h b/include/asterisk/http.h
index f8fa8ad..033ae2d 100644
--- a/include/asterisk/http.h
+++ b/include/asterisk/http.h
@@ -311,8 +311,8 @@
  *       respectively.
  *
  * \param buf a string containing the name/value to point to
- * \param name out parameter pointing to the header name
- * \param value out parameter pointing to header value
+ * \param[out] name header name
+ * \param[out] value header value
  * \retval -1 if buf is empty
  * \retval 0 if buf could be separated into name and value
  * \retval 1 if name or value portion don't exist
diff --git a/include/asterisk/image.h b/include/asterisk/image.h
index f159b41..4f49180 100644
--- a/include/asterisk/image.h
+++ b/include/asterisk/image.h
@@ -76,7 +76,6 @@
  * \brief Unregister an image format
  * \param imgdrv pointer to the ast_imager structure you wish to unregister
  * Unregisters the image format passed in.
- * Returns nothing
  */
 void ast_image_unregister(struct ast_imager *imgdrv);
 
diff --git a/include/asterisk/stasis_app_device_state.h b/include/asterisk/stasis_app_device_state.h
index e01ac4a..a0038f7 100644
--- a/include/asterisk/stasis_app_device_state.h
+++ b/include/asterisk/stasis_app_device_state.h
@@ -39,7 +39,7 @@
  * \param name the name of the device
  * \param state the device state
  * \return JSON representation.
- * \return \c NULL on error.
+ * \retval NULL on error.
  */
 struct ast_json *stasis_app_device_state_to_json(
 	const char *name, enum ast_device_state state);
@@ -48,7 +48,7 @@
  * \brief Convert device states to json array.
  *
  * \return JSON representation.
- * \return \c NULL on error.
+ * \retval NULL on error.
  */
 struct ast_json *stasis_app_device_states_to_json(void);
 
diff --git a/include/asterisk/stasis_cache_pattern.h b/include/asterisk/stasis_cache_pattern.h
index 514d62e..547a1c8 100644
--- a/include/asterisk/stasis_cache_pattern.h
+++ b/include/asterisk/stasis_cache_pattern.h
@@ -55,7 +55,7 @@
  * \param name Base name of the topics.
  * \param id_fn Identity function for the cache.
  * \return All side instance.
- * \return \c NULL on error.
+ * \retval NULL on error.
  */
 struct stasis_cp_all *stasis_cp_all_create(const char *name,
 	snapshot_get_id id_fn);
@@ -68,7 +68,7 @@
  *
  * \param all All side caching pattern object.
  * \return The aggregate topic.
- * \return \c NULL if \a all is \c NULL
+ * \retval NULL if \a all is \c NULL
  */
 struct stasis_topic *stasis_cp_all_topic(struct stasis_cp_all *all);
 
@@ -83,7 +83,7 @@
  *
  * \param all All side caching pattern object.
  * \return The aggregate caching topic.
- * \return \c NULL if \a all is \c NULL
+ * \retval NULL if \a all is \c NULL
  */
 struct stasis_topic *stasis_cp_all_topic_cached(
 	struct stasis_cp_all *all);
@@ -95,7 +95,7 @@
  *
  * \param all All side caching pattern object.
  * \return The cache.
- * \return \c NULL if \a all is \c NULL
+ * \retval NULL if \a all is \c NULL
  */
 struct stasis_cache *stasis_cp_all_cache(struct stasis_cp_all *all);
 
@@ -152,7 +152,7 @@
  *
  * \param one One side of the cache pattern.
  * \return The main topic.
- * \return \c NULL if \a one is \c NULL
+ * \retval NULL if \a one is \c NULL
  */
 struct stasis_topic *stasis_cp_single_topic(struct stasis_cp_single *one);
 
@@ -164,7 +164,7 @@
  *
  * \param one One side of the cache pattern.
  * \return The caching topic.
- * \return \c NULL if \a one is \c NULL
+ * \retval NULL if \a one is \c NULL
  */
 struct stasis_topic *stasis_cp_single_topic_cached(
 	struct stasis_cp_single *one);
diff --git a/include/asterisk/stasis_endpoints.h b/include/asterisk/stasis_endpoints.h
index 71b7c59..454aa5f 100644
--- a/include/asterisk/stasis_endpoints.h
+++ b/include/asterisk/stasis_endpoints.h
@@ -91,7 +91,7 @@
  *             \c NULL, ast_json_null() is put into the object.
  *
  * \return \ref ast_endpoint_blob message.
- * \return \c NULL on error
+ * \retval NULL on error
  */
 struct stasis_message *ast_endpoint_blob_create(struct ast_endpoint *endpoint,
 	struct stasis_message_type *type, struct ast_json *blob);
@@ -133,7 +133,7 @@
  * \brief Create a snapshot of an endpoint
  * \param endpoint Endpoint to snap a shot of.
  * \return Snapshot of the endpoint.
- * \return \c NULL on error.
+ * \retval NULL on error.
  * \since 12
  */
 struct ast_endpoint_snapshot *ast_endpoint_snapshot_create(
@@ -200,7 +200,7 @@
  * \param tech Name of the endpoint's technology.
  * \param resource Resource name of the endpoint.
  * \return Snapshot of the endpoint with the given name.
- * \return \c NULL if endpoint is not found, or on error.
+ * \retval NULL if endpoint is not found, or on error.
  * \since 12
  */
 struct ast_endpoint_snapshot *ast_endpoint_latest_snapshot(const char *tech,
@@ -216,7 +216,7 @@
  * \param sanitize The message sanitizer to use on the snapshot
  *
  * \return JSON object representing endpoint snapshot.
- * \return \c NULL on error
+ * \retval NULL on error
  */
 struct ast_json *ast_endpoint_snapshot_to_json(
 	const struct ast_endpoint_snapshot *snapshot,
diff --git a/include/asterisk/stasis_internal.h b/include/asterisk/stasis_internal.h
index 2495fc0..678812e 100644
--- a/include/asterisk/stasis_internal.h
+++ b/include/asterisk/stasis_internal.h
@@ -58,7 +58,7 @@
  *  relevant if \c needs_mailbox is non-zero.
  * \param file, lineno, func
  * \return New \ref stasis_subscription object.
- * \return \c NULL on error.
+ * \retval NULL on error.
  * \since 12
  */
 struct stasis_subscription *internal_stasis_subscribe(
diff --git a/include/asterisk/threadpool.h b/include/asterisk/threadpool.h
index 9474ede..a72e452 100644
--- a/include/asterisk/threadpool.h
+++ b/include/asterisk/threadpool.h
@@ -259,7 +259,7 @@
  * \param pool \ref ast_threadpool for execution.
  *
  * \return \ref ast_taskprocessor for enqueuing work.
- * \return \c NULL on error.
+ * \retval NULL on error.
  */
 struct ast_taskprocessor *ast_threadpool_serializer(const char *name, struct ast_threadpool *pool);
 
@@ -288,7 +288,7 @@
  * \param shutdown_group Group shutdown controller. (NULL if no group association)
  *
  * \return \ref ast_taskprocessor for enqueuing work.
- * \return \c NULL on error.
+ * \retval NULL on error.
  */
 struct ast_taskprocessor *ast_threadpool_serializer_group(const char *name,
 	struct ast_threadpool *pool, struct ast_serializer_shutdown_group *shutdown_group);
diff --git a/main/stasis_endpoints.c b/main/stasis_endpoints.c
index 289a90e..69399e6 100644
--- a/main/stasis_endpoints.c
+++ b/main/stasis_endpoints.c
@@ -378,7 +378,7 @@
  *
  * \param message Message to extract id from.
  * \return String representing the snapshot's id.
- * \return \c NULL if the message_type of the message isn't a handled snapshot.
+ * \retval NULL if the message_type of the message isn't a handled snapshot.
  * \since 12
  */
 static const char *endpoint_snapshot_get_id(struct stasis_message *message)
diff --git a/res/ari/internal.h b/res/ari/internal.h
index 93ea0b7..610e916 100644
--- a/res/ari/internal.h
+++ b/res/ari/internal.h
@@ -126,7 +126,7 @@
  * ao2_cleanup() when you're done with it.
  *
  * \return ARI configuration object.
- * \return \c NULL on error.
+ * \retval NULL on error.
  */
 struct ast_ari_conf *ast_ari_config_get(void);
 
@@ -136,7 +136,7 @@
  * \param username Name of the user.
  * \param password User's password.
  * \return User object.
- * \return \c NULL if username or password is invalid.
+ * \retval NULL if username or password is invalid.
  */
 struct ast_ari_conf_user *ast_ari_config_validate_user(const char *username,
 	const char *password);
diff --git a/res/parking/parking_ui.c b/res/parking/parking_ui.c
index f12963a..2b0bd92 100644
--- a/res/parking/parking_ui.c
+++ b/res/parking/parking_ui.c
@@ -158,7 +158,7 @@
 	return ret;
 }
 
-/* \brief command parking show <name> */
+/*! \brief command parking show \<name\> */
 static char *handle_show_parking_lot_cmd(struct ast_cli_entry *e, int cmd, struct ast_cli_args *a)
 {
 	switch (cmd) {
diff --git a/res/stasis_recording/stored.c b/res/stasis_recording/stored.c
index 7fe2b7c..6cdca96 100644
--- a/res/stasis_recording/stored.c
+++ b/res/stasis_recording/stored.c
@@ -180,7 +180,7 @@
  * \param dir_name Directory to search (absolute path).
  * \param file File name, without extension.
  * \return Absolute path of the recording file.
- * \return \c NULL if recording is not found.
+ * \retval NULL if recording is not found.
  */
 static char *find_recording(const char *dir_name, const char *file)
 {
diff --git a/tests/test_bridging.c b/tests/test_bridging.c
index 74595c4..f4c81b0 100644
--- a/tests/test_bridging.c
+++ b/tests/test_bridging.c
@@ -48,7 +48,7 @@
 
 /*! \brief A private structure for the test channel */
 struct test_bridging_chan_pvt {
-	/* \brief The expected indication */
+	/*! \brief The expected indication */
 	int condition;
 	/*! \brief The number of indicated things */
 	unsigned int indicated;

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

Gerrit-Project: asterisk
Gerrit-Branch: 19
Gerrit-Change-Id: I5b5cf9c9cbbe00ba8b379a8d162ac67445d39016
Gerrit-Change-Number: 17628
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/20211206/854f8b37/attachment-0001.html>


More information about the asterisk-code-review mailing list