On Wed, 18 Sep 2013 15:30:26 +0200, Luca Barbato <[email protected]> wrote:
> Provide some information for every function and add a group.
> ---
>
> Please send updated patches instead of reviews.
> Note:
> @see is immediately after the body on purpose, do not move it down,
> looks much worse on the html output otherwise.
>
> libavutil/avutil.h | 6 ++++
> libavutil/log.h | 98
> ++++++++++++++++++++++++++++++++++++++++++++++++++----
> 2 files changed, 98 insertions(+), 6 deletions(-)
>
> diff --git a/libavutil/avutil.h b/libavutil/avutil.h
> index ddb34a1..6bb5510 100644
> --- a/libavutil/avutil.h
> +++ b/libavutil/avutil.h
> @@ -127,6 +127,12 @@
> *
> * @}
> *
> + * @defgroup lavu_log Logging Facility
> + *
> + * @{
> + *
> + * @}
> + *
> * @defgroup lavu_misc Other
> *
> * @{
> diff --git a/libavutil/log.h b/libavutil/log.h
> index 7b17330..5692bd5 100644
> --- a/libavutil/log.h
> +++ b/libavutil/log.h
> @@ -89,8 +89,19 @@ typedef struct AVClass {
> const struct AVClass* (*child_class_next)(const struct AVClass *prev);
> } AVClass;
>
> -/* av_log API */
> +/**
> + * @addtogroup lavu_log
> + *
> + * @{
> + *
> + * @defgroup lavu_log_constants Logging Constants
> + *
> + * @{
> + */
>
> +/**
> + * No output.
> + */
> #define AV_LOG_QUIET -8
>
> /**
> @@ -117,7 +128,14 @@ typedef struct AVClass {
> */
> #define AV_LOG_WARNING 24
>
> +/**
> + * Standard information
> + */
> #define AV_LOG_INFO 32
> +
> +/**
> + * Detailed information, usually not interesting.
The caller judges what is interesting or not, it's not for us to decide.
> + */
> #define AV_LOG_VERBOSE 40
>
> /**
> @@ -126,10 +144,15 @@ typedef struct AVClass {
> #define AV_LOG_DEBUG 48
>
> /**
> + * @}
> + */
> +
> +/**
> * Send the specified message to the log if the level is less than or equal
> * to the current av_log_level. By default, all logging messages are sent to
> - * stderr. This behavior can be altered by setting a different av_vlog
> callback
> + * stderr. This behavior can be altered by setting a different logging
> callback
> * function.
> + * @see av_log_set_callback
> *
> * @param avcl A pointer to an arbitrary struct of which the first field is a
> * pointer to an AVClass struct.
> @@ -137,15 +160,74 @@ typedef struct AVClass {
> * higher importance.
> * @param fmt The format string (printf-compatible) that specifies how
> * subsequent arguments are converted to output.
> - * @see av_vlog
> */
> void av_log(void *avcl, int level, const char *fmt, ...) av_printf_format(3,
> 4);
>
> -void av_vlog(void *avcl, int level, const char *fmt, va_list);
> +
> +/**
> + * Send the specified message to the log if the level is less than or equal
> + * to the current av_log_level. By default, all logging messages are sent to
> + * stderr. This behavior can be altered by setting a different logging
> callback
> + * function.
> + * @see av_log_set_callback
> + *
> + * @param avcl A pointer to an arbitrary struct of which the first field is a
> + * pointer to an AVClass struct.
> + * @param level The importance level of the message, lower values signifying
> + * higher importance.
It should mention that it should be one of the AV_LOG_ macros.
Same below
--
Anton Khirnov
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
