[MPlayer-dev-eng] Mplayer's licence (not a flame)

Mike Melanson melanson at pcisys.net
Tue Mar 19 17:10:25 CET 2002


On Tue, 19 Mar 2002, Arpi wrote:

> please add rgb 15/16 support too! asap...
> you already killed xa_rle8 wrapper :(

	It could still be reinstated by modifying no fewer than 5 files
(codec-cfg.c codec-cfg.h etc/codecs.conf libmpcodecs/vd.c
libmpcodecs/Makefile). This was my first foray into the new codec
subsystem. Really, not much has changed from a maintenance standpoint, I
realized. You still have to add a new file and then modify 5 other project
files, 2 of which are different than before. Still, I'll admit that it's
easier than having to modify dec_video.c.

> basic rule of cvs: never reduce features of the code - just fix or extend it.

	We have rules in this project? No way...

> it was requested by our sponsor, they have presentations in this format and
> they play it in 16bpp... silly but fact.

	All right, I'll take care of 15 & 16bpp (if I add one, the other
shouldn't be too hard). Just to be sure I have this straight:
  BGR15 = BGR555
  BGR16 = BGR565
Correct? And how do I test this? I know that I'll want to temporarily
modify my codecs.conf file to force one mode or the other for testing, but
will my x11 vo target accept the data? I would modify the PNG target for
testing, but there are no postproc routines for going from rgb15/16 ->
rgb24 (2 permutations that are missing).

	Thanks...
--
	-Mike Melanson




More information about the MPlayer-dev-eng mailing list