[PATCH 1/2] Indent code before next channge

Douglas E Engert deengert at gmail.com
Wed Jun 30 22:37:49 AEST 2021


Three other options:
  With diff use one of theses options: -E --ignore-tab-expansion, -Z --ignore-trailing-space,
  -b --ignore-space-change,  -w --ignore-all-space, and/or  -B --ignore-blank-lines

With git-blame use the -w option.

With git do all format changes in separate commits, then add commit(s) to .ignoreRevsFile
so these format only commits do not show up in git blame. Best used when doing  mass formatting
changes to the source, then inforce formatting standards when doing new commits.


On 6/30/2021 5:14 AM, David Newall wrote:
> Hi John,
> 
> On 30/6/21 2:57 pm, John Cotton Ericson wrote:
>> The nature of the change is putting a decent chunk of existing autoconf m4 within
>> a newly-introduced if-then-else. That means either the indention will become wrong,
>> or there will be some churn/noise reindenting the old code.
>>
>> I chose to reindent first, making on odd 2x indent, and then add the new code and
>> if-then-else, also fixing the indent.
> 
> The problem is that a diff for a version from before your patch comparing with a version
> after will be noisy.  That's what I want you to avoid.
> 
>> A third option would be to simply do the first of those to patches, making the minimal
>> change and then leaving the indentation incorrect. 
> 
> That's what I think you should do.  Adding a comment at the if-then-else saying why the
> indent is wrong might help avoid flammage about wrongly indented code.
> 
> Regards,
> 
> David
> _______________________________________________
> openssh-unix-dev mailing list
> openssh-unix-dev at mindrot.org
> https://lists.mindrot.org/mailman/listinfo/openssh-unix-dev

-- 

  Douglas E. Engert  <DEEngert at gmail.com>



More information about the openssh-unix-dev mailing list