[MPlayer-dev-eng] [PATCH] remove yuv4mpeg RGB support

Reimar Döffinger Reimar.Doeffinger at gmx.de
Thu Jun 3 21:59:13 CEST 2010


Hello,
in addition to to getting rid of a huge amount of inefficient and strange
code, I think recommending the proper scale filter syntax is more reliable,
going e.g. YUV 4:4:4 -> RGB24 -> YV12 for interlaced probably will not exactly
give the best quality, even more so if we somewhen add proper support for different
YUV variants.
I admit I haven't really verified if the scale filter does the proper interlaced chroma
handling, but it looks to me like it does (does libswscale offer that functionality?
Seems not so great that we have this in vf_scale, IMO it belongs into libswscale).
Any opinions? Does anyone even care about interlaced non-YV12 formats?
And yes, the previous disabling hack left a huge amount of dead code in that file.

Reimar
-------------- next part --------------
A non-text attachment was scrubbed...
Name: yuv4mpegrgb.diff
Type: text/x-diff
Size: 9782 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20100603/8d689eb6/attachment.diff>


More information about the MPlayer-dev-eng mailing list