On Thu, Apr 02, 2015 at 02:10:05PM +0200, Vittorio Giovara wrote:
> On Tue, Mar 31, 2015 at 4:31 PM, Diego Biurrun <[email protected]> wrote:
> > From: Kostya Shishkov <[email protected]>
> > --- /dev/null
> > +++ b/libavcodec/elsdec.c
> > +    { -1,  -72, 169, 145 },
> > +    { -6,   -5, 168,  49 },
> > +    {  0, -108, 171, 171 },
> > +    {  0, -108, 172, 172 },
> > +    { -6,   -5, 173, 173 }
> 
> nit: comma at the end
> 
> > --- a/libavcodec/g2meet.c
> > +++ b/libavcodec/g2meet.c
> >  typedef struct G2MContext {
> > -    JPGContext jc;
> > +    ePICContext ec;
> > +    JPGContext  jc;
> 
> imho needless whitespace
> 
> > @@ -179,7 +223,7 @@ static void jpg_unescape(const uint8_t *src, int 
> > src_size,
> >                           uint8_t *dst, int *dst_size)
> >  {
> >      const uint8_t *src_end = src + src_size;
> > -    uint8_t *dst_start = dst;
> > +    uint8_t *dst_start     = dst;
> 
> unrelated?
> 
> > @@ -663,7 +1372,7 @@ static int g2m_decode_frame(AVCodecContext *avctx, 
> > void *data,
> >                              int *got_picture_ptr, AVPacket *avpkt)
> >  {
> >      const uint8_t *buf = avpkt->data;
> > -    int buf_size = avpkt->size;
> > +    int buf_size       = avpkt->size;
> 
> unrelated?

All fixed locally.  I'll look at the other suggestions tomorrow.

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

Reply via email to