[MPlayer-cvslog] r34881 - in trunk: codec-cfg.c etc/codecs.conf

compn tempn at twmi.rr.com
Mon Apr 30 01:42:03 CEST 2012

On Sun, 29 Apr 2012 18:39:34 +0200 (CEST), cehoyos wrote:
>Author: cehoyos
>Date: Sun Apr 29 18:39:34 2012
>New Revision: 34881
>Support new colour spaces for FFmpeg camstudio decoding on big-endian.

i'm kinda confused this works. i thought we were just supposed to add
the base colorspace like BGR15 , and then BGR15BE or BGR15LE would be
chosen automatically?

is what i wrote about this incorrect in the docs?
please review it if so...

>+    {"BGR15LE",     IMGFMT_BGR15LE},

also why 'big-endian' commit message but 'LE' is added ?

>-  out BGR32,BGR24,BGR15,BGR8 query,flip
>+  out BGRA,BGR32,BGR24,BGR15LE,BGR15,BGR8 query,flip

you switched BGR32 to BGRA ? does the binary codec support that ?

p.s. you added BGR15LE to the binary codec, but not the ffcamtasia

p.p.s. the ffcamtasia codec doesnt need the 'query' flag i dont think.


More information about the MPlayer-cvslog mailing list