From a0f1e460fe90284674e45c6ea692b48fa4e5cc53 Mon Sep 17 00:00:00 2001
From: Bastian Triller <bastian.triller@gmail.com>
Date: Sun, 21 Sep 2025 11:59:48 +0200
Subject: [PATCH] Fix some doxygen, typos and whitespace

---
 agi/eagi-sphinx-test.c              |  2 +-
 apps/app_dial.c                     |  2 +-
 apps/app_queue.c                    |  2 +-
 channels/chan_pjsip.c               |  2 +-
 configs/samples/manager.conf.sample |  2 +-
 include/asterisk/config_options.h   | 10 +++++-----
 include/asterisk/logger.h           | 16 +++++++++-------
 include/asterisk/refer.h            |  4 ----
 main/config_options.c               | 22 +++++++++++-----------
 main/features.c                     |  2 +-
 res/res_fax.c                       |  4 ++--
 res/res_pjsip_session.c             |  4 ++--
 12 files changed, 35 insertions(+), 37 deletions(-)

diff --git a/agi/eagi-sphinx-test.c b/agi/eagi-sphinx-test.c
index c7f6997c2d..df712a0956 100644
--- a/agi/eagi-sphinx-test.c
+++ b/agi/eagi-sphinx-test.c
@@ -12,7 +12,7 @@
  * This code is released into public domain
  * without any warranty of any kind.
  *
- *	\ingroup agi
+ * \ingroup agi
  */
 
 #include <stdio.h>
diff --git a/apps/app_dial.c b/apps/app_dial.c
index 4e583ab783..78b4a77cc8 100644
--- a/apps/app_dial.c
+++ b/apps/app_dial.c
@@ -214,7 +214,7 @@
 					and <emphasis>start</emphasis> execution at that location.</para>
 					<para>NOTE: Any channel variables you want the called channel to inherit from the caller channel must be
 					prefixed with one or two underbars ('_').</para>
-					<para>NOTE: Using this option from a GoSub() might not make sense as there would be no return points.</para>
+					<para>NOTE: Using this option from a Gosub() might not make sense as there would be no return points.</para>
 				</option>
 				<option name="g">
 					<para>Proceed with dialplan execution at the next priority in the current extension if the
diff --git a/apps/app_queue.c b/apps/app_queue.c
index a3aff975a1..2bf59ef038 100644
--- a/apps/app_queue.c
+++ b/apps/app_queue.c
@@ -174,7 +174,7 @@
 						to the specified destination and <emphasis>start</emphasis> execution at that location.</para>
 						<para>NOTE: Any channel variables you want the called channel to inherit from the caller channel must be
 						prefixed with one or two underbars ('_').</para>
-						<para>NOTE: Using this option from a GoSub() might not make sense as there would be no return points.</para>
+						<para>NOTE: Using this option from a Gosub() might not make sense as there would be no return points.</para>
 					</option>
 					<option name="h">
 						<para>Allow <emphasis>callee</emphasis> to hang up by pressing <literal>*</literal>.</para>
diff --git a/channels/chan_pjsip.c b/channels/chan_pjsip.c
index c7a2d6c28c..a639d57630 100644
--- a/channels/chan_pjsip.c
+++ b/channels/chan_pjsip.c
@@ -2463,7 +2463,7 @@ static int hangup_cause2sip(int cause)
 	case AST_CAUSE_NO_USER_RESPONSE:        /* 18 */
 		return 408;
 	case AST_CAUSE_NO_ANSWER:               /* 19 */
-	case AST_CAUSE_UNREGISTERED:        /* 20 */
+	case AST_CAUSE_UNREGISTERED:            /* 20 */
 		return 480;
 	case AST_CAUSE_CALL_REJECTED:           /* 21 */
 		return 403;
diff --git a/configs/samples/manager.conf.sample b/configs/samples/manager.conf.sample
index e5f172ad66..244141ee77 100644
--- a/configs/samples/manager.conf.sample
+++ b/configs/samples/manager.conf.sample
@@ -90,7 +90,7 @@ bindaddr = 0.0.0.0
 ; events from being received in a stream (as opposed to this option which
 ; would prevent specified events from being generated at all).
 
-;disabledevents = Newexten,Varset
+;disabledevents = Newexten,VarSet
 
 ;[mark]
 ;secret = mysecret
diff --git a/include/asterisk/config_options.h b/include/asterisk/config_options.h
index 8f7edfa6cd..146fe89956 100644
--- a/include/asterisk/config_options.h
+++ b/include/asterisk/config_options.h
@@ -167,12 +167,12 @@ struct aco_file {
 };
 
 struct aco_info {
-	const char *module; /*!< The name of the module whose config is being processed */
-	unsigned int hidden:1;                /*!< If enabled, this config item is hidden from users */
-	aco_pre_apply_config pre_apply_config; /*!< A callback called after processing, but before changes are applied */
+	const char *module;			/*!< The name of the module whose config is being processed */
+	unsigned int hidden:1;			/*!< If enabled, this config item is hidden from users */
+	aco_pre_apply_config pre_apply_config;	/*!< A callback called after processing, but before changes are applied */
 	aco_post_apply_config post_apply_config;/*!< A callback called after changes are applied */
-	aco_snapshot_alloc snapshot_alloc;     /*!< Allocate an object to hold all global configs and item containers */
-	struct ao2_global_obj *global_obj;     /*!< The global object array that holds the user-defined config object */
+	aco_snapshot_alloc snapshot_alloc;	/*!< Allocate an object to hold all global configs and item containers */
+	struct ao2_global_obj *global_obj;	/*!< The global object array that holds the user-defined config object */
 	struct aco_info_internal *internal;
 	struct aco_file *files[];    /*!< An array of aco_files to process */
 };
diff --git a/include/asterisk/logger.h b/include/asterisk/logger.h
index 3b7823f2b8..c23d851b47 100644
--- a/include/asterisk/logger.h
+++ b/include/asterisk/logger.h
@@ -33,7 +33,7 @@ extern "C" {
 #endif
 
 #define EVENTLOG "event_log"
-#define	QUEUELOG	"queue_log"
+#define	QUEUELOG "queue_log"
 
 #define DEBUG_M(a) { \
 	a; \
@@ -55,7 +55,7 @@ extern "C" {
 #define AST_CALLID_BUFFER_LENGTH 13
 
 enum ast_logger_results {
-	AST_LOGGER_SUCCESS = 0, /*!< Log channel was created or deleted successfully*/
+	AST_LOGGER_SUCCESS = 0, /*!< Log channel was created or deleted successfully */
 	AST_LOGGER_FAILURE = 1, /*!< Log channel already exists for create or doesn't exist for deletion of log channel */
 	AST_LOGGER_DECLINE = -1, /*!< Log channel request was not accepted */
 	AST_LOGGER_ALLOC_ERROR = -2 /*!< filename allocation error */
@@ -68,7 +68,7 @@ enum ast_logger_results {
 	on which log you wish to output to. These are implemented as macros, that
 	will provide the function with the needed arguments.
 
- 	\param level	Type of log event
+	\param level	Type of log event
 	\param file	Will be provided by the AST_LOG_* macro
 	\param line	Will be provided by the AST_LOG_* macro
 	\param function	Will be provided by the AST_LOG_* macro
@@ -162,7 +162,9 @@ void __attribute__((format(printf, 5, 6))) ast_queue_log(const char *queuename,
  *
  * \details This works like ast_log, but prints verbose messages to the console depending on verbosity level set.
  *
+ * \code
  * ast_verbose(VERBOSE_PREFIX_3 "Whatever %s is happening\n", "nothing");
+ * \endcode
  *
  * This will print the message to the console if the verbose level is set to a level >= 3
  *
@@ -224,9 +226,9 @@ void ast_console_toggle_mute(int fd, int silent);
 
 /*!
  * \brief enables or disables logging of a specified level to the console
- * fd specifies the index of the console receiving the level change
- * level specifies the index of the logging level being toggled
- * state indicates whether logging will be on or off (0 for off, 1 for on)
+ * \param fd specifies the index of the console receiving the level change
+ * \param level specifies the index of the logging level being toggled
+ * \param state indicates whether logging will be on or off (0 for off, 1 for on)
  */
 void ast_console_toggle_loglevel(int fd, int level, int state);
 
@@ -365,7 +367,7 @@ const char *ast_logger_get_dateformat(void);
 ast_callid ast_create_callid(void);
 
 /*!
- * \brief extracts the callerid from the thread
+ * \brief extracts the callid from the thread
  *
  * \retval Non-zero Call id related to the thread
  * \retval 0 if no call_id is present in the thread
diff --git a/include/asterisk/refer.h b/include/asterisk/refer.h
index 10fb6ee716..2ec3d566a7 100644
--- a/include/asterisk/refer.h
+++ b/include/asterisk/refer.h
@@ -340,10 +340,6 @@ int ast_refer_notify_transfer_request(struct ast_channel *originating_chan, cons
 				      const char *protocol_id, struct ast_channel *dest, struct ast_refer_params *params,
 				      enum ast_control_transfer state);
 
-/*!
- *  @}
- */
-
 #if defined(__cplusplus) || defined(c_plusplus)
 }
 #endif
diff --git a/main/config_options.c b/main/config_options.c
index 12c3b49d63..631b0b567a 100644
--- a/main/config_options.c
+++ b/main/config_options.c
@@ -83,19 +83,19 @@ static struct ao2_container *xmldocs;
 
 /*! \brief Value of the aco_option_type enum as strings */
 static char *aco_option_type_string[] = {
-	"ACL",				/* OPT_ACL_T, */
-	"Boolean",			/* OPT_BOOL_T, */
-	"Boolean",			/* OPT_BOOLFLAG_T, */
-	"String",			/* OPT_CHAR_ARRAY_T, */
-	"Codec",			/* OPT_CODEC_T, */
-	"Custom",			/* OPT_CUSTOM_T, */
-	"Double",			/* OPT_DOUBLE_T, */
-	"Integer",			/* OPT_INT_T, */
-	"None",				/* OPT_NOOP_T, */
+	"ACL",			/* OPT_ACL_T, */
+	"Boolean",		/* OPT_BOOL_T, */
+	"Boolean",		/* OPT_BOOLFLAG_T, */
+	"String",		/* OPT_CHAR_ARRAY_T, */
+	"Codec",		/* OPT_CODEC_T, */
+	"Custom",		/* OPT_CUSTOM_T, */
+	"Double",		/* OPT_DOUBLE_T, */
+	"Integer",		/* OPT_INT_T, */
+	"None",			/* OPT_NOOP_T, */
 	"IP Address",		/* OPT_SOCKADDR_T, */
-	"String",			/* OPT_STRINGFIELD_T, */
+	"String",		/* OPT_STRINGFIELD_T, */
 	"Unsigned Integer",	/* OPT_UINT_T, */
-	"Boolean",			/* OPT_YESNO_T, */
+	"Boolean",		/* OPT_YESNO_T, */
 	"Time Length",		/* OPT_TIMELEN_T, */
 };
 #endif /* AST_XML_DOCS */
diff --git a/main/features.c b/main/features.c
index 6f9cab9359..ffa4c18e42 100644
--- a/main/features.c
+++ b/main/features.c
@@ -120,7 +120,7 @@
 							prefixed with one or two underbars ('_').</para>
 						</note>
 						<note>
-							<para>Using this option from a or GoSub() might not make sense as there would be no return points.</para>
+							<para>Using this option from a or Gosub() might not make sense as there would be no return points.</para>
 						</note>
 						<note>
 							<para>This option will override the 'x' option</para>
diff --git a/res/res_fax.c b/res/res_fax.c
index 551783d442..af51bad9a9 100644
--- a/res/res_fax.c
+++ b/res/res_fax.c
@@ -119,7 +119,7 @@
 		<description>
  			<para>This application is provided by res_fax, which is a FAX technology agnostic module
  			that utilizes FAX technology resource modules to complete a FAX transmission.</para>
- 			<para>Session arguments can be set by the FAXOPT function and to check results of the ReceiveFAX() application.</para>
+			<para>Session arguments can be set by the FAXOPT function and to check results of the ReceiveFAX() application.</para>
 		</description>
 		<see-also>
 			<ref type="function">FAXOPT</ref>
@@ -161,7 +161,7 @@
 		<description>
  			<para>This application is provided by res_fax, which is a FAX technology agnostic module
  			that utilizes FAX technology resource modules to complete a FAX transmission.</para>
- 			<para>Session arguments can be set by the FAXOPT function and to check results of the SendFAX() application.</para>
+			<para>Session arguments can be set by the FAXOPT function and to check results of the SendFAX() application.</para>
 		</description>
 		<see-also>
 			<ref type="function">FAXOPT</ref>
diff --git a/res/res_pjsip_session.c b/res/res_pjsip_session.c
index db5bdd1202..bca226c857 100644
--- a/res/res_pjsip_session.c
+++ b/res/res_pjsip_session.c
@@ -1836,8 +1836,8 @@ end:
  * \brief Merge media states for a delayed session refresh
  *
  * \param session_name For log messages
- * \param delayed_pending_state The pending media state at the time the resuest was queued
- * \param delayed_active_state The active media state  at the time the resuest was queued
+ * \param delayed_pending_state The pending media state at the time the request was queued
+ * \param delayed_active_state The active media state at the time the request was queued
  * \param current_active_state The current active media state
  * \param run_post_validation Whether to run validation on the resulting media state or not
  *