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

[Michael Niedermayer]

On Sun, Apr 27, 2008 at 11:10:35PM +0200, Baptiste Coudurier wrote:
> 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 ?

Ive no strong oppinion on this either way, so if you prefer it feel free
to change it.

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The misfortune of the wise is better than the prosperity of the fool.
-- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20080428/1e189535/attachment.pgp>