[Ffmpeg-devel] Re: [PATCH] DVCPRO50 support

[Roman Shaposhnik]

On Mon, Mar 06, 2006 at 01:01:28PM +0100, Michael Niedermayer wrote:
> > Sorry about that... I wasn't sure whether or not to re-indent the big
> > blocks that got surrounded by new for() loops :). This should be the
> > last such change from me. Just set n_difchan=4 and you've got DV100
> > (well, almost :).
> 
> they should be reindented but in a seperate cosmetic only patch
> (for files maintained by me i would have rejected the cosmetic-functional
> mix)

  I'm just curious, would you actually approve a patch lacking proper 
  indentation ? Something that basically adds an outer if or for 
  statement around large existing chunks of code without reindenting
  them?

  I mean, after all, things like:

  if (cond) {
  existing code
  which wasn't 
  reindented
  } else {
    small chunk
    of new properly
    indented code
  }

  look rather ugly.

  Again -- as a reviewer I was presented with a need to filter these
  indentation changes out to focus on the actual guts of the patch.
  
  diff -ub was right on the money and I felt like sharing this "stupid trick"
  ;-)
  
Thanks,
Roman.