[FFmpeg-cvslog] r12919 - trunk/ffmpeg.c

[Baptiste Coudurier]

Hi,

benoit wrote:
> Author: benoit
> Date: Mon Apr 21 10:59:38 2008
> New Revision: 12919
> 
> Log:
> Make some ffmpeg opts use parse-number-or-die.
> Patch by Stefano Sabatini stefano sabatini-lala posteit
> 
> 
> Modified:
>    trunk/ffmpeg.c
> 
> [...]
>  
>  static void opt_verbose(const char *arg)
>  {
> -    verbose = atoi(arg);
> +    verbose = parse_number_or_die("v", arg, OPT_INT64, AV_LOG_QUIET, AV_LOG_DEBUG);
>      av_log_set_level(verbose);
>  }
>  

This broke some ffmpeg verbose messages:
The value for v was 3 which is not within -1.000000 - 2.000000

While in ffmpeg.c:
if (verbose>2)
    fprintf(stderr, "*** drop!\n");

Also while this is related, would it be possible to not set av_log_level
when using verbose option (-v) ?
This way you can have ffmpeg binary debug output (audio/video sync),
while not having decoder/encoder debug output.
Decoder/encoder debug output can still be enabled with -debug.

How does this sound ?

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
SMARTJOG S.A.                                    http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
Phone: +33 1 49966312