On Fri, Mar 07, 2014 at 03:36:30PM +0100, Diego Biurrun wrote: > 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?
without _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
