Quoting Diego Biurrun (2015-03-01 21:27:30)
> On Fri, Feb 27, 2015 at 01:00:57PM +0200, Martin Storsjö wrote:
> > Intentionally keeping some conditional statements on single lines
> > in rtpenc_h263.c.
> > ---
> >  libavformat/rtpenc.c      | 3 ++-
> >  libavformat/rtpenc_xiph.c | 6 ++++--
> >  2 files changed, 6 insertions(+), 3 deletions(-)
> 
> How about K&Ring the whole files instead?  Yes, I'm volunteering.
> 

Stab.

As I've said many times before -- complete whole-file reformatting
should be a last-resort measure applied to terrible unreadable files
(like ye olde days mpegvideo). We used to have many such files, so it
seems some of use have gotten used to doing it all the time, but
it's a very bad idea to keep doing this on files that are mostly fine.
It creates unnecessary noise in the history, is hard and annoying to
review properly and breaks outstanding work.

So -- don't.

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

Reply via email to