[MPlayer-dev-eng] [PATCH] GIF support, release quality?

Arpi arpi at thot.banki.hu
Wed Jan 29 23:42:18 CET 2003


Hi,

> On Wed, Jan 29, 2003 at 11:09:02PM +0100, Arpi wrote:
> > > Yes, except that the format is still BGR8.  lavc and scale can be loaded
> > why?
> 
> Because paletted BGR8 still uses IMGFMT_BGR8.
> When vo sees BGR8, it assumes it's not indexed, but packed 1 byte ber pixel.
> This is not always true.

heh?
BGR8 is always indexed, which means it has palette (or if it has no palette
it's assumed grayscale)

> > > based on requested format or format incompatibilities.  How should
> > > mpcodecs_config_vo() be notified of the need to load vf_palette?
> > > vf_palette checks for a palette by checking for the number of planes in
> > > the mpi it gets, but mpi has not been built when mpcodecs_config_vo() is
> 
> > it has NOTHING with the palette or number of colors.
> > if the colorspace is BGR8 and the filter chain/vo driver doesn't accept BGR8
> > then you have to instert vf_palette. easy.
> 
> I'm sorry, but you are wrong about this.
no

> If the vo supports BGR8, but your video is BGR8 w/ palette, then you
> will see garbage on the screen without vf_palette.  There is not a

why?

> single vo available that takes note of a palette in plane 1.

bad

afaik only vo_svga and vo_ggi supports BGR8, and both (should) use the palette.
otherwise it has no sense of supporting bgr8, as you said it would show garbage.

> > because if the vo accepts BGR8, then inserting vf_palette is worse than
> > nothing.
> 
> No, if vo accepts BGR8 and ignores the palette, you see garbage.

Sure. But i don't think that any vo behaves so.
if yes, it means that _vo_ is broken.

> 2) make all vo's supporting BGR8 take notice of palette
yes


A'rpi / Astral & ESP-team

--
Developer of MPlayer, the Movie Player for Linux - http://www.MPlayerHQ.hu
    "However, many people beg for its inclusion in Debian. Why?" - Gabucino
  "Because having new software in Debian is good." - Josselin Mouette
"Because having good software in Debian is new." - Gabucino


More information about the MPlayer-dev-eng mailing list