On Fri, Mar 07, 2014 at 03:34:33PM +0100, Kostya Shishkov wrote:
> On Fri, Mar 07, 2014 at 03:31:12PM +0100, Diego Biurrun wrote:
> > --- a/libavcodec/g2meet.c
> > +++ b/libavcodec/g2meet.c
> > @@ -320,10 +320,14 @@ static void kempf_restore_buf(const uint8_t *src, int 
> > len,
> >  
> > -    if (npal <= 2)       nb = 1;
> > -    else if (npal <= 4)  nb = 2;
> > -    else if (npal <= 16) nb = 4;
> > -    else                 nb = 8;
> > +    if (npal <= 2)
> > +        nb = 1;
> > +    else if (npal <= 4)
> > +        nb = 2;
> > +    else if (npal <= 16)
> > +        nb = 4;
> > +    else
> > +        nb = 8;
> 
> I'd rather leave it as it - it was kinda tabular form that makes it easier to
> undertand IMO.

I consider it just idiosyncratic formatting.  If you prefer I'll leave
it as-is.  Patch OK with or without this hunk?

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

Reply via email to