On 07 Mar 2014, at 15:36, Diego Biurrun <[email protected]> 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. For short lines like this, I like this kind of formatting too. Otherwise, make it a switch :-) Tim _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
