[FFmpeg-devel] [PATCH] add ME_T/ESA to avcodec.h

[Michael Niedermayer]

On Wed, Jun 04, 2008 at 01:05:20AM -0700, Baptiste Coudurier wrote:
> Jason Garrett-Glaser wrote:
> >>>      /**
> >>>       * Motion estimation algorithm used for video coding.
> >>>       * 1 (zero), 2 (full), 3 (log), 4 (phods), 5 (epzs), 6 (x1), 7 (hex),
> >>> -     * 8 (umh), 9 (iter) [7, 8 are x264 specific, 9 is snow specific]
> >>> +     * 8 (umh), 9 (iter) [7, 8, 10, 11 are x264 specific, 9 is snow specific]
> >>>       * - encoding: MUST be set by user.
> >>>       * - decoding: unused
> > 
> > Is there any reason to distinguish ESA from FULL?  Shouldn't the two
> > input strings just map to the same internal option?  Yes, x264's ESA
> > is actually Sequential Elimination (SEA), but given that is
> > mathematically equivalent to a full search, there is no possible
> > reason anyone would want to FULL if x264-style ESA was available, so
> > there is no good reason to imply that there is any real distinction
> > between FULL and ESA, I think.
> > 
> 
> Well, up to you guys, if you are comfortable with ME_FULL, Im ok too :)

Id just tell users to use FULL if they mean FULL ...

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Old school: Use the lowest level language in which you can solve the problem
            conveniently.
New school: Use the highest level language in which the latest supercomputer
            can solve the problem without the user falling asleep waiting.
-------------- 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-devel/attachments/20080604/02974608/attachment.pgp>