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

[zhihang wang]

Thank you very much for your comments. I have 2 questions.

>
> [...]
> > +/**
> > + * mapping the mb type to the entropy code
> > + */
> > +static int mbtype_to_code(AVSContext *h, enum cavs_block blk_sz, int
> best_ref[4])
> > +{
> > +    MpegEncContext* s = &h->s;
>
> > +    const int dir2offset[3][3] = {
>
> id make that static const
>
> *What's the advantage of using static const here?

*

> we already have ME  code, so this could be considered duplicated
> iam against having seperate ME code in every encoder (h264, avs and mpeg*)
>
> *Yes. there are ME code in ffmpeg. But perhaps it is difficult to use for
264 or avs because of using RDO.
Let me check again.* * If you have some idea, welcome your help!*
-- 
Best Regards
zhihang wang(???)