[MPlayer-dev-eng] [PATCH] synch to new x264 revision

Guillaume POIRIER guillaume.poirier at ifsic.univ-rennes1.fr
Sat Sep 25 19:14:05 CEST 2004


Le sam 25/09/2004 à 18:24, Diego Biurrun a écrit :
> Loren Merritt writes:
> > 
> >  .TP
> > -.B iframe=<value>
> > -I frame frequency (default: 60)
> > +.B keyint=<value>
> > +maximum interval between I-Frames
> > +Larger values save bits at the cost of seeking precision. (default: 150)
> 
> Nit: For consistency the period should be after the parentheses.

That's fixed on the CVS.


> > +.B frameref=<1\-15>
> > +Number of previous frames used as predictors in a P-frame (default: 1).
> > +This is effective in Anime, but seems to make little difference in 
> > +live-action source material.
> > +Some decoders are unable to deal with large frameref values.
> 
> Guillaume already committed this paragraph, you should remove it from
> your patch.

The goal of my patch was to have good descriptions based on Loren's
feedbacks ASAP, which is now done. Whenever the front-end will get
changed, only the options names will have to be changed.


> > -    {"iframe", &iframe, CONF_TYPE_INT, CONF_RANGE, 1, 24000000, NULL},
> > -    {"idrframe", &idrframe, CONF_TYPE_INT, CONF_RANGE, 1, 24000000, NULL},
> > +    {"keyint", &iframe, CONF_TYPE_INT, CONF_RANGE, 1, 24000000, NULL},
> > +    {"idrint", &idrframe, CONF_TYPE_INT, CONF_RANGE, 1, 24000000, NULL},
> 
> IMO (others may disagree) you should rename the variables as well,
> having variable names and option names differ is a recipe for code
> obfuscation IMO.

I agree with that too (even if others don't care about what I may think
on that field ;-).




More information about the MPlayer-dev-eng mailing list