On Sat, Oct 12, 2024 at 2:07 AM Eric Sunshine [off-list ref] wrote:

On Sat, Oct 12, 2024 at 12:38 AM Abhijeetsingh Meena via GitGitGadget
[off-list ref] wrote:

quoted

+       /*
+       * By default, add .git-blame-ignore-revs to the list of files
+       * containing revisions to ignore if it exists.
+       */
+       if (access(".git-blame-ignore-revs", F_OK) == 0) {
+               string_list_append(&ignore_revs_file_list, ".git-blame-ignore-revs");
+       }

A couple style nits and a couple questions...

One other observation: The comment above this code block doesn't say
anything that isn't already stated just as clearly by the code itself.
Hence, the comment adds no value, thus should be dropped.