diff mbox

[alsa-lib,PATCHv2] ucm: docs: typeset lists of identifiers explicitly

Message ID 20160923161857.26573-1-ao2@ao2.it (mailing list archive)
State Mainlined
Headers show

Commit Message

Antonio Ospite Sept. 23, 2016, 4:18 p.m. UTC
Doxygen doesn't preserve formatting that relies only on indentation,
typeset lists of identifiers explicitly.

This makes the HTML docs a lot more readable.

This change comes along the lines of commit 72aa0f8332fb ("ucm: reformat
snd_use_case_get() doc").

Some TABs has been added too in order to preserve the aligned look when
reading the source code.

Signed-off-by: Antonio Ospite <ao2@ao2.it>
---

Changes since v1:

  * fix a typo where a '=' (equal) was used instead of a '-' (minus) at the
    start of one line.

Thanks,
   Antonio

 include/use-case.h | 57 +++++++++++++++++++++++++++---------------------------
 1 file changed, 29 insertions(+), 28 deletions(-)

Comments

Takashi Iwai Sept. 29, 2016, 7:57 a.m. UTC | #1
On Fri, 23 Sep 2016 18:18:57 +0200,
Antonio Ospite wrote:
> 
> Doxygen doesn't preserve formatting that relies only on indentation,
> typeset lists of identifiers explicitly.
> 
> This makes the HTML docs a lot more readable.
> 
> This change comes along the lines of commit 72aa0f8332fb ("ucm: reformat
> snd_use_case_get() doc").
> 
> Some TABs has been added too in order to preserve the aligned look when
> reading the source code.
> 
> Signed-off-by: Antonio Ospite <ao2@ao2.it>
> ---
> 
> Changes since v1:
> 
>   * fix a typo where a '=' (equal) was used instead of a '-' (minus) at the
>     start of one line.

Applied, thanks.


Takashi


> 
> Thanks,
>    Antonio
> 
>  include/use-case.h | 57 +++++++++++++++++++++++++++---------------------------
>  1 file changed, 29 insertions(+), 28 deletions(-)
> 
> diff --git a/include/use-case.h b/include/use-case.h
> index 9aac6e2..8911645 100644
> --- a/include/use-case.h
> +++ b/include/use-case.h
> @@ -192,20 +192,21 @@ int snd_use_case_free_list(const char *list[], int items);
>   * \return Number of list entries if success, otherwise a negative error code
>   *
>   * Defined identifiers:
> - *   NULL 		- get card list
> - *			  (in pair cardname+comment)
> - *   _verbs		- get verb list
> - *			  (in pair verb+comment)
> - *   _devices[/{verb}]	- get list of supported devices
> - *			  (in pair device+comment)
> - *   _modifiers[/{verb}]- get list of supported modifiers
> - *			  (in pair modifier+comment)
> - *   TQ[/{verb}]	- get list of TQ identifiers
> - *   _enadevs		- get list of enabled devices
> - *   _enamods		- get list of enabled modifiers
> + *   - NULL			- get card list
> + *				 (in pair cardname+comment)
> + *   - _verbs			- get verb list
> + *				  (in pair verb+comment)
> + *   - _devices[/{verb}]	- get list of supported devices
> + *				  (in pair device+comment)
> + *   - _modifiers[/{verb}]	- get list of supported modifiers
> + *				  (in pair modifier+comment)
> + *   - TQ[/{verb}]		- get list of TQ identifiers
> + *   - _enadevs			- get list of enabled devices
> + *   - _enamods			- get list of enabled modifiers
> + *
> + *   - _supporteddevs/{modifier}|{device}[/{verb}]   - list of supported devices
> + *   - _conflictingdevs/{modifier}|{device}[/{verb}] - list of conflicting devices
>   *
> - *   _supporteddevs/{modifier}|{device}[/{verb}]   - list of supported devices
> - *   _conflictingdevs/{modifier}|{device}[/{verb}] - list of conflicting devices
>   *   Note that at most one of the supported/conflicting devs lists has
>   *   any entries, and when neither is present, all devices are supported.
>   *
> @@ -331,8 +332,8 @@ int snd_use_case_get(snd_use_case_mgr_t *uc_mgr,
>   * \return Zero if success, otherwise a negative error code
>   *
>   * Known identifiers:
> - *   _devstatus/{device}	- return status for given device
> - *   _modstatus/{modifier}	- return status for given modifier
> + *   - _devstatus/{device}	- return status for given device
> + *   - _modstatus/{modifier}	- return status for given modifier
>   */
>  int snd_use_case_geti(snd_use_case_mgr_t *uc_mgr,
>  		      const char *identifier,
> @@ -346,19 +347,19 @@ int snd_use_case_geti(snd_use_case_mgr_t *uc_mgr,
>   * \return Zero if success, otherwise a negative error code
>   *
>   * Known identifiers:
> - *   _verb 		- set current verb = value
> - *   _enadev		- enable given device = value
> - *   _disdev		- disable given device = value
> - *   _swdev/{old_device} - new_device = value
> - *			- disable old_device and then enable new_device
> - *			- if old_device is not enabled just return
> - *			- check transmit sequence firstly
> - *   _enamod		- enable given modifier = value
> - *   _dismod		- disable given modifier = value
> - *   _swmod/{old_modifier} - new_modifier = value
> - *			- disable old_modifier and then enable new_modifier
> - *			- if old_modifier is not enabled just return
> - *			- check transmit sequence firstly
> + *   - _verb			- set current verb = value
> + *   - _enadev			- enable given device = value
> + *   - _disdev			- disable given device = value
> + *   - _swdev/{old_device}	- new_device = value
> + *				  - disable old_device and then enable new_device
> + *				  - if old_device is not enabled just return
> + *				  - check transmit sequence firstly
> + *   - _enamod			- enable given modifier = value
> + *   - _dismod			- disable given modifier = value
> + *   - _swmod/{old_modifier}	- new_modifier = value
> + *				  - disable old_modifier and then enable new_modifier
> + *				  - if old_modifier is not enabled just return
> + *				  - check transmit sequence firstly
>   */
>  int snd_use_case_set(snd_use_case_mgr_t *uc_mgr,
>                       const char *identifier,
> -- 
> 2.9.3
>
diff mbox

Patch

diff --git a/include/use-case.h b/include/use-case.h
index 9aac6e2..8911645 100644
--- a/include/use-case.h
+++ b/include/use-case.h
@@ -192,20 +192,21 @@  int snd_use_case_free_list(const char *list[], int items);
  * \return Number of list entries if success, otherwise a negative error code
  *
  * Defined identifiers:
- *   NULL 		- get card list
- *			  (in pair cardname+comment)
- *   _verbs		- get verb list
- *			  (in pair verb+comment)
- *   _devices[/{verb}]	- get list of supported devices
- *			  (in pair device+comment)
- *   _modifiers[/{verb}]- get list of supported modifiers
- *			  (in pair modifier+comment)
- *   TQ[/{verb}]	- get list of TQ identifiers
- *   _enadevs		- get list of enabled devices
- *   _enamods		- get list of enabled modifiers
+ *   - NULL			- get card list
+ *				 (in pair cardname+comment)
+ *   - _verbs			- get verb list
+ *				  (in pair verb+comment)
+ *   - _devices[/{verb}]	- get list of supported devices
+ *				  (in pair device+comment)
+ *   - _modifiers[/{verb}]	- get list of supported modifiers
+ *				  (in pair modifier+comment)
+ *   - TQ[/{verb}]		- get list of TQ identifiers
+ *   - _enadevs			- get list of enabled devices
+ *   - _enamods			- get list of enabled modifiers
+ *
+ *   - _supporteddevs/{modifier}|{device}[/{verb}]   - list of supported devices
+ *   - _conflictingdevs/{modifier}|{device}[/{verb}] - list of conflicting devices
  *
- *   _supporteddevs/{modifier}|{device}[/{verb}]   - list of supported devices
- *   _conflictingdevs/{modifier}|{device}[/{verb}] - list of conflicting devices
  *   Note that at most one of the supported/conflicting devs lists has
  *   any entries, and when neither is present, all devices are supported.
  *
@@ -331,8 +332,8 @@  int snd_use_case_get(snd_use_case_mgr_t *uc_mgr,
  * \return Zero if success, otherwise a negative error code
  *
  * Known identifiers:
- *   _devstatus/{device}	- return status for given device
- *   _modstatus/{modifier}	- return status for given modifier
+ *   - _devstatus/{device}	- return status for given device
+ *   - _modstatus/{modifier}	- return status for given modifier
  */
 int snd_use_case_geti(snd_use_case_mgr_t *uc_mgr,
 		      const char *identifier,
@@ -346,19 +347,19 @@  int snd_use_case_geti(snd_use_case_mgr_t *uc_mgr,
  * \return Zero if success, otherwise a negative error code
  *
  * Known identifiers:
- *   _verb 		- set current verb = value
- *   _enadev		- enable given device = value
- *   _disdev		- disable given device = value
- *   _swdev/{old_device} - new_device = value
- *			- disable old_device and then enable new_device
- *			- if old_device is not enabled just return
- *			- check transmit sequence firstly
- *   _enamod		- enable given modifier = value
- *   _dismod		- disable given modifier = value
- *   _swmod/{old_modifier} - new_modifier = value
- *			- disable old_modifier and then enable new_modifier
- *			- if old_modifier is not enabled just return
- *			- check transmit sequence firstly
+ *   - _verb			- set current verb = value
+ *   - _enadev			- enable given device = value
+ *   - _disdev			- disable given device = value
+ *   - _swdev/{old_device}	- new_device = value
+ *				  - disable old_device and then enable new_device
+ *				  - if old_device is not enabled just return
+ *				  - check transmit sequence firstly
+ *   - _enamod			- enable given modifier = value
+ *   - _dismod			- disable given modifier = value
+ *   - _swmod/{old_modifier}	- new_modifier = value
+ *				  - disable old_modifier and then enable new_modifier
+ *				  - if old_modifier is not enabled just return
+ *				  - check transmit sequence firstly
  */
 int snd_use_case_set(snd_use_case_mgr_t *uc_mgr,
                      const char *identifier,