Re: [PATCH v3] diff: teach --stat to ignore uninteresting modifications
From: Junio C Hamano <hidden>
Date: 2020-08-20 00:56:32
"Matthew Rogers via GitGitGadget" [off-list ref] writes:
From: Matthew Rogers <redacted> When options such as --ignore-space-change are in use, files with modifications can have no interesting textual changes worth showing. In such cases, "git diff --stat" shows 0 lines of additions and deletions. Teach "git diff --stat" not to show such a path in its output, which would be more natural. However, we don't want to prevent the display of all files that have 0 effective diffs since they could be the result of a rename, permission change, or other similar operation that may still be of interest so we special case additions and deletions as they are always interesting. Signed-off-by: Matthew Rogers <redacted> ---
Looks good, thanks. Will queue. By the way, before making your commits, please make sure you do not have whitespace errors. I've let my "git am" to fix them, so no need to resend, but for future reference... .git/rebase-apply/patch:116: trailing whitespace. struct diffstat_file *file = .git/rebase-apply/patch:119: trailing whitespace. * Omit diffstats of modified files where nothing changed. .git/rebase-apply/patch:122: trailing whitespace. * .git/rebase-apply/patch:124: trailing whitespace. * renames, and mode changes as adding an empty file, .git/rebase-apply/patch:127: trailing whitespace. if ((p->status == DIFF_STATUS_MODIFIED) warning: 5 lines applied after fixing whitespace errors. Applying: diff: teach --stat to ignore uninteresting modifications