[Ffmpeg-devel] Changing "-vstats" option behaviour

[Stefano Sabatini]

On date Tuesday 2007-04-17 17:28:50 +0200, Michael Niedermayer encoded:
> the externally vissible options are functional yes, the variable names are
> not, renaming them is fine if it makes sense but in a seperate patch
> 
> 
> [...]
> >      if (enc->codec_type == CODEC_TYPE_VIDEO) {
> >          frame_number = ost->frame_number;
> > -        fprintf(fvstats, "frame= %5d q= %2.1f ", frame_number, enc->coded_frame->quality/(float)FF_QP2LAMBDA);
> > +        fprintf(vstats_file, "frame= %5d q= %2.1f ", frame_number, enc->coded_frame->quality/(float)FF_QP2LAMBDA);
> 
> cosmetic
> 
> 
> [...]
> 
> > @@ -3449,6 +3432,15 @@
> >      }
> >  }
> >  
> > +static void opt_vstats_file (const char *arg)
> > +{
> > +    vstats_file = fopen(arg ,"w");
> > +    if (!vstats_file) {
> > +	perror("fopen");
> > +	exit(1);
> > +    }
> 
> tabs are forbidden in svn

Hope this is OK.

Cheers
-- 
Stefano Sabatini
Linux user number 337176 (see http://counter.li.org)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: vstats_file.patch
Type: text/x-diff
Size: 3725 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070417/31d69bb1/attachment.patch>