[MPlayer-dev-eng] [PATCH] decoder and encoder support for DivX 5.0.5 API
Arpi
arpi at thot.banki.hu
Fri Jun 6 22:03:44 CEST 2003
Hi,
> I've found some time to implement the encoding support for the new
> DivX API. Now it's possible to play and encode movies with the
> latest DivX release.
> One thing that doesn't work is the new Video Buffer Verifier (VBV)
> multipass encoding. The encoder segfaults. Maybe it just isn't
> supported with the standard profile of the released binary encoder.
nice work, thanks!
i've commited with 2 small changes:
ve_divx4 added param "mpeg2" for old divx, but it doesn't have (at least my
5.0.2), removed. and it renamed param psychovisual to pv, it is somewhat ok,
nut is against our cvs policy point 4:
4. Do not change behavior of the program (renaming options etc) without
discussing it first at the mplayer-dev-eng mailing list. Do not remove
functionality from the code. Just improve!
Do not commit changes to the build system (Makefiles, configure script)
which change behaviour, defaults etc, without asking (and your change being
accepted) on the mplayer-dev-eng mailing list first.
such change requires changing users' mind and the documentation :)
(decide which takes longer :))
the other part was in vd_odivx.c, moving get_image call up and using
mpi->width. it's incorrect when IMGTYPE_EXPORT is used, because it means all
params (including stride, width) is provided by the codec (so codec=>mpi and
not mpi=>codec). i've changed it to sh->disp_w.
(it would be inetersting to use stride hints from next filter/vo, but it's
not implemenetd in mplayer-g1 architecture).
A'rpi / Astral & ESP-team
--
Developer of MPlayer G2, the Movie Framework for all - http://www.MPlayerHQ.hu
More information about the MPlayer-dev-eng
mailing list