[FFmpeg-devel] [PATCH] LATM Bitstream Filter & Parser

Diego Biurrun diego
Mon May 25 12:12:04 CEST 2009


On Mon, May 25, 2009 at 09:12:46PM +1200, Paul Kendall wrote:
> 
> Comments please and I'll try to get them done ASAP.

Just nocticed two small nits.

> --- libavcodec/latm_bsf.c	(revision 0)
> +++ libavcodec/latm_bsf.c	(revision 0)
> @@ -0,0 +1,319 @@
> +    
> +    if (audio_object_type == AOT_AAC_SCALABLE
> +            || audio_object_type == AOT_ER_AAC_SCALABLE)

If you place the || on the provious line, you can align this nicely.

> --- libavcodec/latm_parser.c	(revision 0)
> +++ libavcodec/latm_parser.c	(revision 0)
> @@ -0,0 +1,108 @@
> +
> +/**
> + * finds the end of the current frame in the bitstream.

Capitalize.

Diego



More information about the ffmpeg-devel mailing list