[FFmpeg-devel] [PATCH v2] tools/qt-faststart: add -movflags +faststart note

[Lou Logan]

On Mon, 6 Jan 2014 22:15:50 +0100, Clément Bœsch wrote:

> On Mon, Jan 06, 2014 at 11:49:21AM -0900, Lou Logan wrote:
> > ---
> >  tools/qt-faststart.c | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/tools/qt-faststart.c b/tools/qt-faststart.c
> > index cb36c43..61bf93b 100644
> > --- a/tools/qt-faststart.c
> > +++ b/tools/qt-faststart.c
> > @@ -102,7 +102,8 @@ int main(int argc, char *argv[])
> >      int bytes_to_copy;
> >  
> >      if (argc != 3) {
> > -        printf("Usage: qt-faststart <infile.mov> <outfile.mov>\n");
> > +        printf("Usage: qt-faststart <infile.mov> <outfile.mov>\n"
> > +               "This tool has been deprecated by -movflags +faststart in ffmpeg\n");
> 
> I wouldn't say deprecated, but maybe just mention the feature.

Ok. I never did like that word anyway.

> maybe, "Note: ffmpeg can now do this operation using -movflags
> +faststart". As you quoted, the tool has its benefits (no need to worry
> about a partial header reconstruction, and probably faster).

I changed it to:
"Note: Alternatively you can use -movflags +faststart in ffmpeg"

Updated patch attached.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: qtfaststart_movflags_note_v2.patch
Type: text/x-patch
Size: 795 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140106/c2cc4bf3/attachment.bin>