[gstreamer-bugs] [Bug 164268] [PATCH][ffmpegcolorspace] Fix BGR <-> RGB handling

[bugzilla-daemon at bugzilla.gnome.org]

Please DO NOT reply to this by email. All additional comments should be made in
the comments box of this bug report.

 http://bugzilla.gnome.org/show_bug.cgi?id=164268
 GStreamer | gst-plugins | Ver: HEAD CVS





------- Additional Comments From Gergely Nagy  2005-01-17 05:52 -------
If it does not support the rearranged masks, it probably doesn't want BGR, either.

gstreamer/docs/random/mimetypes says:

    24 and 32 bit RGB should always be specified as big endian, since any little
    endian format can be transformed into big endian by rearranging the color
    masks. 15 and 16 bit formats should generally have the same byte order as
    the CPU.

So, if one element wants BGR, it should support the appropriate masks. Or, it
might want 15/16bit format, which wasn't dealt with before, either. (RGB555 and
RGB565 was dealt with, BGR555/BGR565 was not, as far as I could see)

So far, I did not find any element that would be broken by this patch (mind you,
I haven't tested all possible combinations). Furthermore, I think that if this
breaks something, then the broken elements should be fixed to support the
appropriate (rearranged) masks.

------- You are receiving this mail because: -------
You are the assignee for the bug.
You are the QA contact for the bug.