[MPlayer-cvslog] r26198 - in trunk: Changelog DOCS/man/en/mplayer.1 libmpcodecs/vd_ffmpeg.c libmpcodecs/ve_lavc.c

[Reimar Döffinger]

On Sat, Mar 08, 2008 at 08:29:34PM +0100, Diego Biurrun wrote:
> On Fri, Mar 07, 2008 at 11:28:08PM +0100, Reimar Döffinger wrote:
> > not to mention there is no reason to make it excessively hard to
> > re-enable this.
> 
> It's no harder than using
> 
> svn merge -r 26198:26197 .
> 
> in other words, extremely simple.

This assumes that the merge will even work at all, which is not that
likely in the far future.

> > In the future please send patches before you remove features of which
> > you have no clue what they actually do!
> 
> http://lists.mplayerhq.hu/pipermail/mplayer-cvslog/2008-March/033520.html

Hm. I did not even read that one. I assumed it would only update the
documentation (yeah, lack of proper reading).

> I got confused by what Uoti said there (and on IRC), so please enlighten
> me now: What else needs to be updated apart from the manual page?

Actually (AFAICT) only the documentation must be updated to say
something like "change config.h to enable for MPEG(1,2,ASP,AVC)".
Possibly giving the codecs that support it:
4xm, asv1, dnxhd, h264, mpeg, huffyuv, indeo3, mdec, svq1, svq3, vc1,
vp3/theora, wmv2, MPEG-2 via xvmc.
Of course, ideally all of these would be tested and any bugs fixed ;-)