[FFmpeg-devel] [PATCH] tools/qt-faststart: add note to use -movflags +faststart instead

[Lou Logan]

On Mon,  6 Jan 2014 11:49:21 -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");
>          return 0;
>      }
>  

A similar note upon normal usage should also be added, but I must
return to the non-coding cave and won't entertain you with my "skills".

As for why I added a note:

@ubitux | qt-faststart is not reliable
@ubitux | use -movflags +faststart
@ubitux | do you know how it locates the chunk offset atoms?
@ubitux | it's a memory search in the header
@ubitux | a memory search of 4 bytes
@ubitux | so yeah it's fast and has its benefits   
@ubitux | (like not altering the header)
@ubitux | but it's also a bit dangerous