On Sat, Jul 26, 2014 at 02:23:51PM +0530, Nidhi Makhijani wrote:
> --- a/libavcodec/mpeg4video.h
> +++ b/libavcodec/mpeg4video.h
> @@ -92,6 +92,9 @@ typedef struct Mpeg4DecContext {
> int xvid_build;
> int lavc_build;
> + ///< does the stream contain the low_delay flag
> + int vol_control_parameters;
> +
> ///< flag for having shown the warning about divxs invalid b frames
> int showed_packed_warning;
The ///< syntax documents the struct member to the left (or above) of the
Doxygen comment. That's probably not what you intended. Yes, the usage
of ///< directly below is wrong.
> --- a/libavcodec/mpegvideo.h
> +++ b/libavcodec/mpegvideo.h
> @@ -518,7 +518,6 @@ typedef struct MpegEncContext {
> int low_delay; ///< no reordering needed / has no
> b-frames
> int vo_type;
> - int vol_control_parameters; ///< does the stream contain the
> low_delay flag, used to workaround buggy encoders
> PutBitContext tex_pb; ///< used for data partitioned VOPs
> PutBitContext pb2; ///< used for data partitioned VOPs
Don't truncate the comment you are moving.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
