[FFmpeg-devel] [PATCH] Playlist API

Diego Biurrun diego
Fri Aug 7 00:30:05 CEST 2009


On Thu, Aug 06, 2009 at 12:35:35PM -0700, Geza Kovacs wrote:
> 
> This updated patch implements seeking over the entire playlist. As I
> stated in my previous message, the API is able to handle playlist items
> with differing numbers of streams.

Please find all the opportunities to prettyprint your code yourself.
You could also be a tad more generous with whitespace around operators
and sometimes break some long lines.

> diff --git a/libavformat/Makefile b/libavformat/Makefile
> index 7838efe..2377365 100644
> --- a/libavformat/Makefile
> +++ b/libavformat/Makefile
> @@ -33,6 +33,7 @@ OBJS-$(CONFIG_BETHSOFTVID_DEMUXER)       += bethsoftvid.o
>  OBJS-$(CONFIG_CAVSVIDEO_DEMUXER)         += raw.o
> +OBJS-$(CONFIG_CONCAT_DEMUXER)            += m3u.o pls.o xspf.o playlist.o concat.o concatgen.o

The other demuxers cannot be compiled standalone?  This smells of
insufficient separation.

Some of the #ifdefs in your files are probably unnecessary.

This is missing Changelog and documentation updates.

Diego



More information about the ffmpeg-devel mailing list