Roman V. Shaposhnik
Thu Feb 5 01:41:18 CET 2009
On Wed, 2009-02-04 at 23:00 +0100, Michael Niedermayer wrote:
> > Comments are welcome.
> see vsad_intra16_c() and ildct_cmp
Thanks, from the first glance these look exactly like what I need.
Although I'm confused about one thing: ildct_cmp is only set
in MPV_encode_init, shouldn't it be initialized for all the
> these would need to be extended with a width=8 intra variant, arrays would
> need to be enlarged to  ...
What's the convention for array indexes? IOW, how's one to know that
a width=8 intra variant supposed to be @ ?
Also, I'm a bit confused about one thing: if I only implement
width=8 intra for vsad/vsse case and the user picks a different
class via -ildctcmp, should I just issue an error message
"non implemented" or something?
> also supporting full RD would be nice, that is encoding both and picking
> the actually better instead of some heuristic
Sure. But wouldn't that be horrendously slow?
More information about the ffmpeg-devel