[Ffmpeg-cvslog] r7227 - trunk/libavcodec/dsputil.h

Måns Rullgård mru
Tue Dec 5 23:00:36 CET 2006


takis <subversion at mplayerhq.hu> writes:

> Author: takis
> Date: Tue Dec  5 22:57:31 2006
> New Revision: 7227
>
> Modified:
>    trunk/libavcodec/dsputil.h
>
> Log:
> Add comment denoting group of H.264 functions
>
> Modified: trunk/libavcodec/dsputil.h
> ==============================================================================
> --- trunk/libavcodec/dsputil.h	(original)
> +++ trunk/libavcodec/dsputil.h	Tue Dec  5 22:57:31 2006
> @@ -378,6 +378,7 @@
>  #define BASIS_SHIFT 16
>  #define RECON_SHIFT 6
>
> +    /* h264 functions */
>      void (*h264_idct_add)(uint8_t *dst, DCTELEM *block, int stride);
>      void (*h264_idct8_add)(uint8_t *dst, DCTELEM *block, int stride);
>      void (*h264_idct_dc_add)(uint8_t *dst, DCTELEM *block, int stride);

Isn't it fairly obvious from the names of those functions that they
deal with H.264?  Redundant comments are IMHO only slightly worse than
incorrect comments.

-- 
M?ns Rullg?rd
mru at inprovide.com




More information about the ffmpeg-cvslog mailing list