Re: [PATCH 1/2] Indent code before next channge

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]

 



On 30/6/21 5:42 am, list@xxxxxxxxxxxxxx wrote:
From: John Ericson <John.Ericson@Obsidian.Systems>

This hopefully makes for a better diff.
---
  configure.ac | 136 +++++++++++++++++++++++++--------------------------
  1 file changed, 68 insertions(+), 68 deletions(-)

I rely on diff to see what is different between two versions, and not necessarily consecutive versions.  It is an immensely effective technique for finding where
something became broken.

Diffs like that which John proposes add noise, dramatically increasing the effort required to see real changes.  Did he simply change indent or did he also change
text?  I can't tell at a glance.

I request that his diff be rejected.
_______________________________________________
openssh-unix-dev mailing list
openssh-unix-dev@xxxxxxxxxxx
https://lists.mindrot.org/mailman/listinfo/openssh-unix-dev




[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux