[FFmpeg-devel] [FFMPEG] [PATCH] cavs encoder

Michael Niedermayer michaelni
Tue Dec 8 19:46:03 CET 2009


On Sat, Nov 21, 2009 at 07:25:33AM +0800, zhihang wang wrote:
> > *On my first attempt of an encoder I decided to put my own motion
> > estimation code in, even if it was a very simple one.
> > The code in motion_est_template.c and motion_est.c seems to have
> > some unsuitable assumptions:
> > - that a skipped macroblock has motion vector (0,0)
> > - that qpel needs different filter functions
> > - how the chroma vector is calculated from luma (in qpel case and else)
> > - that mid_pred is used for mv prediction
> > - and maybe more
> > Even if not all of above is correct I think it would be very difficult
> > to extend this optimized-for-MPEG-type-video code to a more generic
> > handling without having a speed regression. But I do certainly agree
> > that H.264 and AVS encoders could share a lot of motion estimation
> > code if that were to be written.*
> >
> Yes. Need I split the me mode into a common file just  like motion_est.c to
> enable h264 and AVS share the same me code? 

yes


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

it is not once nor twice but times without number that the same ideas make
their appearance in the world. -- Aristotle
-------------- 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/20091208/148e5e1b/attachment.pgp>



More information about the ffmpeg-devel mailing list