[FFmpeg-devel] [PATCH] RoQ muxer

[Diego Biurrun]

On Sun, May 06, 2007 at 01:07:45PM +0200, Vitor wrote:
> 

Since I'm sure this will go through another round of review anyway..

> --- libavformat/idroqenc.c	(revision 0)
> +++ libavformat/idroqenc.c	(revision 0)
> @@ -0,0 +1,181 @@
> +/*
> + * RoQ encoder packet format.  It's very alike raw, but is strict in the
> + *   interleaving of audio and video frames.

This is somehow very confusing.  At the top of a source file I just
expect a note regarding what the code is about, not a general comment on
a multimedia format.  Also, what is "RoQ encoder packet format"?  Do you
just mean "RoQ packet format encoder"?  Then it's redundant and should
just be "RoQ muxer" IMO.  Or is this the packet format part of the
encoder?

Diego