On Fri, Nov 02, 2012 at 06:58:08PM +0100, Anton Khirnov wrote:
> 
> crc: move doxy to the header
> 
> --- a/libavutil/crc.c
> +++ b/libavutil/crc.c
> @@ -108,13 +87,6 @@ const AVCRC *av_crc_get_table(AVCRCId crc_id)
>  
> -/**
> - * Calculate the CRC of a block.
> - * @param crc CRC of previous blocks if any or initial value for CRC
> - * @return CRC updated with the data from the given block
> - *
> - * @see av_crc_init() "le" parameter
> - */
>  uint32_t av_crc(const AVCRC *ctx, uint32_t crc,
>                  const uint8_t *buffer, size_t length)
> --- a/libavutil/crc.h
> +++ b/libavutil/crc.h
> @@ -36,8 +36,38 @@ typedef enum {
> +/**
> + * Calculate the CRC of a block.
> + * @param crc CRC of previous blocks if any or initial value for CRC
> + * @return CRC updated with the data from the given block
> + *
> + * @see av_crc_init() "le" parameter
> + */
>  uint32_t av_crc(const AVCRC *ctx, uint32_t start_crc, const uint8_t *buffer, 
> size_t length) av_pure;

The names of the function parameters do not match between .c and .h file
(why does gcc not have an option to warn about this?), thus the Doxygen
in the header is now incorrect.

Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to