On Wed, Sep 21, 2016 at 08:56:58AM +0200, Anton Khirnov wrote:
> Quoting Vittorio Giovara (2016-09-20 23:12:02)
> > On Tue, Sep 20, 2016 at 3:07 PM, Anton Khirnov <an...@khirnov.net> wrote:
> > > Quoting Vittorio Giovara (2016-09-19 22:33:43)
> > >> --- a/libavcodec/rscc.c
> > >> +++ b/libavcodec/rscc.c
> > >> @@ -31,7 +31,7 @@
> > >> * and it can be deflated or not. Similarly, pixel data comes after the
> > >> header
> > >> * and a variable size value, and it can be deflated or just raw.
> > >> *
> > >> - * Supports: BGRA, BGR24, RGB555, RGB8
> > >> + * Supports: PAL8, BGRA, BGR24, RGB555, RGB8
> > >> */
> > >>
> > >> @@ -59,6 +59,8 @@ typedef struct RsccContext {
> > >> unsigned int tiles_size;
> > >> int component_size;
> > >>
> > >> + uint8_t palette[AVPALETTE_SIZE];
> > >
> > > What is this for?
> >
> > I think to track palette change under /* Palette handling */
>
> But it's not actually used in that block from what I can see.
And doesn't it generate an unused variable warning?
Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
