[Ffmpeg-devel] [PATCH] ffplay document update

[Diego Biurrun]

On Wed, Feb 28, 2007 at 03:14:01AM +0100, Michael Niedermayer wrote:
> 
> On Tue, Feb 27, 2007 at 09:54:07AM +0800, Limin Wang wrote:
> > 
> > * Michael Niedermayer <michaelni at gmx.at> [2007-02-26 15:43:10 +0100]:
> > 
> > > On Mon, Feb 26, 2007 at 11:12:32AM +0800, Limin Wang wrote:
> > > > 
> > > > Some new options are added for ffplay, the patch try to update the
> > > > ffplay texi document. These options without any comments such as
> > > > -ast etc weren't add in. Maybe it's proper to add comments in code
> > > > first.
> > > > 
> > > > --- doc/ffplay-doc.texi	(revision 8129)
> > > > +++ doc/ffplay-doc.texi	(working copy)
> > > > @@ -35,10 +35,16 @@
> > > >  force displayed width
> > > >  @item -y height
> > > >  force displayed height
> > > > + at item -s size
> > > > +set frame size (WxH or abbreviation)
> > > 
> > > we should mention that this is specifically for raw video
> > 
> > How about the frame size option in ffmpeg? not only for raw video?
> > Any necessary to mention it in the hints of the command? 
> 
> what i meant but i didnt clearly say it is to write something like
> 
> set frame size (WxH or abbreviation), this is needed for videos which dont
> contain a header with the framesize like raw yuv

Applied along with suggestion.

Diego