Hi,

On Sat, Jul 16, 2011 at 3:05 AM, Kostya <[email protected]> wrote:
> On Sat, Jul 16, 2011 at 06:53:43AM +0200, Kostya wrote:
>> Since it would take Diego too much time to finish his work, here's working
>> Flash Screen Video 2 decoder which decodes frames properly.
>
> Here's a version updated for the latest Git revision.
[..]
> +static int decode_hybrid(const uint8_t *sptr, uint8_t *dptr, int dx, int dy,
> +                         int h, int w, int stride, const uint32_t *pal)

One thing I don't like about this function is that it reads 15bpp data
but outputs it as if it were 24bpp... Since the palette itself is also
just 7bpp, I wonder if we shouldn't make the decoder output in a
different bpp...

The code I mostly reviewed before and LGTM.

Ronald
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to