[FFmpeg-devel] Path for minor problems related to ffmpeg help text and x264 configuration

[Stefano Sabatini]

On date Thursday 2008-09-25 22:17:06 +0100, Robert Swain encoded:
> On 25 Sep 2008, at 20:43, Stefano Sabatini wrote:
> > On date Tuesday 2008-09-23 14:35:03 +0200, Francesco Cosoleto encoded:
[...] 
> >> Index: libavcodec/avcodec.h
> >> ===================================================================
> >> --- libavcodec/avcodec.h	(revisione 15383)
> >> +++ libavcodec/avcodec.h	(copia locale)
> >> @@ -2103,7 +2103,7 @@
> >> #define X264_PART_B8X8 0x100  /* Analyze b16x8, b8x16 and b8x8 */
> >>
> >>     /**
> >> -     * direct MV prediction mode - 0 (none), 1 (spatial), 2  
> >> (temporal)
> >> +     * direct MV prediction mode - 0 (none), 1 (spatial), 2  
> >> (temporal), 3 (auto)
> >>      * - encoding: Set by user.
> >>      * - decoding: unused
> >>      */
> >
> > Looks OK, but I prefer to leave it to someone else with expertise in
> > libx264.
> 
> Yes, that's OK. 
> >> Index: ffpresets/libx264-fastfirstpass.ffpreset
> >> ===================================================================
> >> --- ffpresets/libx264-fastfirstpass.ffpreset	(revisione 15383)
> >> +++ ffpresets/libx264-fastfirstpass.ffpreset	(copia locale)
> >> @@ -2,7 +2,7 @@
> >> flags=+loop
> >> cmp=+chroma
> >> partitions=-parti8x8-parti4x4-partp8x8-partp4x4-partb8x8
> >> -me=dia
> >> +me_method=dia
> >> subq=1
> >> me_range=16
> >> g=250
> >> Index: ffpresets/libx264-default.ffpreset
> >> ===================================================================
> >> --- ffpresets/libx264-default.ffpreset	(revisione 15383)
> >> +++ ffpresets/libx264-default.ffpreset	(copia locale)
> >> @@ -2,7 +2,7 @@
> >> flags=+loop
> >> cmp=+chroma
> >> partitions=+parti8x8+parti4x4+partp8x8+partb8x8
> >> -me=hex
> >> +me_method=hex
> >> subq=5
> >> me_range=16
> >> g=250
> >> Index: ffpresets/libx264-max.ffpreset
> >> ===================================================================
> >> --- ffpresets/libx264-max.ffpreset	(revisione 15383)
> >> +++ ffpresets/libx264-max.ffpreset	(copia locale)
> >> @@ -2,7 +2,7 @@
> >> flags=+loop
> >> cmp=+chroma
> >> partitions=+parti8x8+parti4x4+partp8x8+partp4x4+partb8x8
> >> -me=tesa
> >> +me_method=tesa
> >> subq=7
> >> me_range=32
> >> g=250
> >> Index: ffpresets/libx264-hq.ffpreset
> >> ===================================================================
> >> --- ffpresets/libx264-hq.ffpreset	(revisione 15383)
> >> +++ ffpresets/libx264-hq.ffpreset	(copia locale)
> >> @@ -2,7 +2,7 @@
> >> flags=+loop
> >> cmp=+chroma
> >> partitions=+parti8x8+parti4x4+partp8x8+partb8x8
> >> -me=umh
> >> +me_method=umh
> >> subq=7
> >> me_range=16
> >> g=250
> >> Index: ffpresets/libx264-normal.ffpreset
> >> ===================================================================
> >> --- ffpresets/libx264-normal.ffpreset	(revisione 15383)
> >> +++ ffpresets/libx264-normal.ffpreset	(copia locale)
> >> @@ -2,7 +2,7 @@
> >> flags=+loop
> >> cmp=+chroma
> >> partitions=+parti8x8+parti4x4+partp8x8+partb8x8
> >> -me=hex
> >> +me_method=hex
> >> subq=6
> >> me_range=16
> >> g=250
> >
> > This is surely correct, I'll apply in few days with the first one if
> > no one objects.
> 
> The above are OK, as long as they've been tested working. I hadn't  
> noticed the 'me' option had been removed. :)

Applied both.

[...]

Regards.
-- 
FFmpeg = Foolish & Fiendish Martial Purposeless Exxagerate Gadget